Re: [PATCH v2 07/13] xfs: Introduce FORCEALIGN inode flag

From: John Garry
Date: Wed Jul 24 2024 - 03:40:27 EST


On 24/07/2024 00:38, Dave Chinner wrote:
What's the problem with supporting it right from the start?

Simply because I wanted to focus on regular data volume support first.

We
already support forcealign for RT, just it's a global config
under the "big rt alloc" moniker rather than a per-inode flag.

Like all on-disk format change based features,
forcealign should add the EXPERIMENTAL flag to the filesystem for a
couple of releases after merge, so there will be plenty of time to
test both data and rt dev functionality before removing the
EXPERIMENTAL flag from it.

So why not just enable the per-inode flag with RT right from the
start given that this functionality is supposed to work and be
globally supported by the rtdev right now? It seems like a whole lot
less work to just enable it for RT now than it is to disable it...

I'll have a look... if it really is that easy, then - yes - we can have it from the start.