On Tue, Mar 26, 2019 at 02:02:50PM -0500, Goldwyn Rodrigues wrote: > From: Goldwyn Rodrigues <rgoldwyn@xxxxxxxx> > > The IOMAP_F_COW is a flag to notify dax that it needs to copy > the data from iomap->cow_addr to iomap->addr, if the start/end > of I/O are not page aligned. I see what you are trying to do here, but this is kinda gross. > This also introduces dax_to_dax_copy() which performs a copy > from one part of the device to another, to a maximum of one page. > > Question: Using iomap.cow_addr == 0 means the CoW is to be copied > (or memset) from a hole. Would this be better handled through a flag? That's what all these checks in the iomap code do: if (iomap->type == IOMAP_HOLE || iomap->type == IOMAP_UNWRITTEN) Oh, wait, you're trying to map two ranges in a single iomap and then infer state from information that got chucked away.... IOWs, you're doing it wrong - iomap algorithms are driven by how we manipulate iomaps to do data operations efficiently, not how we copy data page by page. IOWs, what we really should have here is two iomaps - a source and a destination iomap. The source is a read mapping of the current address (where we are going to copy the data from), the destination is the post-cow allocation mapping (where the data goes). Now you just copy the data from one map to the other iterating source mappings until the necessary range of the destination has been covered. And you can check if the source map is IOMAP_HOLE or IOMAP_UNWRITTEN and hence optimise the copy (i.e. zero the new allocation) before copying in the new data. Even better, only do the source mapping if the write isn't page/filesystem block aligned, and hence only do the slow path copying if a source mapping exists.... > diff --git a/include/linux/iomap.h b/include/linux/iomap.h > index 0fefb5455bda..391785de1428 100644 > --- a/include/linux/iomap.h > +++ b/include/linux/iomap.h > @@ -35,6 +35,7 @@ struct vm_fault; > #define IOMAP_F_NEW 0x01 /* blocks have been newly allocated */ > #define IOMAP_F_DIRTY 0x02 /* uncommitted metadata */ > #define IOMAP_F_BUFFER_HEAD 0x04 /* file system requires buffer heads */ > +#define IOMAP_F_COW 0x08 /* cow before write */ "Copy on write before write"? :) Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx
