diff options
author | Christoph Lameter <cl@linux.com> | 2011-03-15 13:45:21 -0400 |
---|---|---|
committer | Pekka Enberg <penberg@kernel.org> | 2011-03-20 12:11:07 -0400 |
commit | a24c5a0ea902bcda348f086bd909cc2d6e305bf8 (patch) | |
tree | 82ac0b5bdc810b6f427261b8b254ac2b05023f56 | |
parent | 8a5ec0ba42c4919e2d8f4c3138cc8b987fdb0b79 (diff) |
slub: Dont define useless label in the !CONFIG_CMPXCHG_LOCAL case
The redo label needs #ifdeffery. Fixes the following problem introduced by
commit 8a5ec0ba42c4 ("Lockless (and preemptless) fastpaths for slub"):
mm/slub.c: In function 'slab_free':
mm/slub.c:2124: warning: label 'redo' defined but not used
Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: Christoph Lameter <cl@linux.com>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
-rw-r--r-- | mm/slub.c | 4 |
1 files changed, 3 insertions, 1 deletions
@@ -2073,9 +2073,11 @@ static __always_inline void slab_free(struct kmem_cache *s, | |||
2073 | 2073 | ||
2074 | #ifndef CONFIG_CMPXCHG_LOCAL | 2074 | #ifndef CONFIG_CMPXCHG_LOCAL |
2075 | local_irq_save(flags); | 2075 | local_irq_save(flags); |
2076 | #endif | ||
2077 | 2076 | ||
2077 | #else | ||
2078 | redo: | 2078 | redo: |
2079 | #endif | ||
2080 | |||
2079 | /* | 2081 | /* |
2080 | * Determine the currently cpus per cpu slab. | 2082 | * Determine the currently cpus per cpu slab. |
2081 | * The cpu may change afterward. However that does not matter since | 2083 | * The cpu may change afterward. However that does not matter since |