diff options
| author | Markus Elfring <elfring@users.sourceforge.net> | 2014-12-13 01:57:18 +0100 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-12-13 21:42:50 +0100 |
| commit | 442cc432e68b24dc6bd70b1ee1f5d477ee1aa385 (patch) | |
| tree | 51c430bb8e21b0e393cd5d94e1f9cd4d3005c6e7 /mm/zswap.c | |
| parent | mm/zswap: add __init to some functions in zswap (diff) | |
| download | linux-442cc432e68b24dc6bd70b1ee1f5d477ee1aa385.tar.xz linux-442cc432e68b24dc6bd70b1ee1f5d477ee1aa385.zip | |
mm/zswap: delete unnecessary check before calling free_percpu()
free_percpu() tests whether its argument is NULL and then returns
immediately. Thus the test around the call is not needed.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Cc: Seth Jennings <sjennings@variantweb.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/zswap.c')
| -rw-r--r-- | mm/zswap.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/mm/zswap.c b/mm/zswap.c index c162a2e6cc51..0cfce9bc51e4 100644 --- a/mm/zswap.c +++ b/mm/zswap.c @@ -152,8 +152,7 @@ static int __init zswap_comp_init(void) static void __init zswap_comp_exit(void) { /* free percpu transforms */ - if (zswap_comp_pcpu_tfms) - free_percpu(zswap_comp_pcpu_tfms); + free_percpu(zswap_comp_pcpu_tfms); } /********************************* |