RE: [PATCH] slub/slab: fix kmemleak didn't work on some case

2015-06-11 Thread Liu, XinwuX
une 10, 2015 5:49 PM To: Zhang, Yanmin Cc: Christoph Lameter; Liu, XinwuX; penb...@kernel.org; m...@selenic.com; linux...@kvack.org; linux-kernel@vger.kernel.org; He, Bo; Chen, Lin Z Subject: Re: [PATCH] slub/slab: fix kmemleak didn't work on some case On Wed, Jun 10, 2015 at 08:45:50AM +0100, Zhang,

RE: [PATCH] slub/slab: fix kmemleak didn't work on some case

2015-06-11 Thread Liu, XinwuX
...@kernel.org; m...@selenic.com; linux...@kvack.org; linux-kernel@vger.kernel.org; He, Bo; Chen, Lin Z Subject: Re: [PATCH] slub/slab: fix kmemleak didn't work on some case On Wed, Jun 10, 2015 at 08:45:50AM +0100, Zhang, Yanmin wrote: On 2015/6/9 23:03, Catalin Marinas wrote: On Tue, Jun 09, 2015 at 09

Re: [PATCH] slub/slab: fix kmemleak didn't work on some case

2015-06-10 Thread Catalin Marinas
On Wed, Jun 10, 2015 at 08:45:50AM +0100, Zhang, Yanmin wrote: > On 2015/6/9 23:03, Catalin Marinas wrote: > > On Tue, Jun 09, 2015 at 09:10:45AM +0100, Zhang, Yanmin wrote: > >> On 2015/6/8 18:13, Catalin Marinas wrote: > >>> As I replied already, I don't think this is that bad, or at least not >

Re: [PATCH] slub/slab: fix kmemleak didn't work on some case

2015-06-10 Thread Zhang, Yanmin
On 2015/6/9 23:03, Catalin Marinas wrote: > On Tue, Jun 09, 2015 at 09:10:45AM +0100, Zhang, Yanmin wrote: >> On 2015/6/8 18:13, Catalin Marinas wrote: >>> As I replied already, I don't think this is that bad, or at least not >>> worse than what kmemleak already does (looking at all data whether

Re: [PATCH] slub/slab: fix kmemleak didn't work on some case

2015-06-10 Thread Catalin Marinas
On Wed, Jun 10, 2015 at 08:45:50AM +0100, Zhang, Yanmin wrote: On 2015/6/9 23:03, Catalin Marinas wrote: On Tue, Jun 09, 2015 at 09:10:45AM +0100, Zhang, Yanmin wrote: On 2015/6/8 18:13, Catalin Marinas wrote: As I replied already, I don't think this is that bad, or at least not worse

Re: [PATCH] slub/slab: fix kmemleak didn't work on some case

2015-06-10 Thread Zhang, Yanmin
On 2015/6/9 23:03, Catalin Marinas wrote: On Tue, Jun 09, 2015 at 09:10:45AM +0100, Zhang, Yanmin wrote: On 2015/6/8 18:13, Catalin Marinas wrote: As I replied already, I don't think this is that bad, or at least not worse than what kmemleak already does (looking at all data whether it's

Re: [PATCH] slub/slab: fix kmemleak didn't work on some case

2015-06-09 Thread Catalin Marinas
On Tue, Jun 09, 2015 at 09:10:45AM +0100, Zhang, Yanmin wrote: > On 2015/6/8 18:13, Catalin Marinas wrote: > > As I replied already, I don't think this is that bad, or at least not > > worse than what kmemleak already does (looking at all data whether it's > > pointer or not). > > It depends. As

Re: [PATCH] slub/slab: fix kmemleak didn't work on some case

2015-06-09 Thread Zhang, Yanmin
On 2015/6/8 18:13, Catalin Marinas wrote: > On Mon, Jun 08, 2015 at 10:38:13AM +0100, Christoph Lameter wrote: >> On Mon, 8 Jun 2015, Liu, XinwuX wrote: >> >>> when kernel uses kmalloc to allocate memory, slub/slab will find >>> a suitable kmem_cache. Ususally the cache's object size is often >>>

Re: [PATCH] slub/slab: fix kmemleak didn't work on some case

2015-06-09 Thread Zhang, Yanmin
On 2015/6/8 18:13, Catalin Marinas wrote: On Mon, Jun 08, 2015 at 10:38:13AM +0100, Christoph Lameter wrote: On Mon, 8 Jun 2015, Liu, XinwuX wrote: when kernel uses kmalloc to allocate memory, slub/slab will find a suitable kmem_cache. Ususally the cache's object size is often greater than

Re: [PATCH] slub/slab: fix kmemleak didn't work on some case

2015-06-09 Thread Catalin Marinas
On Tue, Jun 09, 2015 at 09:10:45AM +0100, Zhang, Yanmin wrote: On 2015/6/8 18:13, Catalin Marinas wrote: As I replied already, I don't think this is that bad, or at least not worse than what kmemleak already does (looking at all data whether it's pointer or not). It depends. As for

Re: [PATCH] slub/slab: fix kmemleak didn't work on some case

2015-06-08 Thread Catalin Marinas
On Mon, Jun 08, 2015 at 10:38:13AM +0100, Christoph Lameter wrote: > On Mon, 8 Jun 2015, Liu, XinwuX wrote: > > > when kernel uses kmalloc to allocate memory, slub/slab will find > > a suitable kmem_cache. Ususally the cache's object size is often > > greater than requested size. There is unused

Re: [PATCH] slub/slab: fix kmemleak didn't work on some case

2015-06-08 Thread Catalin Marinas
On Mon, Jun 08, 2015 at 06:14:32AM +0100, Liu, XinwuX wrote: > when kernel uses kmalloc to allocate memory, slub/slab will find > a suitable kmem_cache. Ususally the cache's object size is often > greater than requested size. There is unused space which contains > dirty data. These dirty data

Re: [PATCH] slub/slab: fix kmemleak didn't work on some case

2015-06-08 Thread Christoph Lameter
On Mon, 8 Jun 2015, Liu, XinwuX wrote: > when kernel uses kmalloc to allocate memory, slub/slab will find > a suitable kmem_cache. Ususally the cache's object size is often > greater than requested size. There is unused space which contains > dirty data. These dirty data might have pointers

Re: [PATCH] slub/slab: fix kmemleak didn't work on some case

2015-06-08 Thread Catalin Marinas
On Mon, Jun 08, 2015 at 06:14:32AM +0100, Liu, XinwuX wrote: when kernel uses kmalloc to allocate memory, slub/slab will find a suitable kmem_cache. Ususally the cache's object size is often greater than requested size. There is unused space which contains dirty data. These dirty data might

Re: [PATCH] slub/slab: fix kmemleak didn't work on some case

2015-06-08 Thread Catalin Marinas
On Mon, Jun 08, 2015 at 10:38:13AM +0100, Christoph Lameter wrote: On Mon, 8 Jun 2015, Liu, XinwuX wrote: when kernel uses kmalloc to allocate memory, slub/slab will find a suitable kmem_cache. Ususally the cache's object size is often greater than requested size. There is unused space

Re: [PATCH] slub/slab: fix kmemleak didn't work on some case

2015-06-08 Thread Christoph Lameter
On Mon, 8 Jun 2015, Liu, XinwuX wrote: when kernel uses kmalloc to allocate memory, slub/slab will find a suitable kmem_cache. Ususally the cache's object size is often greater than requested size. There is unused space which contains dirty data. These dirty data might have pointers pointing