diff options
| author |  Naohiro Aota <naohiro.aota@wdc.com>
| 2023-08-08 01:12:39 +0900 |
|---|---|---|
| committer |  David Sterba <dsterba@suse.com>
| 2023-08-21 14:52:19 +0200 |
| commit | 5a7d107e5ef9b452cf0aa96ac931b29ea980d997 (patch) | |
| tree | 94775880df47748865e074883d1158d971cf892a /fs/btrfs/extent-tree.c | |
| parent | btrfs: zoned: no longer count fresh BG region as zone unusable (diff) | |
| download | linux-5a7d107e5ef9b452cf0aa96ac931b29ea980d997.tar.gz linux-5a7d107e5ef9b452cf0aa96ac931b29ea980d997.tar.bz2 linux-5a7d107e5ef9b452cf0aa96ac931b29ea980d997.zip | |
btrfs: zoned: don't activate non-DATA BG on allocation
Now that a non-DATA block group is activated at write time, don't
activate it on allocation time.
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Signed-off-by: Naohiro Aota <naohiro.aota@wdc.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/extent-tree.c')
| -rw-r--r-- | fs/btrfs/extent-tree.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/fs/btrfs/extent-tree.c b/fs/btrfs/extent-tree.c index 6dfe086523f9..bc9ff192196f 100644 --- a/fs/btrfs/extent-tree.c +++ b/fs/btrfs/extent-tree.c @@ -3690,7 +3690,9 @@ static int do_allocation_zoned(struct btrfs_block_group *block_group, } spin_unlock(&block_group->lock); - if (!ret && !btrfs_zone_activate(block_group)) { + /* Metadata block group is activated at write time. */ + if (!ret && (block_group->flags & BTRFS_BLOCK_GROUP_DATA) && + !btrfs_zone_activate(block_group)) { ret = 1; /* * May need to clear fs_info->{treelog,data_reloc}_bg. @@ -3866,6 +3868,10 @@ static void found_extent(struct find_free_extent_ctl *ffe_ctl, static int can_allocate_chunk_zoned(struct btrfs_fs_info *fs_info, struct find_free_extent_ctl *ffe_ctl) { + /* Block group's activeness is not a requirement for METADATA block groups. */ + if (!(ffe_ctl->flags & BTRFS_BLOCK_GROUP_DATA)) + return 0; + /* If we can activate new zone, just allocate a chunk and use it */ if (btrfs_can_activate_zone(fs_info->fs_devices, ffe_ctl->flags)) return 0; |