Commit b943c0cd authored Jan 22, 2021 by Darrick J. Wong

xfs: hide xfs_icache_free_cowblocks



Change the one remaining caller of xfs_icache_free_cowblocks to use our
new combined blockgc scan function instead, since we will soon be
combining the two scans.  This introduces a slight behavior change,
since a readonly remount now clears out post-EOF preallocations and not
just CoW staging extents.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>

parent 0461a320

fs/xfs/xfs_icache.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -1567,7 +1567,7 @@ xfs_inode_free_cowblocks(
		return ret;
		}

		int
		static int
		xfs_icache_free_cowblocks(
		struct xfs_mount *mp,
		struct xfs_eofblocks *eofb)

fs/xfs/xfs_icache.h

+0 −1

Original line number	Diff line number	Diff line
		@@ -67,7 +67,6 @@ void xfs_queue_eofblocks(struct xfs_mount *);

		void xfs_inode_set_cowblocks_tag(struct xfs_inode *ip);
		void xfs_inode_clear_cowblocks_tag(struct xfs_inode *ip);
		int xfs_icache_free_cowblocks(struct xfs_mount , struct xfs_eofblocks );
		void xfs_cowblocks_worker(struct work_struct *);
		void xfs_queue_cowblocks(struct xfs_mount *);

fs/xfs/xfs_super.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -1713,7 +1713,7 @@ xfs_remount_ro(
		xfs_stop_block_reaping(mp);

		/* Get rid of any leftover CoW reservations... */
		error = xfs_icache_free_cowblocks(mp, NULL);
		error = xfs_blockgc_free_space(mp, NULL);
		if (error) {
		xfs_force_shutdown(mp, SHUTDOWN_CORRUPT_INCORE);
		return error;