On Sun, Jun 13, 2021 at 10:20:13AM -0700, Darrick J. Wong wrote:
> - xfs_inactive(ip);
> + if (!need_inactive) {
> + /* Going straight to reclaim, so drop the dquots. */
> + xfs_qm_dqdetach(ip);
> + } else {
> + xfs_inactive(ip);
> + }
It seems like most of the checks contained in xfs_inode_needs_inactive
could be dropped from xfs_inactive now.
But given that I feel the heat already I'm not going to suggest that
now and just going to look into cleaning that up after the series
goes in..
Reviewed-by: Christoph Hellwig <hch@xxxxxx>
