On Tue, Oct 08, 2019 at 08:59:44AM +1100, Dave Chinner wrote: > On Sun, Oct 06, 2019 at 05:46:06PM +0200, Christoph Hellwig wrote: > > In preparation for moving the writeback code to iomap.c, replace the > > XFS-specific COW fork concept with the iomap IOMAP_F_SHARED flag. > > "In preparation for switching XFS to use the fs/iomap writeback > code..."? > > I suspect the IOMAP_F_SHARED hunk I pointed out in the previous > patch should be in this one... All these made a whole lot more sense before Darrick reshuffled the code. In my original version I started massaging the XFS code so that it can be moved to iomap without functional changes, but Darrick wanted the iomap code added in one series and then XFS switched over, which made a lot of things more confusing than they were intended to be.