From: Matthew Wilcox <willy@infradead.org>
To: David Hildenbrand <david@redhat.com>
Cc: "lsf-pc@lists.linux-foundation.org"
<lsf-pc@lists.linux-foundation.org>,
linux-fsdevel@vger.kernel.org,
"linux-mm@kvack.org" <linux-mm@kvack.org>,
Jeff Layton <jlayton@kernel.org>, Zi Yan <ziy@nvidia.com>,
Shakeel Butt <shakeel.butt@linux.dev>,
Joanne Koong <joannelkoong@gmail.com>,
Miklos Szeredi <miklos@szeredi.hu>
Subject: Re: [LSF/MM/BPF TOPIC] Migrating the un-migratable
Date: Thu, 30 Jan 2025 21:48:42 +0000 [thread overview]
Message-ID: <Z5vzuii9-zS-WsCH@casper.infradead.org> (raw)
In-Reply-To: <882b566c-34d6-4e68-9447-6c74a0693f18@redhat.com>
On Wed, Jan 29, 2025 at 05:10:03PM +0100, David Hildenbrand wrote:
> As one example, in context of FUSE we recently discovered that folios that
> are under writeback cannot be migrated, and user space in control of when
> writeback will end. Something similar can happen ->readahead() where user
> space is in charge of supplying page content. Networking filesystems in
> general seem to be prone to this as well.
You're not wrong. The question is whether we're willing to put the
asterisk on "In the presence of a misbehaving server (network or fuse),
our usual guarantees do not apply". I'm not sure it's a soluble
problem, though. Normally writeback (or, as you observed, readahead)
completes just fine and we don't need to use non-movable pages for them.
But if someone trips over the network cable, anything in flight becomes
unmovable until someone plugs it back in. We've given the DMA address
of the memory to a network adapter, and that's generally a non-revokable
step (maybe the iommu could save us, but at what cost?)
> As another example, failing to split large folios can prevent migration if
> memory is fragmented. XFS (IOMAP in general) refuses to split folios that
> are dirty [3]. Splitting of folios and page migration have a lot in common.
Welll ... yes and no. iomap refuses to split a dirty folio because it
has a per-folio data structure which tells us which blocks in the folio
are dirty. If we split the folio, we have to allocate an extra data
structure for each new folio that we create. It's not impossible, but
it's a big ask for slab. It'll be a lot better once Zi Yan's patch is
in to only split folios as needed rather than all the way.
That problem doesn't arise for migration. filemap_release_folio() is
only called by fallback_migrate_folio(), which is only called if the
filesystem doesn't provide a ->migrate_folio callback. All iomap
users should use filemap_migrate_folio() which just has to move the
data structure from the old folio to the new.
next prev parent reply other threads:[~2025-01-30 21:48 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-01-29 16:10 David Hildenbrand
2025-01-30 19:39 ` Frank van der Linden
2025-01-31 9:21 ` Shakeel Butt
2025-01-30 21:48 ` Matthew Wilcox [this message]
2025-01-30 22:33 ` Shakeel Butt
2025-01-30 22:52 ` David Hildenbrand
2025-03-24 18:55 ` David Hildenbrand
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=Z5vzuii9-zS-WsCH@casper.infradead.org \
--to=willy@infradead.org \
--cc=david@redhat.com \
--cc=jlayton@kernel.org \
--cc=joannelkoong@gmail.com \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=lsf-pc@lists.linux-foundation.org \
--cc=miklos@szeredi.hu \
--cc=shakeel.butt@linux.dev \
--cc=ziy@nvidia.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox