From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bob Peterson Date: Fri, 2 Jun 2023 07:30:46 -0500 Subject: [Cluster-devel] [PATCH 0/6] gfs2/buffer folio changes In-Reply-To: <20230517032442.1135379-1-willy@infradead.org> References: <20230517032442.1135379-1-willy@infradead.org> Message-ID: <4530554c-c2b3-f93b-6c2c-c411e62d1e45@redhat.com> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit On 5/16/23 10:24 PM, Matthew Wilcox (Oracle) wrote: > This kind of started off as a gfs2 patch series, then became entwined > with buffer heads once I realised that gfs2 was the only remaining > caller of __block_write_full_page(). For those not in the gfs2 world, > the big point of this series is that block_write_full_page() should now > handle large folios correctly. > > It probably makes most sense to take this through Andrew's tree, once > enough people have signed off on it? > > Matthew Wilcox (Oracle) (6): > gfs2: Use a folio inside gfs2_jdata_writepage() > gfs2: Pass a folio to __gfs2_jdata_write_folio() > gfs2: Convert gfs2_write_jdata_page() to gfs2_write_jdata_folio() > buffer: Convert __block_write_full_page() to > __block_write_full_folio() > gfs2: Support ludicrously large folios in gfs2_trans_add_databufs() > buffer: Make block_write_full_page() handle large folios correctly > > fs/buffer.c | 75 ++++++++++++++++++------------------- > fs/gfs2/aops.c | 66 ++++++++++++++++---------------- > fs/gfs2/aops.h | 2 +- > fs/ntfs/aops.c | 2 +- > fs/reiserfs/inode.c | 2 +- > include/linux/buffer_head.h | 2 +- > 6 files changed, 75 insertions(+), 74 deletions(-) > Hi Willy, I did some fundamental testing with this patch set in a five-node cluster, as well as xfstests, and it seemed to work properly. The testing was somewhat limited, but it passed basic cluster coherency testing. Sorry it took so long. If you want you can add: Tested-by: Bob Peterson Reviewed-by: Bob Peterson Regards, Bob Peterson