On Tue, Apr 26, 2022 at 05:52:31PM -0700, Darrick J. Wong wrote: > From: Darrick J. Wong <djwong@xxxxxxxxxx> > > Before to the introduction of deferred refcount operations, reflink > would try to cram refcount btree updates into the same transaction as an > allocation or a free event. Mainline XFS has never actually done that, > but we never refactored the transaction reservations to reflect that we > now do all refcount updates in separate transactions. Fix this to > reduce the transaction reservation size even farther, so that between > this patch and the previous one, we reduce the tr_write and tr_itruncate > sizes by 66%. > > Signed-off-by: Darrick J. Wong <djwong@xxxxxxxxxx> Much easier to understand than the previous version! Thanks for reworking this :) Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx> -- Dave Chinner david@xxxxxxxxxxxxx