Re: [PATCH v2] btrfs: zoned: remove fs_info->max_zone_append_size
From: Damien Le Moal <hidden>
Date: 2021-07-01 10:06:25
From: Damien Le Moal <hidden>
Date: 2021-07-01 10:06:25
On 2021/07/01 17:02, Johannes Thumshirn wrote:
On 30/06/2021 20:51, David Sterba wrote:quoted
On Tue, Jun 29, 2021 at 03:36:45AM +0900, Johannes Thumshirn wrote:quoted
Remove fs_info->max_zone_append_size, it doesn't serve any purpose. Signed-off-by: Johannes Thumshirn <redacted> --- Changes to v1: - also remove the local max_zone_append_size variable in btrfs_check_zoned_mode() (Anand)And what about btrfs_zoned_device_info::max_zone_append_size, should it also be removed? In case you don't have plans with it I'll remove it, no need to resend, it's just a few more lines but want to know if it's accidental or intentional.I /think/ this one can stay until we work on multi-device/RAID support.
We are nowhere near close to this for now, so I am all for removing it.
We could though also remove it and bring it back in case we need it again. @Naohiro what's your take on that?
-- Damien Le Moal Western Digital Research