Re: [PATCH V2] mm/slab: add a leak decoder callback

2013-01-17 Thread Joonsoo Kim
Hello, Liu Bo. On Wed, Jan 16, 2013 at 11:03:13AM +0800, Liu Bo wrote: This adds a leak decoder callback so that slab destruction can use to generate debugging output for the allocated objects. Callers like btrfs are using their own leak tracking which will manage allocated objects in a

Re: [PATCH V2] mm/slab: add a leak decoder callback

2013-01-17 Thread Liu Bo
On Thu, Jan 17, 2013 at 05:34:46PM +0900, Joonsoo Kim wrote: Hello, Liu Bo. On Wed, Jan 16, 2013 at 11:03:13AM +0800, Liu Bo wrote: This adds a leak decoder callback so that slab destruction can use to generate debugging output for the allocated objects. Callers like btrfs are using

Re: [PATCH V2] mm/slab: add a leak decoder callback

2013-01-17 Thread Liu Bo
On Wed, Jan 16, 2013 at 03:20:57PM +, Christoph Lameter wrote: On Wed, 16 Jan 2013, Liu Bo wrote: --- a/include/linux/slub_def.h +++ b/include/linux/slub_def.h @@ -93,6 +93,7 @@ struct kmem_cache { gfp_t allocflags; /* gfp flags to use on each alloc */ int refcount;

Re: [PATCH V2] mm/slab: add a leak decoder callback

2013-01-17 Thread Liu Bo
On Wed, Jan 16, 2013 at 01:34:38PM +0800, Miao Xie wrote: On wed, 16 Jan 2013 11:03:13 +0800, Liu Bo wrote: This adds a leak decoder callback so that slab destruction can use to generate debugging output for the allocated objects. Callers like btrfs are using their own leak tracking

Re: [PATCH V2] mm/slab: add a leak decoder callback

2013-01-16 Thread Christoph Lameter
On Wed, 16 Jan 2013, Liu Bo wrote: --- a/include/linux/slub_def.h +++ b/include/linux/slub_def.h @@ -93,6 +93,7 @@ struct kmem_cache { gfp_t allocflags; /* gfp flags to use on each alloc */ int refcount; /* Refcount for slab cache destroy */ void

[PATCH V2] mm/slab: add a leak decoder callback

2013-01-15 Thread Liu Bo
This adds a leak decoder callback so that slab destruction can use to generate debugging output for the allocated objects. Callers like btrfs are using their own leak tracking which will manage allocated objects in a list(or something else), this does indeed the same thing as what slab does. So

Re: [PATCH V2] mm/slab: add a leak decoder callback

2013-01-15 Thread Miao Xie
On wed, 16 Jan 2013 11:03:13 +0800, Liu Bo wrote: This adds a leak decoder callback so that slab destruction can use to generate debugging output for the allocated objects. Callers like btrfs are using their own leak tracking which will manage allocated objects in a list(or something else),