From: Christoph Lameter Date: Wed, 1 Jun 2011 17:25:59 +0000 (-0500) Subject: slub: Not necessary to check for empty slab on load_freelist X-Git-Tag: upstream/snapshot3+hdmi~9422^2~6 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=4eade540fc35353813097bfdb39465c9b8847a15;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git slub: Not necessary to check for empty slab on load_freelist load_freelist is now only branched to only if there are objects available. So no need to check the object variable for NULL. Signed-off-by: Pekka Enberg --- diff --git a/mm/slub.c b/mm/slub.c index 25dac48..78c4882 100644 --- a/mm/slub.c +++ b/mm/slub.c @@ -1997,9 +1997,6 @@ static void *__slab_alloc(struct kmem_cache *s, gfp_t gfpflags, int node, NULL, new.counters, "__slab_alloc")); -load_freelist: - VM_BUG_ON(!page->frozen); - if (unlikely(!object)) { c->page = NULL; stat(s, DEACTIVATE_BYPASS); @@ -2008,6 +2005,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);