From: Johannes Thumshirn <[email protected]>

Since fs_info->zoned is unioned with fs_info->zone_size, loading
fs_info->zoned from the incompat flag screw up the zone_size. So, let's
avoid to load it from the flag. It will be eventually set by
btrfs_get_dev_zone_info_all_devices().

Signed-off-by: Johannes Thumshirn <[email protected]>
---
 fs/btrfs/disk-io.c | 2 --
 fs/btrfs/zoned.c   | 8 ++++++++
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c
index e7b451d30ae2..192e366f8afc 100644
--- a/fs/btrfs/disk-io.c
+++ b/fs/btrfs/disk-io.c
@@ -3136,8 +3136,6 @@ int __cold open_ctree(struct super_block *sb, struct 
btrfs_fs_devices *fs_device
        if (features & BTRFS_FEATURE_INCOMPAT_SKINNY_METADATA)
                btrfs_info(fs_info, "has skinny extents");
 
-       fs_info->zoned = (features & BTRFS_FEATURE_INCOMPAT_ZONED);
-
        /*
         * flag our filesystem as having big metadata blocks if
         * they are bigger than the page size
diff --git a/fs/btrfs/zoned.c b/fs/btrfs/zoned.c
index e5619c8bcebb..ae566a7da088 100644
--- a/fs/btrfs/zoned.c
+++ b/fs/btrfs/zoned.c
@@ -431,6 +431,14 @@ int btrfs_check_zoned_mode(struct btrfs_fs_info *fs_info)
        fs_info->zone_size = zone_size;
        fs_info->max_zone_append_size = max_zone_append_size;
 
+       /*
+        * Check mount options here, because we might change fs_info->zoned
+        * from fs_info->zone_size.
+        */
+       ret = btrfs_check_mountopts_zoned(fs_info);
+       if (ret)
+               goto out;
+
        btrfs_info(fs_info, "zoned mode enabled with zone size %llu", 
zone_size);
 out:
        return ret;
-- 
2.27.0

Reply via email to