]> www.infradead.org Git - users/hch/xfs.git/commit
btrfs: fix uninitialized return value from btrfs_reclaim_sweep()
authorFilipe Manana <fdmanana@suse.com>
Tue, 27 Aug 2024 10:30:10 +0000 (11:30 +0100)
committerDavid Sterba <dsterba@suse.com>
Tue, 27 Aug 2024 14:42:09 +0000 (16:42 +0200)
commitecb54277cb63c273e8d74272e5b9bfd80c2185d9
tree968a715beab4f09a0c7563398395d2cb8cf91b2b
parent10d9d8c3512f16cad47b2ff81ec6fc4b27d8ee10
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>
fs/btrfs/space-info.c
fs/btrfs/space-info.h