mm/zswap: delete unnecessary check before calling free_percpu()
authorMarkus Elfring <elfring@users.sourceforge.net>
Sat, 13 Dec 2014 00:57:18 +0000 (16:57 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Sat, 13 Dec 2014 20:42:50 +0000 (12:42 -0800)
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>
mm/zswap.c

index c162a2e6cc51ba72f44184d5af89082721ab5c65..0cfce9bc51e4946c561e48b8cb5f9e6cf7669d5f 100644 (file)
@@ -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);
 }
 
 /*********************************