Commit 550d6da288df57f154ca27c4acb1c398ced42ea9

Authored by Wei Yongjun
Committed by Dave Kleikamp
1 parent 16221d9071

JFS: use list_move instead of list_del/list_add

Using list_move() instead of list_del() + list_add().

spatch with a semantic match is used to found this problem.
(http://coccinelle.lip6.fr/)

Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: Dave Kleikamp <dave.kleikamp@oracle.com>

Showing 1 changed file with 3 additions and 6 deletions Side-by-side Diff

... ... @@ -2977,12 +2977,9 @@
2977 2977 * put back on the anon_list.
2978 2978 */
2979 2979  
2980   - /* Take off anon_list */
2981   - list_del(&jfs_ip->anon_inode_list);
2982   -
2983   - /* Put on anon_list2 */
2984   - list_add(&jfs_ip->anon_inode_list,
2985   - &TxAnchor.anon_list2);
  2980 + /* Move from anon_list to anon_list2 */
  2981 + list_move(&jfs_ip->anon_inode_list,
  2982 + &TxAnchor.anon_list2);
2986 2983  
2987 2984 TXN_UNLOCK();
2988 2985 iput(ip);