btrfs: consolidate btrfs_previous_item ret val handling in btrfs_shrink_device
authorNikolay Borisov <nborisov@suse.com>
Thu, 17 Dec 2020 13:21:16 +0000 (15:21 +0200)
committerDavid Sterba <dsterba@suse.com>
Mon, 8 Feb 2021 21:58:51 +0000 (22:58 +0100)
commit7056bf69e5a338811738a7932b8e707aaca9fdd0
tree66a9ec7443fabbe547d81c3a406d7dacfa7a116f
parent1478143ac81acc4094f8501a88e9e6ef9ff0e4a5
btrfs: consolidate btrfs_previous_item ret val handling in btrfs_shrink_device

Instead of having three 'if' to handle non-NULL return value consolidate
this in one 'if (ret)'. That way the code is more obvious:

 - Always drop delete_unused_bgs_mutex if ret is not NULL
 - If ret is negative -> goto done
 - If it's 1 -> reset ret to 0, release the path and finish the loop.

Reviewed-by: Josef Bacik <josef@toxicpanda.com>
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/volumes.c