Re: [PATCH] SLUB: clear c->freelist in __slab_alloc()/load_freelist:/SlabDebugpath

From: Benny Halevy
Date: Wed May 14 2008 - 13:55:19 EST


On May. 14, 2008, 10:44 -0700, Christoph Lameter <clameter@xxxxxxx> wrote:
> On Tue, 13 May 2008, Benny Halevy wrote:
>
>>> But for debug pages, we never load c->page->freelist to c->freelist so
>>> it should always be NULL.
>> Hmm, I see. Then it might have got corrupted...
>> I'll keep looking for the root cause.
>
> Correct.

Yeah, I've moved to SLAB and the mem corruption now pops up at a different
place.

>
>>> How did you make SLUB hit the debug path since you have
>>> CONFIG_SLUB_DEBUG_ON disabled?
>
> I guess he passed slub_debug on the kernel command line.
>

I did not.

I probably have misunderstood how the slub debugging infrastructure works
and did not execute the debug path at all.

Thanks for your help!

Benny
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/