diff options
author | Pavel Emelyanov <xemul@parallels.com> | 2010-10-28 11:50:37 +0200 |
---|---|---|
committer | Pekka Enberg <penberg@kernel.org> | 2010-11-14 15:53:11 +0100 |
commit | 68cee4f118c21a1c67e5764a91d766661db5b360 (patch) | |
tree | 17c487ce969cc66a4be5048ed12cb5ec070c2fc2 /mm/slub.c | |
parent | Linux 2.6.37-rc1 (diff) | |
download | linux-68cee4f118c21a1c67e5764a91d766661db5b360.tar.xz linux-68cee4f118c21a1c67e5764a91d766661db5b360.zip |
slub: Fix slub_lock down/up imbalance
There are two places, that do not release the slub_lock.
Respective bugs were introduced by sysfs changes ab4d5ed5 (slub: Enable
sysfs support for !CONFIG_SLUB_DEBUG) and 2bce6485 ( slub: Allow removal
of slab caches during boot).
Acked-by: Christoph Lameter <cl@linux.com>
Signed-off-by: Pavel Emelyanov <xemul@openvz.org>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Diffstat (limited to 'mm/slub.c')
-rw-r--r-- | mm/slub.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/mm/slub.c b/mm/slub.c index 8fd5401bb071..981fb730aa04 100644 --- a/mm/slub.c +++ b/mm/slub.c @@ -3273,9 +3273,9 @@ struct kmem_cache *kmem_cache_create(const char *name, size_t size, kfree(n); kfree(s); } +err: up_write(&slub_lock); -err: if (flags & SLAB_PANIC) panic("Cannot create slabcache %s\n", name); else @@ -3862,6 +3862,7 @@ static ssize_t show_slab_objects(struct kmem_cache *s, x += sprintf(buf + x, " N%d=%lu", node, nodes[node]); #endif + up_read(&slub_lock); kfree(nodes); return x + sprintf(buf + x, "\n"); } |