From: Luiz Capitulino <luizcap@redhat.com>
To: Lorenzo Stoakes <ljs@kernel.org>
Cc: linux-kernel@vger.kernel.org, linux-mm@kvack.org,
david@kernel.org, baolin.wang@linux.alibaba.com, ziy@nvidia.com,
lance.yang@linux.dev, corbet@lwn.net, tsbogend@alpha.franken.de,
maddy@linux.ibm.com, mpe@ellerman.id.au, agordeev@linux.ibm.com,
gerald.schaefer@linux.ibm.com, hca@linux.ibm.com,
gor@linux.ibm.com, x86@kernel.org, dave.hansen@linux.intel.com,
djbw@kernel.org, vishal.l.verma@intel.com, dave.jiang@intel.com,
akpm@linux-foundation.org
Subject: Re: [PATCH v4 0/9] mm: thp: always enable mTHP support
Date: Thu, 14 May 2026 08:14:12 -0400 [thread overview]
Message-ID: <ce7ff1f1-cdf7-4219-8422-0e52e65ed818@redhat.com> (raw)
In-Reply-To: <agVs9bWkkx6uJdfR@lucifer>
On 2026-05-14 02:35, Lorenzo Stoakes wrote:
> -cc my previous kernel mail
>
> (Minor annoying reminder to update my email to ljs@kernel.org :P)
Nah, my fault. I promise I updated your email now.
>
> On Fri, May 01, 2026 at 03:18:42PM -0400, Luiz Capitulino wrote:
>> Today, if an architecture implements has_transparent_hugepage() and the CPU
>> lacks support for PMD-sized pages, the THP code disables all THP, including
>> mTHP. In addition, the kernel lacks a well defined API to check for
>> PMD-sized page support. It currently relies on has_transparent_hugepage()
>> and thp_disabled_by_hw(), but they are not well defined and are tied to
>> THP support.
>>
>> This series addresses both issues by introducing a new well defined API
>> to query PMD-sized page support: pgtable_has_pmd_leaves(). Using this
>> new helper, we ensure that mTHP remains enabled even when the
>> architecture or CPU doesn't support PMD-sized pages.
>>
>> Thanks to David Hildenbrand for suggesting this improvement and for
>> providing guidance (all bugs and misconceptions are mine).
>>
>> This applies to Linus tree 08d0d3466664 ("Merge tag 'net-7.1-rc2'
>> of git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net")
>>
>> NOTE: I used Claude Code Opus 4.6 to *review* the series before
>> posting. It did find one issue where a pgtable_has_pmd_leaves()
>> check was missing when assining huge_shmem_orders_inherit in
>> shmem_init().
>>
>> v4
>> --
>> - Use static key for pgtable_has_pmd_leaves() API (Lance)
>> - Moved shmem pgtable_has_pmd_leaves() check to
>> shmem_allowable_huge_orders() (Baolin)
>> - Default pgtable_has_pmd_leaves() implementation to
>> IS_ENABLED(CONFIG_HAVE_ARCH_TRANSPARENT_HUGEPAGE) (Zi)
>> - Dropped patch “mm: thp: x86: cleanup PSE feature bit usage” (Dave)
>>
>> v3
>> --
>> - Rebased on top of latest Linus tree
>> - Removed i915 patch as driver dropped has_transparent_hugepage() usage
>> - Moved init_arch_has_pmd_leaves() call in start_kernel() to avoid conflict
>> with early_param handlers clearing CPU feature flags
>> - Fixed build error with CONFIG_MMU=n (kernel test robot)
>> - Fixed huge_anon_orders_inherit default setting when !pgtable_pmd_leaves() (Baolin)
>> - Small commit changelog improvements
>>
>> v2
>> --
>> - Added support for always enabling mTHPs for shmem (Baolin)
>> - Improved commits changelog & added reviewed-by
>>
>> v1
>> --
>> - Call init_arch_has_pmd_leaves() from start_kernel()
>> - Keep pgtable_has_pmd_leaves() calls tied to CONFIG_TRANSPARENT_HUGEPAGE (David)
>> - Clear PUD_ORDER when clearing PMD_ORDER (David)
>> - Small changelog improvements (David)
>> - Rebased on top of latest mm-new
>>
>> Luiz Capitulino (9):
>> docs: tmpfs: remove implementation detail reference
>> mm: introduce pgtable_has_pmd_leaves()
>> drivers: dax: use pgtable_has_pmd_leaves()
>> drivers: nvdimm: use pgtable_has_pmd_leaves()
>> mm: debug_vm_pgtable: use pgtable_has_pmd_leaves()
>> mm: shmem: drop has_transparent_hugepage() usage
>> treewide: introduce arch_has_pmd_leaves()
>> mm: replace thp_disabled_by_hw() with pgtable_has_pmd_leaves()
>> mm: thp: always enable mTHP support
>>
>> Documentation/filesystems/tmpfs.rst | 5 ++--
>> arch/mips/include/asm/pgtable.h | 4 +--
>> arch/mips/mm/tlb-r4k.c | 4 +--
>> arch/powerpc/include/asm/book3s/64/hash-4k.h | 2 +-
>> arch/powerpc/include/asm/book3s/64/hash-64k.h | 2 +-
>> arch/powerpc/include/asm/book3s/64/pgtable.h | 10 ++++----
>> arch/powerpc/include/asm/book3s/64/radix.h | 2 +-
>> arch/powerpc/mm/book3s64/hash_pgtable.c | 4 +--
>> arch/s390/include/asm/pgtable.h | 4 +--
>> arch/x86/include/asm/pgtable.h | 4 +--
>> drivers/dax/dax-private.h | 2 +-
>> drivers/nvdimm/pfn_devs.c | 6 +++--
>> include/linux/huge_mm.h | 7 ------
>> include/linux/pgtable.h | 19 ++++++++++++--
>> init/main.c | 1 +
>> mm/debug_vm_pgtable.c | 20 +++++++--------
>> mm/huge_memory.c | 25 +++++++++++++------
>> mm/memory.c | 11 +++++++-
>> mm/shmem.c | 21 +++++++++-------
>> 19 files changed, 93 insertions(+), 60 deletions(-)
>>
>> --
>> 2.53.0
>>
>
prev parent reply other threads:[~2026-05-14 12:14 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <cover.1777663129.git.luizcap@redhat.com>
[not found] ` <20260503080236.4ea7f3fec0e5788d50113599@linux-foundation.org>
2026-05-04 19:11 ` [PATCH v4 0/9] mm: thp: always enable mTHP support Luiz Capitulino
[not found] ` <2a0bae00cdd2b6ef6b962610b523ebfc97806ba7.1777663129.git.luizcap@redhat.com>
2026-05-06 17:50 ` (sashiko review) Re: [PATCH v4 2/9] mm: introduce pgtable_has_pmd_leaves() Luiz Capitulino
2026-05-13 15:30 ` David Hildenbrand (Arm)
2026-05-14 1:52 ` Luiz Capitulino
2026-05-13 15:36 ` David Hildenbrand (Arm)
2026-05-14 2:18 ` Luiz Capitulino
[not found] ` <d2d1d0deaa5732d82bc2daea5033a1578781d641.1777663129.git.luizcap@redhat.com>
2026-05-06 18:12 ` (sashiko review) Re: [PATCH v4 6/9] mm: shmem: drop has_transparent_hugepage() usage Luiz Capitulino
[not found] ` <85674af7934cd363ceadd1d4f2b784d833a183cc.1777663129.git.luizcap@redhat.com>
2026-05-06 18:22 ` (sashiko review) Re: [PATCH v4 7/9] treewide: introduce arch_has_pmd_leaves() Luiz Capitulino
2026-05-06 18:30 ` Luiz Capitulino
[not found] ` <cb854ba26a4b8036706c6da5f6844f10f386087f.1777663129.git.luizcap@redhat.com>
2026-05-13 15:40 ` [PATCH v4 3/9] drivers: dax: use pgtable_has_pmd_leaves() David Hildenbrand (Arm)
2026-05-14 2:25 ` Luiz Capitulino
[not found] ` <d4fbc41a51f8fe9ab55cc6792d60c6c2c504b4d4.1777663129.git.luizcap@redhat.com>
2026-05-13 15:50 ` [PATCH v4 8/9] mm: replace thp_disabled_by_hw() with pgtable_has_pmd_leaves() David Hildenbrand (Arm)
[not found] ` <f67da00a825da9097b5faf2f390ad344450b88be.1777663129.git.luizcap@redhat.com>
2026-05-06 5:46 ` [PATCH v4 9/9] mm: thp: always enable mTHP support Baolin Wang
2026-05-06 18:34 ` (sashiko review) " Luiz Capitulino
2026-05-13 15:58 ` David Hildenbrand (Arm)
2026-05-14 1:14 ` Baolin Wang
2026-05-14 6:35 ` [PATCH v4 0/9] " Lorenzo Stoakes
2026-05-14 12:14 ` Luiz Capitulino [this message]
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=ce7ff1f1-cdf7-4219-8422-0e52e65ed818@redhat.com \
--to=luizcap@redhat.com \
--cc=agordeev@linux.ibm.com \
--cc=akpm@linux-foundation.org \
--cc=baolin.wang@linux.alibaba.com \
--cc=corbet@lwn.net \
--cc=dave.hansen@linux.intel.com \
--cc=dave.jiang@intel.com \
--cc=david@kernel.org \
--cc=djbw@kernel.org \
--cc=gerald.schaefer@linux.ibm.com \
--cc=gor@linux.ibm.com \
--cc=hca@linux.ibm.com \
--cc=lance.yang@linux.dev \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=ljs@kernel.org \
--cc=maddy@linux.ibm.com \
--cc=mpe@ellerman.id.au \
--cc=tsbogend@alpha.franken.de \
--cc=vishal.l.verma@intel.com \
--cc=x86@kernel.org \
--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