Re: [bug] SLUB & freeing locks [was: Re: 2.6.23-rc1: known regressionswith patches]

From: Christoph Lameter
Date: Thu Jul 26 2007 - 13:47:43 EST


On Thu, 26 Jul 2007, Pekka Enberg wrote:

> On 7/26/07, Peter Zijlstra <peterz@xxxxxxxxxxxxx> wrote:
> > --- linux-2.6.orig/mm/slub.c
> > +++ linux-2.6/mm/slub.c
> > @@ -1656,6 +1656,7 @@ static void __always_inline slab_free(st
> > unsigned long flags;
> >
> > local_irq_save(flags);
> > + debug_check_no_locks_freed(object, s->size);
>
> Although it doesn't matter much, I think this should be s->objsize. Christoph?

Correct. Locks can only be in the first objsize range of bytes.


-
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/