Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 3af423b0 authored by Darrick J. Wong's avatar Darrick J. Wong
Browse files

xfs: evict CoW fork extents when performing finsert/fcollapse 



When we perform an finsert/fcollapse operation, cancel all the CoW
extents for the affected file offset range so that they don't end up
pointing to the wrong blocks.

Reported-by: default avatarAmir Goldstein <amir73il@gmail.com>
Reviewed-by: default avatarCarlos Maiolino <cmaiolino@redhat.com>
Reviewed-by: default avatarChristoph Hellwig <hch@lst.de>
Signed-off-by: default avatarDarrick J. Wong <darrick.wong@oracle.com>
parent cc6f7771

fs/xfs/xfs_bmap_util.c

+13 −1
Original line number Diff line number Diff line
@@ -1459,7 +1459,19 @@ xfs_shift_file_space( 
		return error;



	/*

	 * The extent shiting code works on extent granularity. So, if

	 * Clean out anything hanging around in the cow fork now that

	 * we've flushed all the dirty data out to disk to avoid having

	 * CoW extents at the wrong offsets.

	 */

	if (xfs_is_reflink_inode(ip)) {

		error = xfs_reflink_cancel_cow_range(ip, offset, NULLFILEOFF,

				true);

		if (error)

			return error;

	}



	/*

	 * The extent shifting code works on extent granularity. So, if

	 * stop_fsb is not the starting block of extent, we need to split

	 * the extent at stop_fsb.

	 */