Re: [PATCH 02/10] xfs: move RT inode locking out of __xfs_bunmapi

Christoph Hellwig
2024-02-23T16:49:21Z

On Fri, Feb 23, 2024 at 08:46:55AM -0800, Darrick J. Wong wrote:
> > The only "sane" way out would be to always use a deferred item, which we
> > should be doing for anything using new RT features, but we can't really
> > do that for legacy file systems without forcing a log incompat flag.
> > So while I don't particularly like the transaction flag it seems like
> > the least evil solution.
>
> I had thought about doing that for rtgroups=1 filesystems. :)

I actually have a patch doing that in my "not quite finished" queue.
Given that your patch queue already enables it for rmap and reflink
it's pretty trivial.