btrfs: make btrfs_set_inode_last_trans take btrfs_inode
authorNikolay Borisov <nborisov@suse.com>
Fri, 5 Jun 2020 07:41:13 +0000 (10:41 +0300)
committerDavid Sterba <dsterba@suse.com>
Mon, 27 Jul 2020 10:55:37 +0000 (12:55 +0200)
Instead of making multiple calls to BTRFS_I simply take btrfs_inode as
an input paramter.

Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/inode.c
fs/btrfs/transaction.h

index 0a03883..07dd8aa 100644 (file)
@@ -3481,7 +3481,7 @@ static noinline int btrfs_update_inode_item(struct btrfs_trans_handle *trans,
 
        fill_inode_item(trans, leaf, inode_item, inode);
        btrfs_mark_buffer_dirty(leaf);
-       btrfs_set_inode_last_trans(trans, inode);
+       btrfs_set_inode_last_trans(trans, BTRFS_I(inode));
        ret = 0;
 failed:
        btrfs_free_path(path);
@@ -3511,7 +3511,7 @@ noinline int btrfs_update_inode(struct btrfs_trans_handle *trans,
 
                ret = btrfs_delayed_update_inode(trans, root, inode);
                if (!ret)
-                       btrfs_set_inode_last_trans(trans, inode);
+                       btrfs_set_inode_last_trans(trans, BTRFS_I(inode));
                return ret;
        }
 
@@ -6053,7 +6053,7 @@ static struct inode *btrfs_new_inode(struct btrfs_trans_handle *trans,
        inode_tree_add(inode);
 
        trace_btrfs_inode_new(inode);
-       btrfs_set_inode_last_trans(trans, inode);
+       btrfs_set_inode_last_trans(trans, BTRFS_I(inode));
 
        btrfs_update_root_times(trans, root);
 
index bf102e6..6f65fff 100644 (file)
@@ -156,13 +156,13 @@ struct btrfs_pending_snapshot {
 };
 
 static inline void btrfs_set_inode_last_trans(struct btrfs_trans_handle *trans,
-                                             struct inode *inode)
+                                             struct btrfs_inode *inode)
 {
-       spin_lock(&BTRFS_I(inode)->lock);
-       BTRFS_I(inode)->last_trans = trans->transaction->transid;
-       BTRFS_I(inode)->last_sub_trans = BTRFS_I(inode)->root->log_transid;
-       BTRFS_I(inode)->last_log_commit = BTRFS_I(inode)->root->last_log_commit;
-       spin_unlock(&BTRFS_I(inode)->lock);
+       spin_lock(&inode->lock);
+       inode->last_trans = trans->transaction->transid;
+       inode->last_sub_trans = inode->root->log_transid;
+       inode->last_log_commit = inode->root->last_log_commit;
+       spin_unlock(&inode->lock);
 }
 
 /*