[merged mm-stable] mm-hugetlb-move-mpol-interpretation-out-of-dequeue_hugetlb_folio_vma.patch removed from -mm tree
Andrew Morton <[email protected]> Thu, 30 Jul 2026 19:43:03 -0700
| Newsgroups | org.kernel.vger.mm-commits |
|---|---|
| Message-ID | <[email protected]> |
The quilt patch titled
Subject: mm: hugetlb: move mpol interpretation out of dequeue_hugetlb_folio_vma()
has been removed from the -mm tree. Its filename was
mm-hugetlb-move-mpol-interpretation-out-of-dequeue_hugetlb_folio_vma.patch
This patch was dropped because it was merged into the mm-stable branch
of git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm
------------------------------------------------------
From: Ackerley Tng <[email protected]>
Subject: mm: hugetlb: move mpol interpretation out of dequeue_hugetlb_folio_vma()
Date: Thu, 02 Jul 2026 09:21:47 -0700
Move memory policy interpretation out of dequeue_hugetlb_folio_vma() and
into alloc_hugetlb_folio() to separate reading and interpretation of
memory policy from actual allocation.
Also rename dequeue_hugetlb_folio_vma() to
dequeue_hugetlb_folio_with_mpol() to remove association with vma and to
align with alloc_buddy_hugetlb_folio_with_mpol().
This will later allow memory policy to be interpreted outside of the
process of allocating a hugetlb folio entirely. This opens doors for
other callers of the HugeTLB folio allocation function, such as
guest_memfd, where memory may not always be mapped and hence may not have
an associated vma.
No functional change intended.
Link: https://lore.kernel.org/[email protected]
Signed-off-by: Ackerley Tng <[email protected]>
Reviewed-by: James Houghton <[email protected]>
Cc: Alistair Popple <[email protected]>
Cc: Byungchul Park <[email protected]>
Cc: David Hildenbrand <[email protected]>
Cc: David Rientjes <[email protected]>
Cc: "Edgecombe, Rick P" <[email protected]>
Cc: Frank van der Linden <[email protected]>
Cc: Gregory Price <[email protected]>
Cc: "Huang, Ying" <[email protected]>
Cc: Jason Gunthorpe <[email protected]>
Cc: Jiaqi Yan <[email protected]>
Cc: Joshua Hahn <[email protected]>
Cc: Matthew Brost <[email protected]>
Cc: Michael Roth <[email protected]>
Cc: Michal Hocko <[email protected]>
Cc: Muchun Song <[email protected]>
Cc: Oscar Salvador <[email protected]>
Cc: Paolo Bonzini <[email protected]>
Cc: Pasha Tatashin <[email protected]>
Cc: Peter Xu <[email protected]>
Cc: Pratyush Yadav <[email protected]>
Cc: Qi Zheng <[email protected]>
Cc: Rakie Kim <[email protected]>
Cc: Roman Gushchin <[email protected]>
Cc: Sean Christopherson <[email protected]>
Cc: Shakeel Butt <[email protected]>
Cc: Shivank Garg <[email protected]>
Cc: Vishal Annapurve <[email protected]>
Cc: Yan Zhao <[email protected]>
Cc: Zi Yan <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
---
mm/hugetlb.c | 66 +++++++++++++++++++++++--------------------------
1 file changed, 32 insertions(+), 34 deletions(-)
--- a/mm/hugetlb.c~mm-hugetlb-move-mpol-interpretation-out-of-dequeue_hugetlb_folio_vma
+++ a/mm/hugetlb.c
@@ -1322,32 +1322,26 @@ struct mempolicy_interpreted {
enum mempolicy_mode mode;
};
-static struct folio *dequeue_hugetlb_folio_vma(struct hstate *h,
- struct vm_area_struct *vma,
- unsigned long address)
+static struct folio *dequeue_hugetlb_folio(struct hstate *h, gfp_t gfp_mask,
+ struct mempolicy_interpreted *mpoli)
{
+ nodemask_t *nodemask = mpoli->nodemask;
struct folio *folio = NULL;
- struct mempolicy *mpol;
- gfp_t gfp_mask;
- nodemask_t *nodemask;
- int nid;
-
- gfp_mask = htlb_alloc_mask(h);
- nid = huge_node(vma, address, gfp_mask, &mpol, &nodemask);
- if (mpol_is_preferred_many(mpol)) {
+ if (mpoli->mode == MPOL_PREFERRED_MANY) {
folio = dequeue_hugetlb_folio_nodemask(h, gfp_mask,
- nid, nodemask);
+ mpoli->nid,
+ nodemask);
/* Fallback to all nodes if page==NULL */
nodemask = NULL;
}
- if (!folio)
+ if (!folio) {
folio = dequeue_hugetlb_folio_nodemask(h, gfp_mask,
- nid, nodemask);
-
- mpol_cond_put(mpol);
+ mpoli->nid,
+ nodemask);
+ }
return folio;
}
@@ -2854,7 +2848,11 @@ struct folio *alloc_hugetlb_folio(struct
int ret, idx;
struct hugetlb_cgroup *h_cg = NULL;
struct hugetlb_cgroup *h_cg_rsvd = NULL;
+ struct mempolicy_interpreted mpoli;
gfp_t gfp = htlb_alloc_mask(h);
+ struct mempolicy *mpol;
+ nodemask_t *nodemask;
+ int nid;
idx = hstate_index(h);
@@ -2913,6 +2911,18 @@ struct folio *alloc_hugetlb_folio(struct
if (ret)
goto out_uncharge_cgroup_reservation;
+ /* Takes reference on mpol. */
+ nid = huge_node(vma, addr, gfp, &mpol, &nodemask);
+ mpoli = (struct mempolicy_interpreted){
+ .nid = nid,
+#ifdef CONFIG_NUMA
+ .mode = mpol ? mpol->mode : MPOL_DEFAULT,
+#else
+ .mode = MPOL_DEFAULT,
+#endif
+ .nodemask = nodemask,
+ };
+
spin_lock_irq(&hugetlb_lock);
/*
@@ -2923,35 +2933,23 @@ struct folio *alloc_hugetlb_folio(struct
*/
folio = NULL;
if (!gbl_chg || available_huge_pages(h))
- folio = dequeue_hugetlb_folio_vma(h, vma, addr);
+ folio = dequeue_hugetlb_folio(h, gfp, &mpoli);
if (!folio) {
- struct mempolicy_interpreted mpoli;
- struct mempolicy *mpol;
- nodemask_t *nodemask;
- int nid;
-
spin_unlock_irq(&hugetlb_lock);
- nid = huge_node(vma, addr, gfp, &mpol, &nodemask);
- mpoli = (struct mempolicy_interpreted){
- .nid = nid,
-#ifdef CONFIG_NUMA
- .mode = mpol ? mpol->mode : MPOL_DEFAULT,
-#else
- .mode = MPOL_DEFAULT,
-#endif
- .nodemask = nodemask,
- };
folio = alloc_buddy_hugetlb_folio(h, gfp, &mpoli);
- mpol_cond_put(mpol);
- if (!folio)
+ if (!folio) {
+ mpol_cond_put(mpol);
goto out_uncharge_cgroup;
+ }
spin_lock_irq(&hugetlb_lock);
list_add(&folio->lru, &h->hugepage_activelist);
folio_ref_unfreeze(folio, 1);
/* Fall through */
}
+ mpol_cond_put(mpol);
+
/*
* Either dequeued or buddy-allocated folio needs to add special
* mark to the folio when it consumes a global reservation.
_
Patches currently in -mm which might be from [email protected] are