aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>2016-09-03 01:35:29 -0400
committerDave Kleikamp <dave.kleikamp@oracle.com>2016-09-06 13:17:24 -0400
commit240c5185c52d00725ae7da7d1ee82ad8b8306d52 (patch)
treeb61b90fcd932e17ab0ef70e544f4c5fec670a522
parent7cfcd8b79ab2472102d9229863e86ff208f959d4 (diff)
jfs: Simplify code
Calling 'list_splice' followed by 'INIT_LIST_HEAD' is equivalent to 'list_splice_init'. This has been spotted with the following coccinelle script: ///// @@ expression y,z; @@ - list_splice(y,z); - INIT_LIST_HEAD(y); + list_splice_init(y,z); Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr> Signed-off-by: Dave Kleikamp <dave.kleikamp@oracle.com>
-rw-r--r--fs/jfs/jfs_txnmgr.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/jfs/jfs_txnmgr.c b/fs/jfs/jfs_txnmgr.c
index 2e58978d6f45..4d973524c887 100644
--- a/fs/jfs/jfs_txnmgr.c
+++ b/fs/jfs/jfs_txnmgr.c
@@ -2893,8 +2893,7 @@ restart:
2893 * on anon_list2. Let's check. 2893 * on anon_list2. Let's check.
2894 */ 2894 */
2895 if (!list_empty(&TxAnchor.anon_list2)) { 2895 if (!list_empty(&TxAnchor.anon_list2)) {
2896 list_splice(&TxAnchor.anon_list2, &TxAnchor.anon_list); 2896 list_splice_init(&TxAnchor.anon_list2, &TxAnchor.anon_list);
2897 INIT_LIST_HEAD(&TxAnchor.anon_list2);
2898 goto restart; 2897 goto restart;
2899 } 2898 }
2900 TXN_UNLOCK(); 2899 TXN_UNLOCK();