From: Lorenzo Stoakes <ljs@kernel.org>
To: "David Hildenbrand (Arm)" <david@kernel.org>
Cc: Usama Arif <usama.arif@linux.dev>,
Andrew Morton <akpm@linux-foundation.org>,
chrisl@kernel.org, kasong@tencent.com, ziy@nvidia.com,
linux-mm@kvack.org, ying.huang@linux.alibaba.com,
Baoquan He <baoquan.he@linux.dev>,
willy@infradead.org, youngjun.park@lge.com, hannes@cmpxchg.org,
riel@surriel.com, shakeel.butt@linux.dev, alex@ghiti.fr,
kas@kernel.org, baohua@kernel.org, dev.jain@arm.com,
baolin.wang@linux.alibaba.com, npache@redhat.com,
"Liam R. Howlett" <liam@infradead.org>,
ryan.roberts@arm.com, Vlastimil Babka <vbabka@kernel.org>,
lance.yang@linux.dev, linux-kernel@vger.kernel.org,
nphamcs@gmail.com, shikemeng@huaweicloud.com,
kernel-team@meta.com
Subject: Re: [PATCH 6/6] mm: rename ARCH_ENABLE_THP_MIGRATION to ARCH_SUPPORTS_PMD_SOFTLEAF
Date: Thu, 2 Jul 2026 17:55:13 +0100 [thread overview]
Message-ID: <akaXyc12ajUZqgiu@lucifer> (raw)
In-Reply-To: <e37df165-8de0-4924-ba19-0d43a6b7d36e@kernel.org>
On Wed, Jul 01, 2026 at 10:03:21PM +0200, David Hildenbrand (Arm) wrote:
> On 6/30/26 18:34, Usama Arif wrote:
> > CONFIG_ARCH_ENABLE_THP_MIGRATION started life gating just PMD-level
> > migration entries, but has grown to gate the entire PMD-level softleaf
> > machinery: migration entries, device-private entries, and soon swap
> > entries.
>
> device-private really relies on migration support.
>
> See do_huge_pmd_device_private() where we migrate back to CPU memory.
>
> The whole concept is pased on migration. So that's why it currently makes sense.
>
> So "has grown to" is not quite accurate. It's a different story with swap
> entries (that can exist even without migration support).
>
>
> > Rename CONFIG_ARCH_ENABLE_THP_MIGRATION to CONFIG_ARCH_SUPPORTS_PMD
> > _SOFTLEAF to make this clear. This is a pure rename: the set of
> > selecting architectures (x86, arm64, s390, riscv, loongarch, and
> > powerpc on PPC_BOOK3S_64) and the gating semantics are unchanged.
>
> Hm, that makes me wonder what it would take to make this all just be implicitly
> supported by CONFIG_TRANSPARENT_HUGEPAGE.
>
> Looks like someone would have the courage to touch stuff like arc, mips and
> sparc64 (+arm and x86, but that's less of a concern :) ).
>
> So yeah, renaming the config option makes sense. The config space is the wild west.
>
> We have
>
> PGTABLE_HAS_HUGE_LEAVES
>
> and stuff like
>
> HAVE_ARCH_TRANSPARENT_HUGEPAGE_PUD
> HAVE_ARCH_SOFT_DIRTY
> ARCH_HAS_GIGANTIC_PAGE
> ARCH_HAS_PTE_SPECIAL
>
> We do have a limited number of "SUPPORTS" (which is longer than HAS), but I
> would either use
>
>
> ARCH_HAS_PMD_SOFTLEAVES
>
> or
>
> HAVE_ARCH_PMD_SOFTLEAVES
Yeah it'd be GREAT to simplify all this!
>
>
> Apart from that LGTM.
Can I just say I love this patch in general though? Softleaf all the things :))
>
> --
> Cheers,
>
> David
Cheers, Lorenzo
next prev parent reply other threads:[~2026-07-02 16:55 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-06-30 16:34 [PATCH 0/6] mm: preparatory patches for PMD level swap entries Usama Arif
2026-06-30 16:34 ` [PATCH 1/6] mm: add softleaf_to_pmd() and convert existing callers Usama Arif
2026-06-30 16:34 ` [PATCH 2/6] mm: extract mm_prepare_for_swap_entries() helper Usama Arif
2026-07-01 16:56 ` David Hildenbrand (Arm)
2026-06-30 16:34 ` [PATCH 3/6] fs/proc: use softleaf_has_pfn() in pagemap PMD walker Usama Arif
2026-06-30 16:34 ` [PATCH 4/6] mm/huge_memory: move softleaf_to_folio() inside migration branch Usama Arif
2026-06-30 16:34 ` [PATCH 5/6] mm/migrate_device: move softleaf_to_folio() inside device-private branch Usama Arif
2026-07-01 19:40 ` David Hildenbrand (Arm)
2026-06-30 16:34 ` [PATCH 6/6] mm: rename ARCH_ENABLE_THP_MIGRATION to ARCH_SUPPORTS_PMD_SOFTLEAF Usama Arif
2026-07-01 20:03 ` David Hildenbrand (Arm)
2026-07-01 20:39 ` Zi Yan
2026-07-01 20:50 ` David Hildenbrand (Arm)
2026-07-01 20:55 ` Zi Yan
2026-07-02 16:55 ` Lorenzo Stoakes [this message]
2026-06-30 19:50 ` [PATCH 0/6] mm: preparatory patches for PMD level swap entries Andrew Morton
2026-07-01 8:04 ` Lorenzo Stoakes
2026-07-01 23:46 ` Andrew Morton
2026-07-02 10:48 ` Lorenzo Stoakes
2026-07-02 13:08 ` Mike Rapoport
2026-07-02 16:53 ` Lorenzo Stoakes
2026-07-02 16:57 ` Mark Brown
2026-07-02 17:08 ` Lorenzo Stoakes
2026-07-02 17:10 ` David Hildenbrand (Arm)
2026-07-02 17:21 ` Lorenzo Stoakes
2026-07-01 10:44 ` Usama Arif
2026-07-01 14:09 ` Usama Arif
2026-07-01 20:04 ` David Hildenbrand (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=akaXyc12ajUZqgiu@lucifer \
--to=ljs@kernel.org \
--cc=akpm@linux-foundation.org \
--cc=alex@ghiti.fr \
--cc=baohua@kernel.org \
--cc=baolin.wang@linux.alibaba.com \
--cc=baoquan.he@linux.dev \
--cc=chrisl@kernel.org \
--cc=david@kernel.org \
--cc=dev.jain@arm.com \
--cc=hannes@cmpxchg.org \
--cc=kas@kernel.org \
--cc=kasong@tencent.com \
--cc=kernel-team@meta.com \
--cc=lance.yang@linux.dev \
--cc=liam@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=npache@redhat.com \
--cc=nphamcs@gmail.com \
--cc=riel@surriel.com \
--cc=ryan.roberts@arm.com \
--cc=shakeel.butt@linux.dev \
--cc=shikemeng@huaweicloud.com \
--cc=usama.arif@linux.dev \
--cc=vbabka@kernel.org \
--cc=willy@infradead.org \
--cc=ying.huang@linux.alibaba.com \
--cc=youngjun.park@lge.com \
--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