Thread (9 messages) 9 messages, 4 authors, 2021-07-01

Re: [PATCH v2] btrfs: zoned: remove fs_info->max_zone_append_size

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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help