On Tue, Oct 15, 2024 at 09:11:06AM -0400, Brian Foster wrote: > > xfs_initialize_perag( > > struct xfs_mount *mp, > > - xfs_agnumber_t agcount, > > + xfs_agnumber_t old_agcount, > > + xfs_agnumber_t new_agcount, > > What happened to using first/end or whatever terminology here like is > done in one of the later patches? I really find old/new unnecessarily > confusing in this context. I though you only wanted that for the caller. I can fix it up.