diff options
| author | Filipe Manana <fdmanana@suse.com> | 2026-01-21 20:42:53 +0000 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2026-02-03 07:56:19 +0100 |
| commit | 79d51b5c7a2c64f3420ff632df67a76bf01a46ed (patch) | |
| tree | 44dce5988a5345c1f20abaabe3cca6054188ff2a | |
| parent | 7c2830f00c3e086292c1ee9f27b61efaf8e76c9a (diff) | |
btrfs: remove bogus root search condition in sample_block_group_extent_item()
There's no need to pass the maximum between the block group's start offset
and BTRFS_SUPER_INFO_OFFSET (64K) since we can't have any block groups
allocated in the first megabyte, as that's reserved space. Furthermore,
even if we could, the correct thing to do was to pass the block group's
start offset anyway - and that's precisely what we do for block groups
that happen to contain superblock mirror (the range for the super block
is never marked as free and it's marked as dirty in the
fs_info->excluded_extents io tree).
So simplify this and get rid of that maximum expression.
Reviewed-by: Boris Burkov <boris@bur.io>
Reviewed-by: Qu Wenruo <wqu@suse.com>
Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
| -rw-r--r-- | fs/btrfs/block-group.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/btrfs/block-group.c b/fs/btrfs/block-group.c index 11ed303c6640..66c512bc022c 100644 --- a/fs/btrfs/block-group.c +++ b/fs/btrfs/block-group.c @@ -606,8 +606,7 @@ static int sample_block_group_extent_item(struct btrfs_caching_control *caching_ lockdep_assert_held(&caching_ctl->mutex); lockdep_assert_held_read(&fs_info->commit_root_sem); - extent_root = btrfs_extent_root(fs_info, max_t(u64, block_group->start, - BTRFS_SUPER_INFO_OFFSET)); + extent_root = btrfs_extent_root(fs_info, block_group->start); search_offset = index * div_u64(block_group->length, max_index); search_key.objectid = block_group->start + search_offset; |