On Tue, Mar 02, 2021 at 02:28:28PM -0800, Darrick J. Wong wrote: > From: Darrick J. Wong <djwong@xxxxxxxxxx> > > When we're servicing an INUMBERS or BULKSTAT request, grab an empty > transaction so that we don't hit an ABBA buffer deadlock if the inode > btree contains a cycle. > > Found by fuzzing an inode btree pointer to introduce a cycle into the > tree (xfs/365). > > Signed-off-by: Darrick J. Wong <djwong@xxxxxxxxxx> So basically you want to piggy back on the per-trans recursion using xfs_trans_buf_item_match? Why do we need the sb-counter for that? Can the comments be a little more clear? Why don't we want that elsewhere where we're walking the btrees?