FAILED: patch "[PATCH] mm/khugepaged: preserve dirty bits when collapsing PTE-mapped THPs" failed to apply to 6.1-stable tree
Subsystems (3)
Sasha Levin
maintainerThe patch below does not apply to the 6.1-stable tree.
If someone wants it applied there, or to any other stable or longterm tree, then please email the backport, including the original git commit id to <[email protected]>.
To reproduce the conflict and resubmit, you may use the following commands:
git fetch https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/ linux-6.1.y
git checkout FETCH_HEAD
git cherry-pick -x fe2817e3d5ea347214342c9a5cd8bcbff20c0574
# <resolve conflicts, build, test, etc.>
git commit -s
git send-email --to '<[email protected]>' --in-reply-to '[email protected]' --subject-prefix 'PATCH 6.1.y' 'HEAD^..'
Thanks,
Sasha
------------------ original commit in Linus's tree ------------------
From fe2817e3d5ea347214342c9a5cd8bcbff20c0574 Mon Sep 17 00:00:00 2001
From: Chris Mason <[email protected]>
Date: Sat, 31 Oct 2026 10:39:46 +0100
Subject: [PATCH] mm/khugepaged: preserve dirty bits when collapsing PTE-mapped
THPs
try_collapse_pte_mapped_thp() clears PTEs without transferring their dirty bits to the folio. A shmem read fault can map a clean folio writable, so a later store only dirties the PTE. Losing that dirty bit allows reclaim to discard modified data.
The assumption that regular files are read-only is also obsolete since commit 1b6444331bebb ("mm/khugepaged: enable clean pagecache folio collapse for writable files"). In any case, transferring PTE dirty bits to the folio when removing mappings (and while the folio is still mapped) is the right thing to do.
Use get_and_clear_ptes() to collect the dirty bits and mark the folio dirty before unmapping it. Update the comment.
This was reproduced with shmem using MADV_COLLAPSE followed by MADV_PAGEOUT.
Fix generated by an LLM and sent by Chris Mason in private.
--- mm/khugepaged.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)
mm/khugepaged.c +3 -2
@@ -1995,9 +1995,10 @@ static enum scan_result try_collapse_pte_mapped_thp(struct mm_struct *mm, unsign/* * Must clear entry, or a racing truncate may re-remove it. * TLB flush can be left until pmdp_collapse_flush() does it. -* PTE dirty? Shmem page is already dirty; file is read-only. */ -clear_ptes (mm, addr, pte, nr_batch_ptes); + ptent =get_and_clear_ptes (mm, addr, pte, nr_batch_ptes); +if (pte_dirty (ptent)) +folio_mark_dirty (folio);folio_remove_rmap_ptes (folio, page, nr_batch_ptes, vma); nr_mapped_ptes += nr_batch_ptes; }
Signature · 1 line
2.53.0