On 14/04/2021 13:23, Johannes Thumshirn wrote: > From how I understand the code, yes. It's a quick test, so let's just do it > and see what breaks. > > I'd prefer to just drop the ->ro check, it's less special casing for zoned > btrfs that we have to keep in mind when changing things.
OK, no this doesn't work, because btrfs_start_trans_remove_block_group() has this ASSERT(em && em->start == chunk_offset);, but btrfs_remove_block_group() from relocation has already called remove_extent_mapping().
