On Mon, Nov 09, 2020 at 10:17:15AM -0800, Darrick J. Wong wrote: > From: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > > Pass the same oldext argument (which contains the existing rmapping's > unwritten state) to xfs_rmap_lookup_le_range at the start of > xfs_rmap_convert_shared. At this point in the code, flags is zero, > which means that we perform lookups using the wrong key. Looks good, Reviewed-by: Christoph Hellwig <hch@xxxxxx>