diff options
author | Alex,Shi <alex.shi@intel.com> | 2011-09-07 10:26:36 +0800 |
---|---|---|
committer | Pekka Enberg <penberg@kernel.org> | 2011-09-13 20:41:25 +0300 |
commit | 12d79634f8d7af5229b7d21143d50e7cf7d94177 (patch) | |
tree | 06096e3fdcc4c7fe84b9111d2f6274514bfb72b0 /mm/slub.c | |
parent | aca726a07a71ff7aedc0e90a91f80a2701adcca5 (diff) | |
download | op-kernel-dev-12d79634f8d7af5229b7d21143d50e7cf7d94177.zip op-kernel-dev-12d79634f8d7af5229b7d21143d50e7cf7d94177.tar.gz |
slub: Code optimization in get_partial_node()
I find a way to reduce a variable in get_partial_node(). That is also helpful
for code understanding.
Acked-by: Christoph Lameter <cl@linux.com>
Signed-off-by: Alex Shi <alex.shi@intel.com>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Diffstat (limited to 'mm/slub.c')
-rw-r--r-- | mm/slub.c | 6 |
1 files changed, 2 insertions, 4 deletions
@@ -1600,7 +1600,6 @@ static void *get_partial_node(struct kmem_cache *s, { struct page *page, *page2; void *object = NULL; - int count = 0; /* * Racy check. If we mistakenly see no partial slabs then we @@ -1613,17 +1612,16 @@ static void *get_partial_node(struct kmem_cache *s, spin_lock(&n->list_lock); list_for_each_entry_safe(page, page2, &n->partial, lru) { - void *t = acquire_slab(s, n, page, count == 0); + void *t = acquire_slab(s, n, page, object == NULL); int available; if (!t) break; - if (!count) { + if (!object) { c->page = page; c->node = page_to_nid(page); stat(s, ALLOC_FROM_PARTIAL); - count++; object = t; available = page->objects - page->inuse; } else { |