btrfs: make read_block_group_item return void

Since it's inclusion on 9afc66498a0b ("btrfs: block-group: refactor how
we read one block group item") this function always returned 0, so there
is no need to check for the returned value.

Reviewed-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: Marcos Paulo de Souza <mpdesouza@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>

diff --git a/fs/btrfs/block-group.c b/fs/btrfs/block-group.c
index c9d7465501a4759954b811eb13407f8dee116291..01e8ba1da1d3ed8b4e3d63e543cd4441f4aa23f2 100644 (file)
--- a/fs/btrfs/block-group.c
+++ b/fs/btrfs/block-group.c
@@ -1873,7 +1873,7 @@ static int check_chunk_block_group_mappings(struct btrfs_fs_info *fs_info)
        return ret;
 }
 
-static int read_block_group_item(struct btrfs_block_group *cache,
+static void read_block_group_item(struct btrfs_block_group *cache,
                                 struct btrfs_path *path,
                                 const struct btrfs_key *key)
 {
@@ -1887,8 +1887,6 @@ static int read_block_group_item(struct btrfs_block_group *cache,
                           sizeof(bgi));
        cache->used = btrfs_stack_block_group_used(&bgi);
        cache->flags = btrfs_stack_block_group_flags(&bgi);
-
-       return 0;
 }
 
 static int read_one_block_group(struct btrfs_fs_info *info,
@@ -1907,9 +1905,7 @@ static int read_one_block_group(struct btrfs_fs_info *info,
        if (!cache)
                return -ENOMEM;
 
-       ret = read_block_group_item(cache, path, key);
-       if (ret < 0)
-               goto error;
+       read_block_group_item(cache, path, key);
 
        set_free_space_tree_thresholds(cache);