On Fri, Oct 02, 2020 at 03:36:11AM +0900, Naohiro Aota wrote: > --- a/fs/btrfs/ctree.h > +++ b/fs/btrfs/ctree.h > @@ -588,6 +588,9 @@ struct btrfs_fs_info { > struct btrfs_root *free_space_root; > struct btrfs_root *data_reloc_root; > > + /* Zone size when in ZONED mode */ > + u64 zone_size; > + > /* the log root tree is a directory of all the other log roots */ > struct btrfs_root *log_root_tree; This is misplaced, new members should be placed to an existing location if there is one or at the end of the structure. What's the logic behind putting zone_size and later max_zone_append_size into the tree root pointers?