Hi Christoph, On Mon, 1 Jul 2019 at 23:54, Christoph Hellwig <hch@xxxxxx> wrote: > Hi all, > > in this straight from the jetplane edition I present the series to > convert gfs2 to full iomap usage for the ordered and writeback mode, > that is we use iomap_page everywhere and entirely get rid of > buffer_heads in the data path. thank you very much, this is looking very good. I've done some testing with your cleanups applied so that those can go in in this merge window. The result can be found here: https://git.kernel.org/pub/scm/linux/kernel/git/gfs2/linux-gfs2.git/log/?h=for-next.cleanups Patch "gfs2: implement gfs2_block_zero_range using iomap_zero_range" isn't quite ready: the gfs2 iomap operations don't handle IOMAP_ZERO correctly so far, and that needs to be fixed first. The actual buffer head removal will obviously have to wait a little longer because of the required infrastructure changes, but also because that still needs a lot more review and testing work. > This has only seen basic testing which ensured neither 4k or 1k blocksize > in ordered mode regressed vs the xfstests baseline, although that baseline > tends to look pretty bleak. Some of the tests assume that the filesystem supports unwritten extents, trusted xattrs, the usrquota / grpquota / prjquota mount options. There shouldn't be a huge number of failing tests beyond that, but I know things aren't perfect. > The series is to be applied on top of my "lift the xfs writepage code > into iomap v2" series. Again, thanks a lot for the patches! Andreas