diff options
author | Christoph Lameter <cl@linux.com> | 2012-05-09 10:09:55 -0500 |
---|---|---|
committer | Pekka Enberg <penberg@kernel.org> | 2012-06-01 09:25:40 +0300 |
commit | 188fd063208942a4681d8e8a4484ad0d4ae0fda1 (patch) | |
tree | d00d4b5884aa9ecb381be41220fb2b42c0a4c27a | |
parent | f469743673ceda5181970eb6b8090ba728c956fb (diff) | |
download | op-kernel-dev-188fd063208942a4681d8e8a4484ad0d4ae0fda1.zip op-kernel-dev-188fd063208942a4681d8e8a4484ad0d4ae0fda1.tar.gz |
slub: new_slab_objects() can also get objects from partial list
Moving the attempt to get a slab page from the partial lists simplifies
__slab_alloc which is rather complicated.
Signed-off-by: Christoph Lameter <cl@linux.com>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
-rw-r--r-- | mm/slub.c | 16 |
1 files changed, 9 insertions, 7 deletions
@@ -2130,9 +2130,15 @@ static inline void *new_slab_objects(struct kmem_cache *s, gfp_t flags, int node, struct kmem_cache_cpu **pc) { void *freelist; - struct kmem_cache_cpu *c; - struct page *page = new_slab(s, flags, node); + struct kmem_cache_cpu *c = *pc; + struct page *page; + + freelist = get_partial(s, flags, node, c); + if (freelist) + return freelist; + + page = new_slab(s, flags, node); if (page) { c = __this_cpu_ptr(s->cpu_slab); if (c->page) @@ -2269,11 +2275,7 @@ new_slab: goto redo; } - /* Then do expensive stuff like retrieving pages from the partial lists */ - freelist = get_partial(s, gfpflags, node, c); - - if (!freelist) - freelist = new_slab_objects(s, gfpflags, node, &c); + freelist = new_slab_objects(s, gfpflags, node, &c); if (unlikely(!freelist)) { if (!(gfpflags & __GFP_NOWARN) && printk_ratelimit()) |