]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
btrfs: remove redundant root argument from btrfs_update_inode_fallback()
authorFilipe Manana <fdmanana@suse.com>
Fri, 22 Sep 2023 10:37:21 +0000 (11:37 +0100)
committerDavid Sterba <dsterba@suse.com>
Thu, 12 Oct 2023 14:44:12 +0000 (16:44 +0200)
The root argument for btrfs_update_inode_fallback() always matches the
root of the given inode, so remove the root argument and get it from the
inode argument.

Reviewed-by: Qu Wenruo <wqu@suse.com>
Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/btrfs_inode.h
fs/btrfs/inode.c
fs/btrfs/transaction.c

index 4311ac9ca0ae01d0006811a060d6b31e0412a038..d12556627cceb69eb73bbcfc11fe88a80b1774c5 100644 (file)
@@ -484,7 +484,7 @@ struct extent_map *btrfs_get_extent(struct btrfs_inode *inode,
 int btrfs_update_inode(struct btrfs_trans_handle *trans,
                       struct btrfs_root *root, struct btrfs_inode *inode);
 int btrfs_update_inode_fallback(struct btrfs_trans_handle *trans,
-                               struct btrfs_root *root, struct btrfs_inode *inode);
+                               struct btrfs_inode *inode);
 int btrfs_orphan_add(struct btrfs_trans_handle *trans, struct btrfs_inode *inode);
 int btrfs_orphan_cleanup(struct btrfs_root *root);
 int btrfs_cont_expand(struct btrfs_inode *inode, loff_t oldsize, loff_t size);
index fb7d7d0077f0c9e1b170f927d0197176d0d5e2aa..44836d1f99a961df5964ed036663665434357e31 100644 (file)
@@ -3074,7 +3074,7 @@ int btrfs_finish_one_ordered(struct btrfs_ordered_extent *ordered_extent)
                        goto out;
                }
                trans->block_rsv = &inode->block_rsv;
-               ret = btrfs_update_inode_fallback(trans, root, inode);
+               ret = btrfs_update_inode_fallback(trans, inode);
                if (ret) /* -ENOMEM or corruption */
                        btrfs_abort_transaction(trans, ret);
                goto out;
@@ -3144,7 +3144,7 @@ int btrfs_finish_one_ordered(struct btrfs_ordered_extent *ordered_extent)
                                 &cached_state);
 
        btrfs_inode_safe_disk_i_size_write(inode, 0);
-       ret = btrfs_update_inode_fallback(trans, root, inode);
+       ret = btrfs_update_inode_fallback(trans, inode);
        if (ret) { /* -ENOMEM or corruption */
                btrfs_abort_transaction(trans, ret);
                goto out;
@@ -4030,13 +4030,13 @@ int btrfs_update_inode(struct btrfs_trans_handle *trans,
 }
 
 int btrfs_update_inode_fallback(struct btrfs_trans_handle *trans,
-                               struct btrfs_root *root, struct btrfs_inode *inode)
+                               struct btrfs_inode *inode)
 {
        int ret;
 
-       ret = btrfs_update_inode(trans, root, inode);
+       ret = btrfs_update_inode(trans, inode->root, inode);
        if (ret == -ENOSPC)
-               return btrfs_update_inode_item(trans, root, inode);
+               return btrfs_update_inode_item(trans, inode->root, inode);
        return ret;
 }
 
@@ -4316,7 +4316,7 @@ static int btrfs_unlink_subvol(struct btrfs_trans_handle *trans,
        btrfs_i_size_write(dir, dir->vfs_inode.i_size - fname.disk_name.len * 2);
        inode_inc_iversion(&dir->vfs_inode);
        dir->vfs_inode.i_mtime = inode_set_ctime_current(&dir->vfs_inode);
-       ret = btrfs_update_inode_fallback(trans, root, dir);
+       ret = btrfs_update_inode_fallback(trans, dir);
        if (ret)
                btrfs_abort_transaction(trans, ret);
 out:
index b87b00884e470bfcf2d4e63a8ae1b2c3cae85efe..682acdd7c13247dcf5f67387934c324b76b0d224 100644 (file)
@@ -1951,7 +1951,7 @@ static noinline int create_pending_snapshot(struct btrfs_trans_handle *trans,
        btrfs_i_size_write(BTRFS_I(parent_inode), parent_inode->i_size +
                                                  fname.disk_name.len * 2);
        parent_inode->i_mtime = inode_set_ctime_current(parent_inode);
-       ret = btrfs_update_inode_fallback(trans, parent_root, BTRFS_I(parent_inode));
+       ret = btrfs_update_inode_fallback(trans, BTRFS_I(parent_inode));
        if (ret) {
                btrfs_abort_transaction(trans, ret);
                goto fail;