diff options
| author | Zhen Lei <thunder.leizhen@huawei.com> | 2023-06-06 14:55:43 +0800 |
|---|---|---|
| committer | Vlastimil Babka <vbabka@suse.cz> | 2023-06-06 10:37:19 +0200 |
| commit | b9dad156af1fd6c66ffa40f007c09823a8319abe (patch) | |
| tree | 84502ce8462fb620c89902a4eb0711baf17f62c5 /mm/slab_common.c | |
| parent | d2e527f0d8d1124b1fab93e2e7b2c6a114c0e5a2 (diff) | |
| download | lwn-b9dad156af1fd6c66ffa40f007c09823a8319abe.tar.gz lwn-b9dad156af1fd6c66ffa40f007c09823a8319abe.zip | |
mm/slab_common: reduce an if statement in create_cache()
Move the 'out:' statement block out of the successful path to avoid
redundant check on 'err'. The value of 'err' is always zero on success
and negative on failure.
No functional changes, no performance improvements, just a little more
readability.
Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
Reviewed-by: Hyeonggon Yoo <42.hyeyoo@gmail.com>
Signed-off-by: Vlastimil Babka <vbabka@suse.cz>
Diffstat (limited to 'mm/slab_common.c')
| -rw-r--r-- | mm/slab_common.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/mm/slab_common.c b/mm/slab_common.c index 607249785c07..f6fe35105774 100644 --- a/mm/slab_common.c +++ b/mm/slab_common.c @@ -236,14 +236,12 @@ static struct kmem_cache *create_cache(const char *name, s->refcount = 1; list_add(&s->list, &slab_caches); -out: - if (err) - return ERR_PTR(err); return s; out_free_cache: kmem_cache_free(kmem_cache, s); - goto out; +out: + return ERR_PTR(err); } /** |