Re: [PATCH] mm/slab.c: check pointer slabp before using it in alloc_slabmgmt()

2013-12-14 Thread Meelis Roos
> > On Tue, 10 Dec 2013, Ethan Zhao wrote: > > > > > No, stable 3.12.4 and 3.12.3 need this patch. 3.13RC doesn't > > > need it anymore. > > > > Hmmm.. Then this commit may have fixed it: > > Cross-threaded from [Re: Slab BUG with DEBUG_* options ] > > On 12/08/2013 10:00 AM, Meelis

Re: [PATCH] mm/slab.c: check pointer slabp before using it in alloc_slabmgmt()

2013-12-14 Thread Meelis Roos
On Tue, 10 Dec 2013, Ethan Zhao wrote: No, stable 3.12.4 and 3.12.3 need this patch. 3.13RC doesn't need it anymore. Hmmm.. Then this commit may have fixed it: Cross-threaded from [Re: Slab BUG with DEBUG_* options ] On 12/08/2013 10:00 AM, Meelis Roos wrote: ldata

Re: [PATCH] mm/slab.c: check pointer slabp before using it in alloc_slabmgmt()

2013-12-10 Thread Peter Hurley
On 12/10/2013 10:35 AM, Christoph Lameter wrote: On Tue, 10 Dec 2013, Ethan Zhao wrote: No, stable 3.12.4 and 3.12.3 need this patch. 3.13RC doesn't need it anymore. Hmmm.. Then this commit may have fixed it: Cross-threaded from [Re: Slab BUG with DEBUG_* options ] On 12/08/2013

Re: [PATCH] mm/slab.c: check pointer slabp before using it in alloc_slabmgmt()

2013-12-10 Thread Christoph Lameter
On Sun, 8 Dec 2013, ethan.zhao wrote: > Move the NULL check of slabp to the right place before refer its memeber in > function alloc_slabmgmt(). Ok this patch is needed for 2.6.30 to 3.12 Acked-by: Christoph Lameter -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in

Re: [PATCH] mm/slab.c: check pointer slabp before using it in alloc_slabmgmt()

2013-12-10 Thread Christoph Lameter
On Tue, 10 Dec 2013, Ethan Zhao wrote: > No, stable 3.12.4 and 3.12.3 need this patch. 3.13RC doesn't > need it anymore. Hmmm.. Then this commit may have fixed it: commit 0172f779e4314639a8ed440082cfe9e3450954e8 Author: Joonsoo Kim Date: Wed Oct 30 19:04:00 2013 +0900 slab: fix to

Re: [PATCH] mm/slab.c: check pointer slabp before using it in alloc_slabmgmt()

2013-12-10 Thread Ethan Zhao
Christoph, No, stable 3.12.4 and 3.12.3 need this patch. 3.13RC doesn't need it anymore. Thanks, Ethan On Tue, Dec 10, 2013 at 3:08 PM, Ethan Zhao wrote: > Christoph, > Found in the latest stable release V3.12.3, yes, changed in > 3.12.4. not needed for later release anymore. > >

Re: [PATCH] mm/slab.c: check pointer slabp before using it in alloc_slabmgmt()

2013-12-10 Thread Ethan Zhao
Christoph, No, stable 3.12.4 and 3.12.3 need this patch. 3.13RC doesn't need it anymore. Thanks, Ethan On Tue, Dec 10, 2013 at 3:08 PM, Ethan Zhao ethan.ker...@gmail.com wrote: Christoph, Found in the latest stable release V3.12.3, yes, changed in 3.12.4. not needed for later

Re: [PATCH] mm/slab.c: check pointer slabp before using it in alloc_slabmgmt()

2013-12-10 Thread Christoph Lameter
On Tue, 10 Dec 2013, Ethan Zhao wrote: No, stable 3.12.4 and 3.12.3 need this patch. 3.13RC doesn't need it anymore. Hmmm.. Then this commit may have fixed it: commit 0172f779e4314639a8ed440082cfe9e3450954e8 Author: Joonsoo Kim iamjoonsoo@lge.com Date: Wed Oct 30 19:04:00 2013

Re: [PATCH] mm/slab.c: check pointer slabp before using it in alloc_slabmgmt()

2013-12-10 Thread Christoph Lameter
On Sun, 8 Dec 2013, ethan.zhao wrote: Move the NULL check of slabp to the right place before refer its memeber in function alloc_slabmgmt(). Ok this patch is needed for 2.6.30 to 3.12 Acked-by: Christoph Lameter c...@linux.com -- To unsubscribe from this list: send the line unsubscribe

Re: [PATCH] mm/slab.c: check pointer slabp before using it in alloc_slabmgmt()

2013-12-10 Thread Peter Hurley
On 12/10/2013 10:35 AM, Christoph Lameter wrote: On Tue, 10 Dec 2013, Ethan Zhao wrote: No, stable 3.12.4 and 3.12.3 need this patch. 3.13RC doesn't need it anymore. Hmmm.. Then this commit may have fixed it: Cross-threaded from [Re: Slab BUG with DEBUG_* options ] On 12/08/2013

Re: [PATCH] mm/slab.c: check pointer slabp before using it in alloc_slabmgmt()

2013-12-09 Thread Ethan Zhao
Christoph, Found in the latest stable release V3.12.3, yes, changed in 3.12.4. not needed for later release anymore. Thanks, Ethan On Tue, Dec 10, 2013 at 12:11 AM, Christoph Lameter wrote: > On Sun, 8 Dec 2013, ethan.zhao wrote: > >> Move the NULL check of slabp to the right place before

Re: [PATCH] mm/slab.c: check pointer slabp before using it in alloc_slabmgmt()

2013-12-09 Thread Christoph Lameter
On Sun, 8 Dec 2013, ethan.zhao wrote: > Move the NULL check of slabp to the right place before refer its memeber in > function alloc_slabmgmt(). I am having trouble to find the code you are modifying. Which kernel release is this? The code you are referring to has been rewritten in the meantime

Re: [PATCH] mm/slab.c: check pointer slabp before using it in alloc_slabmgmt()

2013-12-09 Thread Christoph Lameter
On Sun, 8 Dec 2013, ethan.zhao wrote: Move the NULL check of slabp to the right place before refer its memeber in function alloc_slabmgmt(). I am having trouble to find the code you are modifying. Which kernel release is this? The code you are referring to has been rewritten in the meantime or

Re: [PATCH] mm/slab.c: check pointer slabp before using it in alloc_slabmgmt()

2013-12-09 Thread Ethan Zhao
Christoph, Found in the latest stable release V3.12.3, yes, changed in 3.12.4. not needed for later release anymore. Thanks, Ethan On Tue, Dec 10, 2013 at 12:11 AM, Christoph Lameter c...@linux.com wrote: On Sun, 8 Dec 2013, ethan.zhao wrote: Move the NULL check of slabp to the right

[PATCH] mm/slab.c: check pointer slabp before using it in alloc_slabmgmt()

2013-12-08 Thread ethan.zhao
Move the NULL check of slabp to the right place before refer its memeber in function alloc_slabmgmt(). This bug may be introduced by rewriting of funcion kmemleak_scan_area(), the first parameter changed from slabp to >list. Signed-off-by: ethan.zhao --- mm/slab.c | 4 ++-- 1 file changed, 2

[PATCH] mm/slab.c: check pointer slabp before using it in alloc_slabmgmt()

2013-12-08 Thread ethan.zhao
Move the NULL check of slabp to the right place before refer its memeber in function alloc_slabmgmt(). This bug may be introduced by rewriting of funcion kmemleak_scan_area(), the first parameter changed from slabp to slabp-list. Signed-off-by: ethan.zhao ethan.ker...@gmail.com --- mm/slab.c |