[RESEND PATCH v3 0/2] clean up block_commit_write

Bean Huo posted 2 patches 2 years, 7 months ago
fs/buffer.c                 | 20 ++++++++------------
fs/ext4/move_extent.c       |  7 ++-----
fs/ocfs2/file.c             |  7 +------
fs/udf/file.c               |  6 +++---
include/linux/buffer_head.h |  2 +-
5 files changed, 15 insertions(+), 27 deletions(-)
[RESEND PATCH v3 0/2] clean up block_commit_write
Posted by Bean Huo 2 years, 7 months ago
change log:
    v1--v2:
        1. reordered patches

    v2--v3:
        1. rebased patches to git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next

Bean Huo (2):
  fs/buffer: clean up block_commit_write
  fs: convert block_commit_write to return void

 fs/buffer.c                 | 20 ++++++++------------
 fs/ext4/move_extent.c       |  7 ++-----
 fs/ocfs2/file.c             |  7 +------
 fs/udf/file.c               |  6 +++---
 include/linux/buffer_head.h |  2 +-
 5 files changed, 15 insertions(+), 27 deletions(-)

-- 
2.34.1
Re: [RESEND PATCH v3 0/2] clean up block_commit_write
Posted by Matthew Wilcox 2 years, 7 months ago
On Mon, Jun 26, 2023 at 07:55:16AM +0200, Bean Huo wrote:
> change log:
>     v1--v2:
>         1. reordered patches
> 
>     v2--v3:
>         1. rebased patches to git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next

It's be nice to have a bit of an explanation for the whole series here,
but I think the two patches work standalone.

If you'd like to extend this work, you could convert the callers of
block_commit_write() to use a folio instead of a page and then unify
block_commit_write() and __block_commit_write() as you did in the earlier
version of your patchset.  It shouldn't be too hard, both callers in
ext4 and the caller in iomap are already done.  That just leaves the
three callers in ocfs2 and the one caller in udf.