On Wed, 11 Jun 2008 06:49:02 +0200 Nick Piggin <npiggin@suse.de> wrote:
No, slab defreg[*] isn't in linux-next.
y:/usr/src/25> diffstat patches/linux-next.patch| grep mm/slub.c
mm/slub.c | 4
That's two spelling fixes in comments.
I have git-pekka in -mm too. Here it is:
--- a/mm/slub.c
+++ b/mm/slub.c
@@ -2765,6 +2765,7 @@ void kfree(const void *x)
page = virt_to_head_page(x);
if (unlikely(!PageSlab(page))) {
+ BUG_ON(!PageCompound(page));
put_page(page);
return;
}
I think I'd prefer that. I may be a bit slow, but we're shoving at
least 100 MM patches through each kernel release and I think I review
things more closely than others choose to. At least, I find problems
and I've seen some pretty wild acked-bys...
[*] It _isn't_ "slab defrag". Or at least, it wasn't last time I saw
it. It's "slub defrag". And IMO it is bad to be adding slub-only
features because afaik slub still isn't as fast as slab on some things
and so some people might want to run slab rather than slub. And
because if this the decision whether to retain slab or slub STILL
hasn't been made. Carrying both versions was supposed to be a
short-term transitional thing :(
--
