mbox series

[00/13] fuse: use folios instead of pages for requests

Message ID 20241002165253.3872513-1-joannelkoong@gmail.com (mailing list archive)
Headers show
Series fuse: use folios instead of pages for requests | expand

Message

Joanne Koong Oct. 2, 2024, 4:52 p.m. UTC 
This patchset converts fuse requests to use folios instead of pages. Right
now, all folios in fuse are one page, but a subsequent patchset will be
enabling larger-size folios on fuse.

This patchset has no functional changes and have been run through fstests with
passthrough_hp.

This patchset is dependent on (and rebased on top of) Josef's folio conversions
patchset here:
https://lore.kernel.org/linux-fsdevel/cover.1727703714.git.josef@toxicpanda.com/

Joanne Koong (13):
  fuse: support folios in struct fuse_args_pages and fuse_copy_pages()
  fuse: add support in virtio for requests using folios
  fuse: convert cuse to use folios
  fuse: convert readlink to use folios
  fuse: convert readdir to use folios
  fuse: convert reads to use folios
  fuse: convert writes (non-writeback) to use folios
  fuse: convert ioctls to use folios
  fuse: convert retrieves to use folios
  fuse: convert writebacks to use folios
  mm/writeback: add folio_mark_dirty_lock()
  fuse: convert direct io to use folios
  fuse: remove pages for requests and exclusively use folios

 fs/fuse/cuse.c      |  31 ++---
 fs/fuse/dev.c       |  40 +++---
 fs/fuse/dir.c       |  28 ++---
 fs/fuse/file.c      | 291 +++++++++++++++++++++++---------------------
 fs/fuse/fuse_i.h    |  36 +++++-
 fs/fuse/ioctl.c     |  31 +++--
 fs/fuse/readdir.c   |  20 +--
 fs/fuse/virtio_fs.c |  57 +++++----
 include/linux/mm.h  |   1 +
 mm/page-writeback.c |  12 ++
 10 files changed, 303 insertions(+), 244 deletions(-)

Comments

Josef Bacik Oct. 18, 2024, 8:07 p.m. UTC | #1 
On Wed, Oct 02, 2024 at 09:52:40AM -0700, Joanne Koong wrote:
> This patchset converts fuse requests to use folios instead of pages. Right
> now, all folios in fuse are one page, but a subsequent patchset will be
> enabling larger-size folios on fuse.
> 
> This patchset has no functional changes and have been run through fstests with
> passthrough_hp.
> 
> This patchset is dependent on (and rebased on top of) Josef's folio conversions
> patchset here:
> https://lore.kernel.org/linux-fsdevel/cover.1727703714.git.josef@toxicpanda.com/

This is a really good example of how to do a big conversion, it was really easy
to review.  Address the few nits and rebase and then you can add

Reviewed-by: Josef Bacik <josef@toxicpanda.com>

Thanks,

Josef