On Thu, Jun 04, 2020 at 05:45:58PM +1000, Dave Chinner wrote:
> From: Dave Chinner <dchinner@xxxxxxxxxx>
>
> Clean up xfs_reclaim_inodes() callers. Most callers want blocking
> behaviour, so just make the existing SYNC_WAIT behaviour the
> default.
>
> For the xfs_reclaim_worker(), just call xfs_reclaim_inodes_ag()
> directly because we just want optimistic clean inode reclaim to be
> done in the background.
>
> For xfs_quiesce_attr() we can just remove the inode reclaim calls as
> they are a historic relic that was required to flush dirty inodes
> that contained unlogged changes. We now log all changes to the
> inodes, so the sync AIL push from xfs_log_quiesce() called by
> xfs_quiesce_attr() will do all the required inode writeback for
> freeze.
>
The above change should probably be a standalone patch, but not worth
changing at this point:
Reviewed-by: Brian Foster <bfoster@xxxxxxxxxx>
BTW, is there any reason we continue to drain the buffer lru for freeze
as well?
Brian
> Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx>
> Reviewed-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx>
> ---
> fs/xfs/xfs_icache.c | 48 ++++++++++++++++++++-------------------------
> fs/xfs/xfs_icache.h | 2 +-
> fs/xfs/xfs_mount.c | 11 +++++------
> fs/xfs/xfs_super.c | 3 ---
> 4 files changed, 27 insertions(+), 37 deletions(-)
>
> diff --git a/fs/xfs/xfs_icache.c b/fs/xfs/xfs_icache.c
> index ebe55124d6cb8..a27470fc201ff 100644
> --- a/fs/xfs/xfs_icache.c
> +++ b/fs/xfs/xfs_icache.c
> @@ -160,24 +160,6 @@ xfs_reclaim_work_queue(
> rcu_read_unlock();
> }
>
> -/*
> - * This is a fast pass over the inode cache to try to get reclaim moving on as
> - * many inodes as possible in a short period of time. It kicks itself every few
> - * seconds, as well as being kicked by the inode cache shrinker when memory
> - * goes low. It scans as quickly as possible avoiding locked inodes or those
> - * already being flushed, and once done schedules a future pass.
> - */
> -void
> -xfs_reclaim_worker(
> - struct work_struct *work)
> -{
> - struct xfs_mount *mp = container_of(to_delayed_work(work),
> - struct xfs_mount, m_reclaim_work);
> -
> - xfs_reclaim_inodes(mp, 0);
> - xfs_reclaim_work_queue(mp);
> -}
> -
> static void
> xfs_perag_set_reclaim_tag(
> struct xfs_perag *pag)
> @@ -1298,24 +1280,17 @@ xfs_reclaim_inodes_ag(
> return skipped;
> }
>
> -int
> +void
> xfs_reclaim_inodes(
> - xfs_mount_t *mp,
> - int mode)
> + struct xfs_mount *mp)
> {
> int nr_to_scan = INT_MAX;
> int skipped;
>
> - xfs_reclaim_inodes_ag(mp, &nr_to_scan);
> - if (!(mode & SYNC_WAIT))
> - return 0;
> -
> do {
> xfs_ail_push_all_sync(mp->m_ail);
> skipped = xfs_reclaim_inodes_ag(mp, &nr_to_scan);
> } while (skipped > 0);
> -
> - return 0;
> }
>
> /*
> @@ -1434,6 +1409,25 @@ xfs_inode_matches_eofb(
> return true;
> }
>
> +/*
> + * This is a fast pass over the inode cache to try to get reclaim moving on as
> + * many inodes as possible in a short period of time. It kicks itself every few
> + * seconds, as well as being kicked by the inode cache shrinker when memory
> + * goes low. It scans as quickly as possible avoiding locked inodes or those
> + * already being flushed, and once done schedules a future pass.
> + */
> +void
> +xfs_reclaim_worker(
> + struct work_struct *work)
> +{
> + struct xfs_mount *mp = container_of(to_delayed_work(work),
> + struct xfs_mount, m_reclaim_work);
> + int nr_to_scan = INT_MAX;
> +
> + xfs_reclaim_inodes_ag(mp, &nr_to_scan);
> + xfs_reclaim_work_queue(mp);
> +}
> +
> STATIC int
> xfs_inode_free_eofblocks(
> struct xfs_inode *ip,
> diff --git a/fs/xfs/xfs_icache.h b/fs/xfs/xfs_icache.h
> index 93b54e7d55f0d..ae92ca53de423 100644
> --- a/fs/xfs/xfs_icache.h
> +++ b/fs/xfs/xfs_icache.h
> @@ -51,7 +51,7 @@ void xfs_inode_free(struct xfs_inode *ip);
>
> void xfs_reclaim_worker(struct work_struct *work);
>
> -int xfs_reclaim_inodes(struct xfs_mount *mp, int mode);
> +void xfs_reclaim_inodes(struct xfs_mount *mp);
> int xfs_reclaim_inodes_count(struct xfs_mount *mp);
> long xfs_reclaim_inodes_nr(struct xfs_mount *mp, int nr_to_scan);
>
> diff --git a/fs/xfs/xfs_mount.c b/fs/xfs/xfs_mount.c
> index 03158b42a1943..c8ae49a1e99c3 100644
> --- a/fs/xfs/xfs_mount.c
> +++ b/fs/xfs/xfs_mount.c
> @@ -1011,7 +1011,7 @@ xfs_mountfs(
> * quota inodes.
> */
> cancel_delayed_work_sync(&mp->m_reclaim_work);
> - xfs_reclaim_inodes(mp, SYNC_WAIT);
> + xfs_reclaim_inodes(mp);
> xfs_health_unmount(mp);
> out_log_dealloc:
> mp->m_flags |= XFS_MOUNT_UNMOUNTING;
> @@ -1088,13 +1088,12 @@ xfs_unmountfs(
> xfs_ail_push_all_sync(mp->m_ail);
>
> /*
> - * And reclaim all inodes. At this point there should be no dirty
> - * inodes and none should be pinned or locked, but use synchronous
> - * reclaim just to be sure. We can stop background inode reclaim
> - * here as well if it is still running.
> + * Reclaim all inodes. At this point there should be no dirty inodes and
> + * none should be pinned or locked. Stop background inode reclaim here
> + * if it is still running.
> */
> cancel_delayed_work_sync(&mp->m_reclaim_work);
> - xfs_reclaim_inodes(mp, SYNC_WAIT);
> + xfs_reclaim_inodes(mp);
> xfs_health_unmount(mp);
>
> xfs_qm_unmount(mp);
> diff --git a/fs/xfs/xfs_super.c b/fs/xfs/xfs_super.c
> index fa58cb07c8fdf..9b03ea43f4fe7 100644
> --- a/fs/xfs/xfs_super.c
> +++ b/fs/xfs/xfs_super.c
> @@ -890,9 +890,6 @@ xfs_quiesce_attr(
> /* force the log to unpin objects from the now complete transactions */
> xfs_log_force(mp, XFS_LOG_SYNC);
>
> - /* reclaim inodes to do any IO before the freeze completes */
> - xfs_reclaim_inodes(mp, 0);
> - xfs_reclaim_inodes(mp, SYNC_WAIT);
>
> /* Push the superblock and write an unmount record */
> error = xfs_log_sbcount(mp);
> --
> 2.26.2.761.g0e0b3e54be
>
