xfs: retry COW fork delalloc conversion when no extent was found
authorChristoph Hellwig <hch@lst.de>
Fri, 15 Feb 2019 16:02:50 +0000 (08:02 -0800)
committerDarrick J. Wong <darrick.wong@oracle.com>
Sun, 17 Feb 2019 19:55:54 +0000 (11:55 -0800)
commit7588cbeec6df925ef6142a7e48762896c06007a8
treede5220928e6b1aad600a3218840ffa854f2c27cb
parent19c8e4e25866fac5ba7138b902cc45b6d3c8e827
xfs: retry COW fork delalloc conversion when no extent was found

While we can only truncate a block under the page lock for the current
page, there is no high-level synchronization for moving extents from the
COW to the data fork.  This means that for example we can have another
thread doing a direct I/O completion that moves extents from the COW to
the data fork race with writeback.  While this race is very hard to hit
the always_cow seems to reproduce it reasonably well, and it also exists
without that.  Because of that there is a chance that a delalloc
conversion for the COW fork might not find any extents to convert.  In
that case we should retry the whole block lookup and now find the blocks
in the data fork.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
fs/xfs/xfs_aops.c