diff options
| author | David Sterba <dsterba@suse.com> | 2024-01-20 04:17:03 +0300 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2024-03-04 18:24:47 +0300 |
| commit | 97ec332068f00bb1da73bc25949b4be3a08e36f8 (patch) | |
| tree | 53a39c96012ae67912a3a3696ffed587ca2afe7e | |
| parent | c093bf306562c34d1c71f89897bb9220f44f5e4c (diff) | |
| download | linux-97ec332068f00bb1da73bc25949b4be3a08e36f8.tar.xz | |
btrfs: handle block group lookup error when it's being removed
The unlikely case of lookup error in btrfs_remove_block_group() can be
handled properly, in its caller this would lead to a transaction abort.
We can't do anything else, a block group must have been loaded first.
Reviewed-by: Josef Bacik <josef@toxicpanda.com>
Reviewed-by: Anand Jain <anand.jain@oracle.com>
Signed-off-by: David Sterba <dsterba@suse.com>
| -rw-r--r-- | fs/btrfs/block-group.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/btrfs/block-group.c b/fs/btrfs/block-group.c index 78cae9f9deca..58f2e8951dbf 100644 --- a/fs/btrfs/block-group.c +++ b/fs/btrfs/block-group.c @@ -1063,7 +1063,9 @@ int btrfs_remove_block_group(struct btrfs_trans_handle *trans, bool remove_rsv = false; block_group = btrfs_lookup_block_group(fs_info, map->start); - BUG_ON(!block_group); + if (!block_group) + return -ENOENT; + BUG_ON(!block_group->ro); trace_btrfs_remove_block_group(block_group); |