Dec 13 2014

mm/zswap: delete unnecessary check before calling free_percpu() 

Author: Markus Elfring <elfring@users.sourceforge.net>

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 
Cc: Seth Jennings 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds 

---
 mm/zswap.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
 
diff --git a/mm/zswap.c b/mm/zswap.c
index c162a2e..0cfce9b 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);
 }
 
 /*********************************

qlambert