btrfs: Remove fs_info argument from convert_free_space_to_bitmaps
authorNikolay Borisov <nborisov@suse.com>
Thu, 10 May 2018 12:44:47 +0000 (15:44 +0300)
committerDavid Sterba <dsterba@suse.com>
Mon, 28 May 2018 16:07:34 +0000 (18:07 +0200)
This function already takes a transaction handle which contains a
reference to fs_info. So use that and remove the extra argument.

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/free-space-tree.c
fs/btrfs/free-space-tree.h
fs/btrfs/tests/free-space-tree-tests.c

index 4a7bef5b8de3073ddf04f135cab664496f265d1a..1dea35ac7139d9c55452854793daf3382b7729e1 100644 (file)
@@ -177,10 +177,10 @@ static void le_bitmap_set(unsigned long *map, unsigned int start, int len)
 }
 
 int convert_free_space_to_bitmaps(struct btrfs_trans_handle *trans,
-                                 struct btrfs_fs_info *fs_info,
                                  struct btrfs_block_group_cache *block_group,
                                  struct btrfs_path *path)
 {
+       struct btrfs_fs_info *fs_info = trans->fs_info;
        struct btrfs_root *root = fs_info->free_space_root;
        struct btrfs_free_space_info *info;
        struct btrfs_key key, found_key;
@@ -477,8 +477,7 @@ static int update_free_space_extent_count(struct btrfs_trans_handle *trans,
 
        if (!(flags & BTRFS_FREE_SPACE_USING_BITMAPS) &&
            extent_count > block_group->bitmap_high_thresh) {
-               ret = convert_free_space_to_bitmaps(trans, fs_info, block_group,
-                                                   path);
+               ret = convert_free_space_to_bitmaps(trans, block_group, path);
        } else if ((flags & BTRFS_FREE_SPACE_USING_BITMAPS) &&
                   extent_count < block_group->bitmap_low_thresh) {
                ret = convert_free_space_to_extents(trans, fs_info, block_group,
index c4adfd05f34fecc69d9147c255c403d193438bab..72ff743611ff0c85d2035be54947dded227687b9 100644 (file)
@@ -43,7 +43,6 @@ int __remove_from_free_space_tree(struct btrfs_trans_handle *trans,
                                  struct btrfs_block_group_cache *block_group,
                                  struct btrfs_path *path, u64 start, u64 size);
 int convert_free_space_to_bitmaps(struct btrfs_trans_handle *trans,
-                                 struct btrfs_fs_info *fs_info,
                                  struct btrfs_block_group_cache *block_group,
                                  struct btrfs_path *path);
 int convert_free_space_to_extents(struct btrfs_trans_handle *trans,
index 86955cf8c9bee809ceec38c3a07486597b188520..d3983ae6173a1c4ee46b2872fefa444a0995dc34 100644 (file)
@@ -137,7 +137,7 @@ static int check_free_space_extents(struct btrfs_trans_handle *trans,
                        return ret;
                }
        } else {
-               ret = convert_free_space_to_bitmaps(trans, fs_info, cache, path);
+               ret = convert_free_space_to_bitmaps(trans, cache, path);
                if (ret) {
                        test_msg("Could not convert to bitmaps\n");
                        return ret;
@@ -498,8 +498,7 @@ static int run_test(test_func_t test_func, int bitmaps, u32 sectorsize,
        }
 
        if (bitmaps) {
-               ret = convert_free_space_to_bitmaps(&trans, root->fs_info,
-                                                   cache, path);
+               ret = convert_free_space_to_bitmaps(&trans, cache, path);
                if (ret) {
                        test_msg("Could not convert block group to bitmaps\n");
                        goto out;