btrfs: remove redundant root argument from fixup_inode_link_count()

The root argument for fixup_inode_link_count() always matches the root of
the given inode, so remove the root argument and get it from the inode
argument. This also applies to the helpers count_inode_extrefs() and
count_inode_refs() used by fixup_inode_link_count() - they don't need the
root argument, as it always matches the root of the inode passed to them.

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>

diff --git a/fs/btrfs/tree-log.c b/fs/btrfs/tree-log.c
index f77fa761ef3afc45cd8fa6e6c878aee1a96ed387..a14ac1a964e28d4a5783814c5e0750de9b3a4827 100644 (file)
--- a/fs/btrfs/tree-log.c
+++ b/fs/btrfs/tree-log.c
@@ -1483,8 +1483,7 @@ out:
        return ret;
 }
 
-static int count_inode_extrefs(struct btrfs_root *root,
-               struct btrfs_inode *inode, struct btrfs_path *path)
+static int count_inode_extrefs(struct btrfs_inode *inode, struct btrfs_path *path)
 {
        int ret = 0;
        int name_len;
@@ -1498,8 +1497,8 @@ static int count_inode_extrefs(struct btrfs_root *root,
        struct extent_buffer *leaf;
 
        while (1) {
-               ret = btrfs_find_one_extref(root, inode_objectid, offset, path,
-                                           &extref, &offset);
+               ret = btrfs_find_one_extref(inode->root, inode_objectid, offset,
+                                           path, &extref, &offset);
                if (ret)
                        break;
 
@@ -1527,8 +1526,7 @@ static int count_inode_extrefs(struct btrfs_root *root,
        return nlink;
 }
 
-static int count_inode_refs(struct btrfs_root *root,
-                       struct btrfs_inode *inode, struct btrfs_path *path)
+static int count_inode_refs(struct btrfs_inode *inode, struct btrfs_path *path)
 {
        int ret;
        struct btrfs_key key;
@@ -1543,7 +1541,7 @@ static int count_inode_refs(struct btrfs_root *root,
        key.offset = (u64)-1;
 
        while (1) {
-               ret = btrfs_search_slot(NULL, root, &key, path, 0, 0);
+               ret = btrfs_search_slot(NULL, inode->root, &key, path, 0, 0);
                if (ret < 0)
                        break;
                if (ret > 0) {
@@ -1595,9 +1593,9 @@ process_slot:
  * will free the inode.
  */
 static noinline int fixup_inode_link_count(struct btrfs_trans_handle *trans,
-                                          struct btrfs_root *root,
                                           struct inode *inode)
 {
+       struct btrfs_root *root = BTRFS_I(inode)->root;
        struct btrfs_path *path;
        int ret;
        u64 nlink = 0;
@@ -1607,13 +1605,13 @@ static noinline int fixup_inode_link_count(struct btrfs_trans_handle *trans,
        if (!path)
                return -ENOMEM;
 
-       ret = count_inode_refs(root, BTRFS_I(inode), path);
+       ret = count_inode_refs(BTRFS_I(inode), path);
        if (ret < 0)
                goto out;
 
        nlink = ret;
 
-       ret = count_inode_extrefs(root, BTRFS_I(inode), path);
+       ret = count_inode_extrefs(BTRFS_I(inode), path);
        if (ret < 0)
                goto out;
 
@@ -1685,7 +1683,7 @@ static noinline int fixup_inode_link_counts(struct btrfs_trans_handle *trans,
                        break;
                }
 
-               ret = fixup_inode_link_count(trans, root, inode);
+               ret = fixup_inode_link_count(trans, inode);
                iput(inode);
                if (ret)
                        break;