The Linux Kernel Mailing List
 help / color / mirror / Atom feed
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

  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