btrfs: fix uninitialized return value from btrfs_reclaim_sweep()

The return variable 'ret' at btrfs_reclaim_sweep() is never assigned if
none of the space infos is reclaimable (for example if periodic reclaim
is disabled, which is the default), so we return an undefined value.

This can be fixed my making btrfs_reclaim_sweep() not return any value
as well as do_reclaim_sweep() because:

1) do_reclaim_sweep() always returns 0, so we can make it return void;

2) The only caller of btrfs_reclaim_sweep() (btrfs_reclaim_bgs()) doesn't
   care about its return value, and in its context there's nothing to do
   about any errors anyway.

Therefore remove the return value from btrfs_reclaim_sweep() and
do_reclaim_sweep().

Fixes: e4ca3932ae90 ("btrfs: periodic block_group reclaim")
Reviewed-by: Josef Bacik <josef@toxicpanda.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/space-info.c b/fs/btrfs/space-info.c
index 68e14fd486384e3d47236f4c40d28501fcc8ccd6..c691784b4660cdc0d45b7e0438f30ddc77f507bc 100644 (file)
--- a/fs/btrfs/space-info.c
+++ b/fs/btrfs/space-info.c
@@ -1985,8 +1985,8 @@ static bool is_reclaim_urgent(struct btrfs_space_info *space_info)
        return unalloc < data_chunk_size;
 }
 
-static int do_reclaim_sweep(struct btrfs_fs_info *fs_info,
-                           struct btrfs_space_info *space_info, int raid)
+static void do_reclaim_sweep(struct btrfs_fs_info *fs_info,
+                            struct btrfs_space_info *space_info, int raid)
 {
        struct btrfs_block_group *bg;
        int thresh_pct;
@@ -2031,7 +2031,6 @@ again:
        }
 
        up_read(&space_info->groups_sem);
-       return 0;
 }
 
 void btrfs_space_info_update_reclaimable(struct btrfs_space_info *space_info, s64 bytes)
@@ -2074,21 +2073,15 @@ bool btrfs_should_periodic_reclaim(struct btrfs_space_info *space_info)
        return ret;
 }
 
-int btrfs_reclaim_sweep(struct btrfs_fs_info *fs_info)
+void btrfs_reclaim_sweep(struct btrfs_fs_info *fs_info)
 {
-       int ret;
        int raid;
        struct btrfs_space_info *space_info;
 
        list_for_each_entry(space_info, &fs_info->space_info, list) {
                if (!btrfs_should_periodic_reclaim(space_info))
                        continue;
-               for (raid = 0; raid < BTRFS_NR_RAID_TYPES; raid++) {
-                       ret = do_reclaim_sweep(fs_info, space_info, raid);
-                       if (ret)
-                               return ret;
-               }
+               for (raid = 0; raid < BTRFS_NR_RAID_TYPES; raid++)
+                       do_reclaim_sweep(fs_info, space_info, raid);
        }
-
-       return ret;
 }

diff --git a/fs/btrfs/space-info.h b/fs/btrfs/space-info.h
index 88b44221ce9782304fc4c7ead17c49fd593e451b..5602026c5e14853995260a7ac12ea3d952d3a729 100644 (file)
--- a/fs/btrfs/space-info.h
+++ b/fs/btrfs/space-info.h
@@ -294,6 +294,6 @@ void btrfs_space_info_update_reclaimable(struct btrfs_space_info *space_info, s6
 void btrfs_set_periodic_reclaim_ready(struct btrfs_space_info *space_info, bool ready);
 bool btrfs_should_periodic_reclaim(struct btrfs_space_info *space_info);
 int btrfs_calc_reclaim_threshold(struct btrfs_space_info *space_info);
-int btrfs_reclaim_sweep(struct btrfs_fs_info *fs_info);
+void btrfs_reclaim_sweep(struct btrfs_fs_info *fs_info);
 
 #endif /* BTRFS_SPACE_INFO_H */