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 <rpeterso@xxxxxxxxxx>
Reviewed-by: Bob Peterson <rpeterso@xxxxxxxxxx>
Regards,
Bob Peterson