Re: [PATCH] slab: fix double-free in fallback_alloc

From: Hugh Dickins
Date: Fri Jan 05 2007 - 15:46:41 EST


On Fri, 5 Jan 2007, Pekka J Enberg wrote:
>
> Here's an alternative fix for the double-free bug you hit. I have only
> compile-tested this on NUMA so can you please confirm it fixes the problem
> for you? Thanks.

It looks nice, and I'm giving it a spin: though hardly worth waiting
a week or so to see if it hits the old double-free (took 3.5 days of
load when I saw it before), since you've fairly clearly rearranged
that out of existence - any bugs will be new ones all your own ;)

And I don't really have any NUMA anyway, just build that way ever
since testing some mempolicy change. So don't pay much attention to
my results, though of course I'll let you know if I see it go wrong.

The main thing is to make sure that your patch doesn't end up applied
to a tree with my patch in, since that would then be a (rare) leak.

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