All of lore.kernel.org
 help / color / mirror / Atom feed
From: Al Viro <viro@zeniv.linux.org.uk>
To: "Matthew Wilcox (Oracle)" <willy@infradead.org>
Cc: linux-fsdevel@vger.kernel.org
Subject: Re: [PATCH 0/5] UFS: Final folio conversions
Date: Sat, 5 Oct 2024 22:21:01 +0100	[thread overview]
Message-ID: <20241005212101.GZ4017910@ZenIV> (raw)
In-Reply-To: <20241005183108.GX4017910@ZenIV>

On Sat, Oct 05, 2024 at 07:31:08PM +0100, Al Viro wrote:
> On Sat, Oct 05, 2024 at 07:02:03PM +0100, Matthew Wilcox (Oracle) wrote:
> > This is the last use of struct page I've been able to find in UFS.
> > All the hard work was done earlier; this is just passing in bh->b_folio
> > instead of bh->b_page.
> > 
> > Matthew Wilcox (Oracle) (5):
> >   ufs: Convert ufs_inode_getblock() to take a folio
> >   ufs: Convert ufs_extend_tail() to take a folio
> >   ufs: Convert ufs_inode_getfrag() to take a folio
> >   ufs: Pass a folio to ufs_new_fragments()
> >   ufs: Convert ufs_change_blocknr() to take a folio
> > 
> >  fs/ufs/balloc.c | 16 ++++++++--------
> >  fs/ufs/inode.c  | 30 ++++++++++++++----------------
> >  fs/ufs/ufs.h    |  8 ++++----
> >  3 files changed, 26 insertions(+), 28 deletions(-)
> 
> Looks sane (for now - there's really interesting shite around the block
> relocation, hole filling, etc.); I've got a bunch of UFS patches and IMO
> it makes sense to keep it in the same branch.  Mine is in #work.ufs
> and there's pending work locally.
> 
> I'll add yours to the mix and see if xfstests catch any problems there;
> will post tonight.

No regressions.

      reply	other threads:[~2024-10-05 21:21 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-05 18:02 [PATCH 0/5] UFS: Final folio conversions Matthew Wilcox (Oracle)
2024-10-05 18:02 ` [PATCH 1/5] ufs: Convert ufs_inode_getblock() to take a folio Matthew Wilcox (Oracle)
2024-10-05 18:02 ` [PATCH 2/5] ufs: Convert ufs_extend_tail() " Matthew Wilcox (Oracle)
2024-10-05 18:02 ` [PATCH 3/5] ufs: Convert ufs_inode_getfrag() " Matthew Wilcox (Oracle)
2024-10-05 18:02 ` [PATCH 4/5] ufs: Pass a folio to ufs_new_fragments() Matthew Wilcox (Oracle)
2024-10-05 18:02 ` [PATCH 5/5] ufs: Convert ufs_change_blocknr() to take a folio Matthew Wilcox (Oracle)
2024-10-05 18:31 ` [PATCH 0/5] UFS: Final folio conversions Al Viro
2024-10-05 21:21   ` Al Viro [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20241005212101.GZ4017910@ZenIV \
    --to=viro@zeniv.linux.org.uk \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=willy@infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.