On 9/3/19 12:43 AM, Christoph Hellwig wrote:
On Sun, Sep 01, 2019 at 03:08:21PM -0500, Goldwyn Rodrigues wrote:
This is an effort to use iomap for btrfs. This would keep most
responsibility of page handling during writes in iomap code, hence
code reduction. For CoW support, changes are needed in iomap code
to make sure we perform a copy before the write.
This is in line with the discussion we had during adding dax support in
btrfs.
This looks pretty good modulo a few comments.
Can you please convert the XFS code to use your two iomaps for COW
approach as well to validate it?
Hi,
The XFS part of dax CoW support has been implementing recently. Please
review this[1] if necessary. It's based on this iomap patchset(the 1st
version), and uses the new srcmap.
[1]: https://lkml.org/lkml/2019/7/31/449
--
Thanks,
Shiyang Ruan.
Also the iomap_file_dirty helper would really benefit from using the
two iomaps, any chance you could look into improving it to use your
new infrastructure?
