diff options
author | Zhao Jin <cronozhj@gmail.com> | 2011-08-26 12:26:17 -0400 |
---|---|---|
committer | Pekka Enberg <penberg@kernel.org> | 2012-01-23 08:32:26 -0500 |
commit | 42c8c99cd891184bf4bcf6f09d62c54e42599453 (patch) | |
tree | 436001fd4cd0aba57c67164d242490a2fbefaa1a /mm | |
parent | dcd6c92267155e70a94b3927bce681ce74b80d1f (diff) |
slab, cleanup: remove unneeded return
The procedure ends right after the if-statement, so remove ``return''.
Also move the last common statement outside.
Signed-off-by: Zhao Jin <cronozhj@gmail.com>
Acked-by: David Rientjes <rientjes@google.com>
Acked-by: Christoph Lameter <cl@linux.com>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Diffstat (limited to 'mm')
-rw-r--r-- | mm/slab.c | 5 |
1 files changed, 2 insertions, 3 deletions
@@ -3693,13 +3693,12 @@ static inline void __cache_free(struct kmem_cache *cachep, void *objp, | |||
3693 | 3693 | ||
3694 | if (likely(ac->avail < ac->limit)) { | 3694 | if (likely(ac->avail < ac->limit)) { |
3695 | STATS_INC_FREEHIT(cachep); | 3695 | STATS_INC_FREEHIT(cachep); |
3696 | ac->entry[ac->avail++] = objp; | ||
3697 | return; | ||
3698 | } else { | 3696 | } else { |
3699 | STATS_INC_FREEMISS(cachep); | 3697 | STATS_INC_FREEMISS(cachep); |
3700 | cache_flusharray(cachep, ac); | 3698 | cache_flusharray(cachep, ac); |
3701 | ac->entry[ac->avail++] = objp; | ||
3702 | } | 3699 | } |
3700 | |||
3701 | ac->entry[ac->avail++] = objp; | ||
3703 | } | 3702 | } |
3704 | 3703 | ||
3705 | /** | 3704 | /** |