Re: [PATCH v2 2/3] btrfs: zoned: fix write time activation failure for metadata block group

[Johannes Thumshirn <Johannes.Thumshirn@wdc.com>]

On 11.07.25 09:24, Naohiro Aota wrote:
> Since commit 13bb483d32ab ("btrfs: zoned: activate metadata block group on
> write time"), we activate a metadata block group at the write time. If the
                                                s/the// ~^

> zone capacity is small enough, we can allocate the entire region before the
> first write. Then, we hit the btrfs_zoned_bg_is_full() in
                                           + condition? ~^
> btrfs_zone_activate() and the activation fails.
> 
> For a data block group, we activate it at the allocation time and we should
                                     s/the// ~^
> check the fullness condition in the caller side. Add, a WARN to check the
                                                s/,// ~^
> fullness condition.
> 
> For a metadata block group, we don't need the fullness check because we
> activate it at the write time. Instead, activating it once it is written
         s/the// ~^
> should be invalid. Catch that with a WARN too.
> 
> Fixes: 13bb483d32ab ("btrfs: zoned: activate metadata block group on write time")
> CC: stable@vger.kernel.org # 6.6+
> Signed-off-by: Naohiro Aota <naohiro.aota@wdc.com>

Otherwise looks good to me,
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>