From: Andreas Gruenbacher Subject: Re: iomap preparations for GFS2 v2 Date: Thu, 14 Jun 2018 15:04:38 +0200 Message-ID: References: <20180614120457.28285-1-hch@lst.de> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Cc: linux-xfs@vger.kernel.org, linux-fsdevel , Dan Williams , cluster-devel , linux-ext4 To: Christoph Hellwig Return-path: In-Reply-To: <20180614120457.28285-1-hch@lst.de> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: cluster-devel-bounces@redhat.com Errors-To: cluster-devel-bounces@redhat.com List-Id: linux-ext4.vger.kernel.org On 14 June 2018 at 14:04, Christoph Hellwig wrote: > Hi all, > > this is a slight rework of the patches from Andreas to prepare for gfs2 > using the iomap code. > > Note: I'd like to start with an immutable branch for iomap patches in either > the XFS tree or a tree of mine at the beginning of the merge window so that > we have a common base for the GFS2 and XFS iomap work. > > Changes since v1: > - move code to a different spot in iomap.c to prefer for readpages > inline data support > - add a forward declaration for struct page to iomap.h > - fix a typo in the dax_dev/bdev unioning > - fix a comment typo I saw that you've pushed this onto the gfs2-iomap branch in your xfs repository. I've rebased the gfs2 iomap-write branch onto that; there's a trivial patch for adding a private pointer to struct iomap at the head of that branch that would sense to move to the shared branch as well now. The next step would probably be to start using iomap_readpage / iomap_readpages in gfs2 for block size == page size. This requires adding inline data support to iomap_readpage which is trivial, but because of gfs2's reliance on buffer heads, that alone isn't enough. Thanks, Andreas