From: Kiryl Shutsemau <kirill@shutemov.name>
To: akpm@linux-foundation.org, david@kernel.org, ljs@kernel.org,
nico.pache@linux.dev
Cc: baolin.wang@linux.alibaba.com, baohua@kernel.org,
dev.jain@arm.com, hughd@google.com, lance.yang@linux.dev,
liam@infradead.org, mhocko@suse.com, rppt@kernel.org,
ryan.roberts@arm.com, shuah@kernel.org, surenb@google.com,
usama.arif@linux.dev, vbabka@kernel.org, ziy@nvidia.com,
usama.anjum@arm.com, agordeev@linux.ibm.com, linux-mm@kvack.org,
linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org,
kas@kernel.org, jannh@google.com, willy@infradead.org,
pfalcato@suse.de, rostedt@goodmis.org, mhiramat@kernel.org,
linux-trace-kernel@vger.kernel.org, bpf@vger.kernel.org
Subject: [RFC PATCH 15/57] mm/collapse: check what a candidate would freeze
Date: Sun, 16 Aug 2026 23:45:27 +0100 [thread overview]
Message-ID: <20260816224609.308019-16-kirill@shutemov.name> (raw)
In-Reply-To: <20260816224609.308019-1-kirill@shutemov.name>
From: "Kiryl Shutsemau (Meta)" <kas@kernel.org>
The freeze takes folio locks, rewrites PTEs and flushes the TLB, and any
of that has to be undone slot by slot if the candidate turns out unfit --
while faulters on those sources wait. So it decides first and acts
second.
This is the deciding half: walk every slot a candidate covers, under the
table's ptl, and answer whether all of it can be frozen. It touches
nothing, so a refusal costs the round only the walk.
The walk goes in source spans, a span being consecutive PTEs mapping
consecutive pages of one folio. No layout is refused for its shape:
where a span ends, the next slot starts one of its own, which is what
lets partially mapped and compound sources collapse. A slot may also be
a hole or the zeropage, both of which the destination just zero-fills.
What a span has to satisfy, beyond being present, anonymous and not
uffd-armed:
- Every live mapping of its folio is this span. The freeze is
whole-folio, so a live PTE anywhere else would race a zap whose
folio_put() underflows the frozen count. Under the ptl this is exact,
since fork -- the only way an exclusive anon folio gains mappings --
takes mmap_write.
- Every page of it is PageAnonExclusive(). A shared folio has no
refcount the freeze can pin down without the other mappers' ptls.
- It is not MADV_FREE'd, unless the caller asked for the collapse.
Copying a lazyfree page into a folio that is not lazyfree would quietly
make memory the user offered up undroppable again, which is why the
policy carries that choice.
Sub-PMD candidates also refuse folios already at or above their own order,
there being nothing to gain; a PMD candidate takes them, that being the
PTE-mapped-THP re-collapse case.
SCAN_PAGE_NOT_EXCLUSIVE joins enum scan_result and the trace symbol list.
Assisted-by: Claude-Code:claude-opus-5
Signed-off-by: Kiryl Shutsemau (Meta) <kas@kernel.org>
---
include/trace/events/huge_memory.h | 1 +
mm/collapse.c | 177 +++++++++++++++++++++++++++++
mm/collapse.h | 1 +
3 files changed, 179 insertions(+)
diff --git a/include/trace/events/huge_memory.h b/include/trace/events/huge_memory.h
index 68693eba82ef..ff938ac9c43c 100644
--- a/include/trace/events/huge_memory.h
+++ b/include/trace/events/huge_memory.h
@@ -41,6 +41,7 @@
EM( SCAN_COPY_MC, "copy_poisoned_page") \
EM( SCAN_PAGE_FILLED, "page_filled") \
EM( SCAN_PAGE_DIRTY_OR_WRITEBACK, "page_dirty_or_writeback") \
+ EM( SCAN_PAGE_NOT_EXCLUSIVE, "page_not_exclusive") \
EMe(SCAN_ALLOC_LIGHT_MISS, "alloc_light_miss")
#undef EM
diff --git a/mm/collapse.c b/mm/collapse.c
index 4ec02071f588..c75d91cb9d48 100644
--- a/mm/collapse.c
+++ b/mm/collapse.c
@@ -386,6 +386,145 @@ static enum scan_result collapse_faultin(struct vm_area_struct *vma,
return result;
}
+/*
+ * How many slots a source span starting at @first may cover: the pages left in
+ * its folio, capped at @max. Every freeze-side walker bounds spans with this,
+ * so per-span batching of clears, locks and freezes cannot reach a slot the span
+ * does not cover.
+ */
+static unsigned int collapse_span_max(pte_t first, unsigned int max)
+{
+ struct page *page = pte_page(first);
+ struct folio *folio = page_folio(page);
+ unsigned int left = folio_nr_pages(folio) - folio_page_idx(folio, page);
+
+ return min(max, left);
+}
+
+/*
+ * Can this candidate's sources be frozen? Every slot is checked and nothing is
+ * touched, so a refusal costs the round nothing but the walk.
+ *
+ * The walk is in source spans: a span is consecutive PTEs mapping consecutive
+ * pages of one folio, and it ends wherever the next PTE stops being the folio's
+ * next page. No layout is refused for its shape -- the next slot simply starts
+ * its own span -- so partially mapped and compound sources collapse too.
+ *
+ * Caller holds mmap_read and the table's ptl.
+ */
+static enum scan_result collapse_check_candidate(struct vm_area_struct *vma,
+ struct collapse_control *cc,
+ struct collapse_candidate *cand,
+ pte_t *pte)
+{
+ const unsigned int nr_pages = candidate_nr_pages(cand);
+ unsigned long addr;
+ unsigned int i;
+
+ for (i = 0, addr = cand->addr; i < nr_pages;) {
+ pte_t ptent = ptep_get(pte + i);
+ unsigned int nr, nr_max, k;
+ struct folio *folio;
+ struct page *page;
+
+ if (!pte_present(ptent)) {
+ /* Holes are population; swap and markers are not */
+ if (pte_none(ptent)) {
+ i++;
+ addr += PAGE_SIZE;
+ continue;
+ }
+ return SCAN_PTE_NON_PRESENT;
+ }
+ if (pte_uffd(ptent))
+ return SCAN_PTE_UFFD;
+
+ /* The zeropage zero-fills like a hole, and has no normal page */
+ if (is_zero_pfn(pte_pfn(ptent))) {
+ i++;
+ addr += PAGE_SIZE;
+ continue;
+ }
+ page = vm_normal_page(vma, addr, ptent);
+ if (!page || unlikely(is_zone_device_page(page)))
+ return SCAN_PAGE_NULL;
+
+ folio = page_folio(page);
+ if (!folio_test_anon(folio))
+ return SCAN_PAGE_ANON;
+
+ /*
+ * Collapsing a MADV_FREE'd page would copy it into a folio that
+ * is not lazyfree, quietly making memory the user offered up
+ * undroppable again.
+ */
+ if (cc->policy.skip_lazyfree &&
+ !(vma->vm_flags & VM_DROPPABLE) &&
+ folio_test_lazyfree(folio) && !pte_dirty(ptent))
+ return SCAN_PAGE_LAZYFREE;
+
+ /*
+ * A sub-PMD candidate refuses folios of its own order and above:
+ * collapsing those would gain nothing. A PMD candidate accepts
+ * every order up to its own -- the PTE-mapped-THP re-collapse
+ * class.
+ */
+ if (folio_order(folio) >= cand->order &&
+ !is_pmd_order(cand->order))
+ return SCAN_PTE_MAPPED_HUGEPAGE;
+
+ /*
+ * Exclusive anon only: the expected refcount of a shared folio
+ * cannot be pinned down without its other mappers' ptls.
+ * Swapcache membership is fine -- folio_expected_ref_count()
+ * accounts those references.
+ */
+ if (folio_maybe_mapped_shared(folio))
+ return SCAN_PAGE_NOT_EXCLUSIVE;
+
+ nr_max = collapse_span_max(ptent, nr_pages - i);
+ for (nr = 1; nr < nr_max; nr++) {
+ pte_t tail = ptep_get(pte + i + nr);
+
+ if (!pte_present(tail) ||
+ pte_pfn(tail) != pte_pfn(ptent) + nr)
+ break;
+ if (pte_uffd(tail))
+ return SCAN_PTE_UFFD;
+ }
+
+ /*
+ * Every live mapping of the folio must be this span: the freeze
+ * is whole-folio, and a live PTE left anywhere else loses to a
+ * racing zap -- its rmap drop is paired with a folio_put() that
+ * would underflow the frozen count. The check is race-free
+ * under our ptl: in-window PTEs are ours, fork (the only way
+ * exclusive anon gains mappings) takes mmap_write, and a folio
+ * whose mappings all sit under this ptl cannot lose one either.
+ * This also refuses a folio scattered across several spans of
+ * the window, whose mapcount exceeds any single span.
+ */
+ if (folio_mapcount(folio) != nr)
+ return SCAN_PAGE_COUNT;
+
+ /*
+ * Every page of the span must be exclusive: the freeze accounts
+ * only references it can see, and a non-exclusive page may be
+ * unshared under us. collapse_faultin() should have arranged
+ * this; enforce it here, where it is depended on.
+ */
+ for (k = 0; k < nr; k++) {
+ if (!PageAnonExclusive(pte_page(ptep_get(pte + i + k))))
+ return SCAN_PAGE_NOT_EXCLUSIVE;
+ }
+
+ i += nr;
+ addr += nr * PAGE_SIZE;
+ }
+
+ return SCAN_SUCCEED;
+}
+
/*
* Raise the two barriers on the sources of every candidate: migration entries in
* their PTEs, then a frozen refcount. Takes the table's ptl once for the whole
@@ -395,6 +534,44 @@ static enum scan_result collapse_faultin(struct vm_area_struct *vma,
static void collapse_freeze(struct vm_area_struct *vma,
struct collapse_control *cc, pmd_t *pmd)
{
+ struct mm_struct *mm = vma->vm_mm;
+ pte_t *pte, *table;
+ spinlock_t *ptl;
+ unsigned int i;
+
+ pte = pte_offset_map_lock(mm, pmd, cc->candidates[0].addr, &ptl);
+ if (!pte) {
+ for (i = 0; i < cc->nr_candidates; i++) {
+ struct collapse_candidate *cand = &cc->candidates[i];
+
+ if (cand->state != CAND_SELECTED)
+ continue;
+ cand->state = CAND_SKIPPED;
+ cand->result = SCAN_NO_PTE_TABLE;
+ }
+ return;
+ }
+
+ /*
+ * Index each candidate from the table base, not relative to
+ * candidates[0]: a round is not necessarily address-ordered, so
+ * candidates[0] need not be the lowest. They all share one table.
+ */
+ table = pte - pte_index(cc->candidates[0].addr);
+
+ for (i = 0; i < cc->nr_candidates; i++) {
+ struct collapse_candidate *cand = &cc->candidates[i];
+ pte_t *cand_pte = table + pte_index(cand->addr);
+
+ if (cand->state != CAND_SELECTED)
+ continue;
+
+ cand->result = collapse_check_candidate(vma, cc, cand, cand_pte);
+ if (cand->result != SCAN_SUCCEED)
+ cand->state = CAND_SKIPPED;
+ }
+
+ pte_unmap_unlock(pte, ptl);
}
/*
diff --git a/mm/collapse.h b/mm/collapse.h
index 0d6f77a7233b..747168104a72 100644
--- a/mm/collapse.h
+++ b/mm/collapse.h
@@ -46,6 +46,7 @@ enum scan_result {
SCAN_COPY_MC,
SCAN_PAGE_FILLED,
SCAN_PAGE_DIRTY_OR_WRITEBACK,
+ SCAN_PAGE_NOT_EXCLUSIVE,
SCAN_ALLOC_LIGHT_MISS,
};
--
2.54.0
next prev parent reply other threads:[~2026-08-16 22:46 UTC|newest]
Thread overview: 64+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-16 22:45 [RFC PATCH 00/57] mm/collapse: rebuild collapse on migration primitives Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 01/57] mm: add pte_folio() Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 02/57] mm: add pte_none_or_zero() Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 03/57] mm/collapse: add collapse.h for the shared collapse state Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 04/57] mm/collapse: rename mthp_present_ptes to eligible_ptes Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 05/57] mm/collapse: state what a collapse may do in the policy Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 06/57] mm/collapse: move the smallest collapse order to collapse.h Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 07/57] mm/collapse: sketch the new anonymous collapse engine Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 08/57] mm/collapse: scan a table for what a collapse could use Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 09/57] mm/collapse: collect candidate windows into a round Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 10/57] mm/collapse: run a round and feed the outcomes back Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 11/57] mm/collapse: sketch the passes of a round Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 12/57] mm/collapse: allocate a destination per candidate Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 13/57] mm/collapse: revalidate a round against the VMA Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 14/57] mm/collapse: fault the sources in before the freeze Kiryl Shutsemau
2026-08-16 22:45 ` Kiryl Shutsemau [this message]
2026-08-16 22:45 ` [RFC PATCH 16/57] mm/collapse: freeze the sources behind migration entries Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 17/57] mm/collapse: copy the sources into the destinations Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 18/57] mm/collapse: install the destinations at PTE level Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 19/57] mm/collapse: install a PMD leaf as the terminal layer Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 20/57] mm/collapse: put the sources back Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 21/57] mm/collapse: settle whatever the round reached Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 22/57] mm/collapse: walk a table with a selection cursor Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 23/57] mm/collapse: give a refused region a second chance Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 24/57] mm/collapse: report each candidate's outcome to tracing Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 25/57] mm/collapse: collapse anonymous memory with the new engine Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 26/57] mm/collapse: give collapse_single_pmd() the range to work on Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 27/57] mm/collapse: scan the windows a VMA can hold Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 28/57] mm/collapse: remove the mechanism the engine replaces Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 29/57] mm/collapse: move what a collapse is judged on into collapse.c Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 30/57] mm/collapse: name the max_ptes ceiling after collapse Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 31/57] mm/khugepaged: count collapses where khugepaged makes them Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 32/57] mm/collapse: move the file collapse into collapse.c Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 33/57] mm/collapse: split collapse into a scan and a run Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 34/57] mm/collapse: implement MADV_COLLAPSE in madvise.c Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 35/57] mm/madvise: drop MADV_COLLAPSE's redundant mm reference Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 36/57] mm/collapse: report what the scan found Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 37/57] mm/collapse: report what the fault-in pass paid Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 38/57] mm/collapse: report the round, and what it made faulters wait Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 39/57] mm/collapse: name the file collapse's tracepoints after collapse Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 40/57] mm/collapse: remove the tracepoints of the mechanism that is gone Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 41/57] mm/collapse: give collapse its own trace header Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 42/57] mm/collapse: allow error injection into the freeze Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 43/57] mm/khugepaged: check the scan budget before the work, not after Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 44/57] mm/khugepaged: hold the address space open across a scan Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 45/57] mm/collapse: take a per-VMA read lock for the round Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 46/57] mm/khugepaged: scan under a per-VMA read lock Kiryl Shutsemau
2026-08-16 22:45 ` [RFC PATCH 47/57] mm/madvise: collapse " Kiryl Shutsemau
2026-08-16 22:46 ` [RFC PATCH 48/57] mm/collapse: assert the mm reference the engine relies on Kiryl Shutsemau
2026-08-16 22:46 ` [RFC PATCH 49/57] mm/khugepaged: drop the mmap_lock barrier from __khugepaged_exit() Kiryl Shutsemau
2026-08-16 22:46 ` [RFC PATCH 50/57] selftests/mm: attribute collapses by candidate event alone Kiryl Shutsemau
2026-08-16 22:46 ` [RFC PATCH 51/57] selftests/mm: cover collapse inside a sub-PMD VMA Kiryl Shutsemau
2026-08-16 22:46 ` [RFC PATCH 52/57] selftests/mm: cover a hole-y window in " Kiryl Shutsemau
2026-08-16 22:46 ` [RFC PATCH 53/57] selftests/mm: cover collapse of mlocked ranges Kiryl Shutsemau
2026-08-16 22:46 ` [RFC PATCH 54/57] selftests/mm: cover collapse beside a MADV_FREE'd page Kiryl Shutsemau
2026-08-16 22:46 ` [RFC PATCH 55/57] selftests/mm: cover collapse beside a pinned page Kiryl Shutsemau
2026-08-16 22:46 ` [RFC PATCH 56/57] selftests/mm: cover the scaled max_ptes_shared limit Kiryl Shutsemau
2026-08-16 22:46 ` [RFC PATCH 57/57] MAINTAINERS: add an entry for collapse Kiryl Shutsemau
2026-08-17 8:04 ` Lorenzo Stoakes (ARM)
2026-08-17 8:08 ` David Hildenbrand (Arm)
2026-08-17 10:12 ` Kiryl Shutsemau
2026-08-17 2:02 ` [RFC PATCH 00/57] mm/collapse: rebuild collapse on migration primitives Zi Yan
2026-08-17 10:07 ` Kiryl Shutsemau
2026-08-17 8:52 ` Lorenzo Stoakes (ARM)
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=20260816224609.308019-16-kirill@shutemov.name \
--to=kirill@shutemov.name \
--cc=agordeev@linux.ibm.com \
--cc=akpm@linux-foundation.org \
--cc=baohua@kernel.org \
--cc=baolin.wang@linux.alibaba.com \
--cc=bpf@vger.kernel.org \
--cc=david@kernel.org \
--cc=dev.jain@arm.com \
--cc=hughd@google.com \
--cc=jannh@google.com \
--cc=kas@kernel.org \
--cc=lance.yang@linux.dev \
--cc=liam@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-kselftest@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=linux-trace-kernel@vger.kernel.org \
--cc=ljs@kernel.org \
--cc=mhiramat@kernel.org \
--cc=mhocko@suse.com \
--cc=nico.pache@linux.dev \
--cc=pfalcato@suse.de \
--cc=rostedt@goodmis.org \
--cc=rppt@kernel.org \
--cc=ryan.roberts@arm.com \
--cc=shuah@kernel.org \
--cc=surenb@google.com \
--cc=usama.anjum@arm.com \
--cc=usama.arif@linux.dev \
--cc=vbabka@kernel.org \
--cc=willy@infradead.org \
--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