diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2014-12-12 19:57:18 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-12-13 15:42:50 -0500 |
commit | 442cc432e68b24dc6bd70b1ee1f5d477ee1aa385 (patch) | |
tree | 51c430bb8e21b0e393cd5d94e1f9cd4d3005c6e7 /mm | |
parent | dd01d7d89a0c492821c6db005c21e68258a833fc (diff) |
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')
-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) | |||
152 | static void __init zswap_comp_exit(void) | 152 | static void __init zswap_comp_exit(void) |
153 | { | 153 | { |
154 | /* free percpu transforms */ | 154 | /* free percpu transforms */ |
155 | if (zswap_comp_pcpu_tfms) | 155 | free_percpu(zswap_comp_pcpu_tfms); |
156 | free_percpu(zswap_comp_pcpu_tfms); | ||
157 | } | 156 | } |
158 | 157 | ||
159 | /********************************* | 158 | /********************************* |