LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH] slub: look up object from the freelist once
@ 2008-02-29  9:07 Pekka J Enberg
  2008-02-29 19:44 ` Christoph Lameter
  0 siblings, 1 reply; 2+ messages in thread
From: Pekka J Enberg @ 2008-02-29  9:07 UTC (permalink / raw)
  To: clameter; +Cc: linux-kernel

From: Pekka Enberg <penberg@cs.helsinki.fi>

We only need to look up object from c->page->freelist once in 
__slab_alloc().

Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
---
 mm/slub.c |    2 --
 1 file changed, 2 deletions(-)

Index: linux-2.6/mm/slub.c
===================================================================
--- linux-2.6.orig/mm/slub.c
+++ linux-2.6/mm/slub.c
@@ -1522,7 +1522,6 @@ load_freelist:
 	if (unlikely(SlabDebug(c->page)))
 		goto debug;
 
-	object = c->page->freelist;
 	c->freelist = object[c->offset];
 	c->page->inuse = s->objects;
 	c->page->freelist = c->page->end;
@@ -1578,7 +1577,6 @@ new_slab:
 
 	return NULL;
 debug:
-	object = c->page->freelist;
 	if (!alloc_debug_processing(s, c->page, object, addr))
 		goto another_slab;
 

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] slub: look up object from the freelist once
  2008-02-29  9:07 [PATCH] slub: look up object from the freelist once Pekka J Enberg
@ 2008-02-29 19:44 ` Christoph Lameter
  0 siblings, 0 replies; 2+ messages in thread
From: Christoph Lameter @ 2008-02-29 19:44 UTC (permalink / raw)
  To: Pekka J Enberg; +Cc: linux-kernel

Ok. Will queue for 2.6.25.


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2008-02-29 19:44 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-02-29  9:07 [PATCH] slub: look up object from the freelist once Pekka J Enberg
2008-02-29 19:44 ` Christoph Lameter

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).