diff options
author | Julia Lawall <Julia.Lawall@lip6.fr> | 2015-11-06 19:31:40 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2015-11-06 20:50:42 -0500 |
commit | da80a39fc962ceca085ddfb7d63e00309b305f17 (patch) | |
tree | cff55838f1c36465c74682251e5d4a1881f602e4 /fs/nilfs2/super.c | |
parent | 6d32f7a391466ed89d8a1018fdff5330766cc272 (diff) |
nilfs2: drop null test before destroy functions
Remove unneeded NULL test.
The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)
// <smpl>
@@ expression x; @@
-if (x != NULL)
\(kmem_cache_destroy\|mempool_destroy\|dma_pool_destroy\)(x);
// </smpl>
Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/nilfs2/super.c')
-rw-r--r-- | fs/nilfs2/super.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/fs/nilfs2/super.c b/fs/nilfs2/super.c index f47585bfeb01..c69455a543bc 100644 --- a/fs/nilfs2/super.c +++ b/fs/nilfs2/super.c | |||
@@ -1405,14 +1405,10 @@ static void nilfs_destroy_cachep(void) | |||
1405 | */ | 1405 | */ |
1406 | rcu_barrier(); | 1406 | rcu_barrier(); |
1407 | 1407 | ||
1408 | if (nilfs_inode_cachep) | 1408 | kmem_cache_destroy(nilfs_inode_cachep); |
1409 | kmem_cache_destroy(nilfs_inode_cachep); | 1409 | kmem_cache_destroy(nilfs_transaction_cachep); |
1410 | if (nilfs_transaction_cachep) | 1410 | kmem_cache_destroy(nilfs_segbuf_cachep); |
1411 | kmem_cache_destroy(nilfs_transaction_cachep); | 1411 | kmem_cache_destroy(nilfs_btree_path_cache); |
1412 | if (nilfs_segbuf_cachep) | ||
1413 | kmem_cache_destroy(nilfs_segbuf_cachep); | ||
1414 | if (nilfs_btree_path_cache) | ||
1415 | kmem_cache_destroy(nilfs_btree_path_cache); | ||
1416 | } | 1412 | } |
1417 | 1413 | ||
1418 | static int __init nilfs_init_cachep(void) | 1414 | static int __init nilfs_init_cachep(void) |