btrfs: zoned: do not account freed region of read-only block group as zone_unusable
authorNaohiro Aota <naohiro.aota@wdc.com>
Wed, 3 Mar 2021 08:55:48 +0000 (17:55 +0900)
committerDavid Sterba <dsterba@suse.com>
Thu, 4 Mar 2021 15:16:58 +0000 (16:16 +0100)
We migrate zone unusable bytes to read-only bytes when a block group is
set to read-only, and account all the free region as bytes_readonly.
Thus, we should not increase block_group->zone_unusable when the block
group is read-only.

Fixes: 169e0da91a21 ("btrfs: zoned: track unusable bytes for zones")
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>
fs/btrfs/free-space-cache.c

index 711a6a751ae99ed34ead38c807c198906a97f539..9988decd5717bd4ca73dcdd9d02fb6c1bda9903a 100644 (file)
@@ -2555,7 +2555,12 @@ static int __btrfs_add_free_space_zoned(struct btrfs_block_group *block_group,
        to_unusable = size - to_free;
 
        ctl->free_space += to_free;
-       block_group->zone_unusable += to_unusable;
+       /*
+        * If the block group is read-only, we should account freed space into
+        * bytes_readonly.
+        */
+       if (!block_group->ro)
+               block_group->zone_unusable += to_unusable;
        spin_unlock(&ctl->tree_lock);
        if (!used) {
                spin_lock(&block_group->lock);