FAILED: patch "[PATCH] mm/hugetlb: do not dissolve gigantic pages without runtime" failed to apply to 6.18-stable tree
Subsystems (3)
[email protected]
maintainerThe patch below does not apply to the 6.18-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.18.y
git checkout FETCH_HEAD
git cherry-pick -x a363c62a653cc8b3e21da9545fa4e028ef50f9c3
# <resolve conflicts, build, test, etc.>
git commit -s
git send-email --to '<[email protected]>' --in-reply-to '2026092902-struck-raving-0494@gregkh' --subject-prefix 'PATCH 6.18.y' 'HEAD^..'
Possible dependencies:
thanks,
greg k-h
------------------ original commit in Linus's tree ------------------
From a363c62a653cc8b3e21da9545fa4e028ef50f9c3 Mon Sep 17 00:00:00 2001
From: Longlong Xia <[email protected]>
Date: Sun, 23 Aug 2026 12:40:51 +0800
Subject: [PATCH] mm/hugetlb: do not dissolve gigantic pages without runtime
support
dissolve_free_hugetlb_folio() doesn't check hstate_is_gigantic_no_runtime(h) though remove_hugetlb_folio()/ update_and_free_hugetlb_folio() silently bail for such folios, so it frees a still-listed folio and, on vmemmap restore failure, the add_hugetlb_folio() rollback corrupts the free list.
mm/hugetlb.c +9 -0
@@ -1967,6 +1967,15 @@ int dissolve_free_hugetlb_folio(struct folio *folio)struct hstate *h =folio_hstate (folio);bool adjust_surplus = false; +/* +* remove_hugetlb_folio()/update_and_free_hugetlb_folio() bail +* for gigantic hstates without runtime support, so dissolving one +* here would leave it on the free list and, on vmemmap restore +* failure, the add_hugetlb_folio() rollback corrupts that list. +*/ +if (hstate_is_gigantic_no_runtime (h)) + goto out; +if (!available_huge_pages (h)) goto out;
Sasha Levin
maintainerPATCH 1/2 mm/hugetlb: create hstate_is_gigantic_no_runtime helper
From: Usama Arif <[email protected]>
[ Upstream commit a743e0af503a633e4ca68a100d9b2a1a071fe8ae ]
This is a common condition used to skip operations that cannot be performed on gigantic pages when runtime support is disabled. This helper is introduced as the condition will exist even more when allowing "overcommit" of gigantic hugepages. No functional change intended with this patch.
Stable-dep-of: a363c62a653c ("mm/hugetlb: do not dissolve gigantic pages without runtime support")
--- mm/hugetlb.c | 21 ++++++++++++++++----- 1 file changed, 16 insertions(+), 5 deletions(-)
mm/hugetlb.c +16 -5
@@ -138,6 +138,17 @@ static void hugetlb_free_folio(struct folio *folio)folio_put (folio); } +/* +* Check if the hstate represents gigantic pages but gigantic page +* runtime support is not available. This is a common condition used to +* skip operations that cannot be performed on gigantic pages when runtime +* support is disabled. +*/ +static inline bool hstate_is_gigantic_no_runtime (struct hstate *h) +{ +return hstate_is_gigantic (h) && !gigantic_page_runtime_supported (); +} +static inline bool subpool_is_free (struct hugepage_subpool *spool) {if (spool->count)
@@ -1538,7 +1549,7 @@ static void remove_hugetlb_folio(struct hstate *h, struct folio *folio,VM_BUG_ON_FOLIO (hugetlb_cgroup_from_folio_rsvd (folio), folio);lockdep_assert_held (&hugetlb_lock); -if (hstate_is_gigantic (h) && !gigantic_page_runtime_supported ()) +if (hstate_is_gigantic_no_runtime (h))return ;list_del (&folio->lru);
@@ -1600,7 +1611,7 @@ static void __update_and_free_hugetlb_folio(struct hstate *h, {bool clear_flag =folio_test_hugetlb_vmemmap_optimized (folio); -if (hstate_is_gigantic (h) && !gigantic_page_runtime_supported ()) +if (hstate_is_gigantic_no_runtime (h))return ;/*
@@ -2496,7 +2507,7 @@ static void return_unused_surplus_pages(struct hstate *h,/* Uncommit the reservation */ h->resv_huge_pages -= unused_resv_pages; -if (hstate_is_gigantic (h) && !gigantic_page_runtime_supported ()) +if (hstate_is_gigantic_no_runtime (h)) goto out;/*
@@ -3711,7 +3722,7 @@ static void __init hugetlb_init_hstates(void)* - If CMA allocation is possible, we can not demote * HUGETLB_PAGE_ORDER or smaller size pages. */ -if (hstate_is_gigantic (h) && !gigantic_page_runtime_supported ()) +if (hstate_is_gigantic_no_runtime (h))continue ;if (hugetlb_cma_total_size () && h->order <=HUGETLB_PAGE_ORDER )continue ;
@@ -4188,7 +4199,7 @@ static ssize_t __nr_hugepages_store_common(bool obey_mempolicy,int err;nodemask_t nodes_allowed, *n_mask; -if (hstate_is_gigantic (h) && !gigantic_page_runtime_supported ()) +if (hstate_is_gigantic_no_runtime (h))return -EINVAL ;if (nid ==NUMA_NO_NODE ) {
Signature · 1 line
2.53.0
Sasha Levin
maintainerPATCH 2/2 mm/hugetlb: do not dissolve gigantic pages without runtime support
From: Longlong Xia <[email protected]>
[ Upstream commit a363c62a653cc8b3e21da9545fa4e028ef50f9c3 ]
dissolve_free_hugetlb_folio() doesn't check hstate_is_gigantic_no_runtime(h) though remove_hugetlb_folio()/ update_and_free_hugetlb_folio() silently bail for such folios, so it frees a still-listed folio and, on vmemmap restore failure, the add_hugetlb_folio() rollback corrupts the free list.
--- mm/hugetlb.c | 9 +++++++++ 1 file changed, 9 insertions(+)
mm/hugetlb.c +9 -0
@@ -2126,6 +2126,15 @@ int dissolve_free_hugetlb_folio(struct folio *folio)struct hstate *h =folio_hstate (folio);bool adjust_surplus = false; +/* +* remove_hugetlb_folio()/update_and_free_hugetlb_folio() bail +* for gigantic hstates without runtime support, so dissolving one +* here would leave it on the free list and, on vmemmap restore +* failure, the add_hugetlb_folio() rollback corrupts that list. +*/ +if (hstate_is_gigantic_no_runtime (h)) + goto out; +if (!available_huge_pages (h)) goto out;
Signature · 1 line
2.53.0