Re: [PATCH] mm/slab: Fix kmem_cache_alloc_node_trace() declaration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 25 Sep 2012, Ezequiel Garcia wrote:

> The bug was introduced in commit 4052147c0afa
> "mm, slab: Match SLAB and SLUB kmem_cache_alloc_xxx_trace() prototype".
> 

This isn't a candidate for kernel-janitors@xxxxxxxxxxxxxxx, these are 
patches that are one of Pekka's branches and would never make it to Linus' 
tree in this form.

> Cc: Pekka Enberg <penberg@xxxxxxxxxx>
> Reported-by: Fengguang Wu <fengguang.wu@xxxxxxxxx>
> Signed-off-by: Ezequiel Garcia <elezegarcia@xxxxxxxxx>

Acked-by: David Rientjes <rientjes@xxxxxxxxxx>

So now we have this for SLAB:

extern void *kmem_cache_alloc_node_trace(size_t size,
					 struct kmem_cache *cachep,
					 gfp_t flags,
					 int nodeid);

and this for SLUB:

extern void *kmem_cache_alloc_node_trace(struct kmem_cache *s,
					 gfp_t gfpflags,
					 int node, size_t size);

Would you like to send a follow-up patch to make these the same?  (My 
opinion is that the SLUB variant is the correct order.)
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux