On Sun, Jun 13, 2021 at 10:20:24AM -0700, Darrick J. Wong wrote: > From: Darrick J. Wong <djwong@xxxxxxxxxx> > > During review of the v6 deferred inode inactivation patchset[1], Dave > commented that _cache_hit should have a clear separation between inode > selection criteria and actions performed on a selected inode. Move a > hunk to make this true, and compact the shrink cases in the function. I'm not sure the backstore really belongs here vs describing what changes for what reason in the code. It might be worth that this is now consistently calling the tracepoints and incrementing xs_ig_frecycle, though. But the actual changes look like a really nice improvement, so: Reviewed-by: Christoph Hellwig <hch@xxxxxx>
