[slubllv6 16/17] slub: Not necessary to check for empty slab on load_freelist
From: Christoph Lameter
Date: Thu May 26 2011 - 14:15:55 EST
load_freelist is now only branched to only if there are objects available.
So no need to check the object variable for NULL.
---
mm/slub.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
Index: linux-2.6/mm/slub.c
===================================================================
--- linux-2.6.orig/mm/slub.c 2011-05-24 09:41:11.294874943 -0500
+++ linux-2.6/mm/slub.c 2011-05-24 09:41:15.454874919 -0500
@@ -1983,9 +1983,6 @@ static void *__slab_alloc(struct kmem_ca
NULL, new.counters,
"__slab_alloc"));
-load_freelist:
- VM_BUG_ON(!page->frozen);
-
if (unlikely(!object)) {
c->page = NULL;
stat(s, DEACTIVATE_BYPASS);
@@ -1994,6 +1991,8 @@ load_freelist:
stat(s, ALLOC_REFILL);
+load_freelist:
+ VM_BUG_ON(!page->frozen);
c->freelist = get_freepointer(s, object);
c->tid = next_tid(c->tid);
local_irq_restore(flags);
--
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/