Skip to content

mm: thp: always enable mTHP support

Luiz Capitulino linux-mm linuxppc-dev lkml 8 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v8 Latest14 patches 37 repliesIn review
  2. v714 patches 4 repliesSuperseded
  3. v614 patches 5 repliesSuperseded
Earlier versions · 5
VersionPostedPatchesRepliesStatus
v514 patches16 repliesSuperseded
v49 patches26 repliesSuperseded
v310 patches38 repliesSuperseded
v211 patches9 repliesSuperseded
v111 patches5 repliesSuperseded

Patches in v8 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/14docs: tmpfs: remove implementation detail referenceDocumentation/filesystems/tmpfs.rst+2 -30 replies
2/14mm: shmem: shmem_getattr(): set blksize to highest supported THP ordermm/shmem.c+4 -21 reply
3/14mm: introduce pgtable_has_pmd_leaves()include/linux/pgtable.h mm/memory.c mm/mm_init.c+27 -02 replies
4/14drivers: dax: use pgtable_has_pmd_leaves()drivers/dax/dax-private.h+1 -13 replies
5/14drivers: nvdimm: use pgtable_has_pmd_leaves()drivers/nvdimm/pfn_devs.c+4 -21 reply
6/14mm: debug_vm_pgtable: use pgtable_has_pmd_leaves()mm/debug_vm_pgtable.c+10 -102 replies
7/14mm: shmem: allow THP support determination at folio allocation timemm/shmem.c+5 -46 replies
8/14s390: move has_transparent_hugepage() out of THP guardarch/s390/include/asm/pgtable.h+1 -12 replies
9/14powerpc: move has_transparent_hugepage() out of THP guardarch/powerpc/include/asm/book3s/64/hash-4k.h arch/powerpc/include/asm/book3s/64/hash-64k.h arch/powerpc/include/asm/book3s/64/pgtable.h +2 more+20 -205 replies
10/14mips: move has_transparent_hugepage() out of THP guardarch/mips/include/asm/pgtable.h arch/mips/mm/pgtable.c arch/mips/mm/tlb-r4k.c+28 -251 reply
11/14x86: move has_transparent_hugepage() out of THP guardarch/x86/include/asm/pgtable.h+6 -60 replies
12/14treewide: introduce arch_has_pmd_leaves()arch/mips/include/asm/pgtable.h arch/mips/mm/pgtable.c arch/powerpc/include/asm/book3s/64/hash-4k.h +9 more+23 -231 reply
13/14mm: replace thp_disabled_by_hw() with pgtable_has_pmd_leaves()include/linux/huge_mm.h mm/huge_memory.c mm/memory.c +1 more+4 -131 reply
14/14mm: thp: always enable mTHP supportmm/huge_memory.c mm/shmem.c+29 -107 replies

Reviews trailers per patch

11 of 14 patches have a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchDavid Hildenbrand (Red Hat)Baolin WangLance YangZi YanDev JainUsama Arif
SeriesA-b
1/14A-b R-b R-b R-b R-b
2/14A-b A-b
3/14A-b A-b
4/14A-b A-b
5/14A-b A-b
6/14A-b R-b R-b
7/14R-b R-b
8/14A-b A-b
9/14
10/14
11/14
12/14A-b
13/14A-b R-b R-b A-b
14/14R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 6 reviewers.

Read the v8 thread

Cover letter [PATCH v8 00/14] mm: thp: always enable mTHP support

Introduction
============

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.

This happens because the has_transparent_hugepage() helper is overloaded:
its name implies it checks whether THP is enabled, but on some architectures it actually checks whether the CPU supports PMD-sized pages. In addition, the THP and shmem code have a big switch on has_transparent_hugepage().

This series solves this by decoupling THP availability checking from querying CPU support for PMD-sized pages. THP availability can be checked with IS_ENABLED(CONFIG_TRANSPARENT_HUGEPAGE). For querying PMD-sized page support, this series introduces a new helper called pgtable_has_pmd_leaves(), which is independent of THP, has well defined semantics and can be used in fast paths.

Core THP code and shmem can use pgtable_has_pmd_leaves() to determine PMD-sized THP support at page fault time (or folio allocation time for shmem), leaving THP and shmem always enabled for other THP sizes. For architectures and CPUs that do support PMD-sized pages there's no intended change in behavior.

We also convert each user of has_transparent_hugepage(), and the related helper thp_disabled_by_hw(), to IS_ENABLED(CONFIG_TRANSPARENT_HUGEPAGE) and/or pgtable_has_pmd_leaves() according to the call site's check semantics. On s390, powerpc, mips and x86 the arch has_transparent_hugepage() implementation is moved out of the CONFIG_TRANSPARENT_HUGEPAGE guard and renamed to arch_has_pmd_leaves().

Thanks to David Hildenbrand for suggesting this improvement and for providing initial guidance (all bugs and misconceptions are mine).

This applies to mm-new 93f615a22169 ("mm/swap, PM: hibernate: atomically replace hibernation pin").

Reporting availability of PMD-sized pages to user-space
=======================================================

Before this series, not having PMD pages available would shut down THP support meaning that /sys/kernel/mm/transparent_hugepage would not be available. After this series, /sys/kernel/mm/transparent_hugepage and hpage_pmd_size are always available but hugepages-<PMD-size>kB is only available if PMD-sized pages are supported.

To me this seems logical, as hugepages-<size>kB is only available if <size> is supported. If this is correct, then MM kselftests that depend on PMD-sized pages will need to be updated to check for it as they fail with this series applied when PMD-sized pages are not available.

The alternative is changing hpage_pmd_size: either don't create the file when PMD-sized pages are not available or set hpage_pmd_size=0. My concern is that this could be considered an ABI breakage as this file was never reported to be optional or report a zero value.

Testing
=======

- Ran MM kselftests on x86_64 and s390
- Tested all mTHP sizes allocation on x86_64 (with and without PMD-sized pages available)
- Tested shmem with within_size w/ mTHP on x86_64 (with and without PMD-sized pages available)
- Performed defconfig build on x86_64, s390, arm64, powerpc, and mips

NOTES:
* I'm forcing arch_has_pmd_leaves() off on x86 to simulate not having PMD-sized pages available

* Running the MM kselftests when PMD pages are not available causes some tests that depend on PMD-sized THP pages to fail as noted earlier

Changelog
=========

v8
--
- Fixed build breakage on MIPS (Lance)
- Rebased on top of mm-new (fixed conflicts in include/linux/pgtable.h and mm/shmem.c - dropped a Reviewed-by as a result)

v7
--
- Applied on top of latest mm-new
- Improved various changelogs including cover-letter
- Changed arch_has_pmd_leaves() default implementation to use IS_ENABLED() instead of IS_BUILTIN()

v6
--
- Rebased on top of mm-unstable (required a minor conflict resolution)
- Added more Reviewed-by and Acked-by tags

v5
--
- Moved init_arch_has_pmd_leaves() to mm_core_init() (David)
- Renamed init_arch_has_pmd_leaves() to pgtable_leaf_support_init() (Lance)
- Added new patch changing shmem_getattr() to set blksize according to highest supported THP order (Baolin)
- Added new patches to move has_transparent_hugepage() out of CONFIG_TRANSPARENT_HUGEPAGE guards
- shmem_allowable_huge_orders(): rename variable and only disable PMD_ORDER (David)
- Added <linux/jmp_label.h> to include/linux/pgtable.h
- Added new tags and removed Reviewed-by from changed patches

v4
--
- Used 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_has_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 (14):
docs: tmpfs: remove implementation detail reference
mm: shmem: shmem_getattr(): set blksize to highest supported THP order
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: allow THP support determination at folio allocation time
s390: move has_transparent_hugepage() out of THP guard
powerpc: move has_transparent_hugepage() out of THP guard
mips: move has_transparent_hugepage() out of THP guard
x86: move has_transparent_hugepage() out of THP guard
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               |  6 ++--
 arch/mips/mm/pgtable.c                        | 25 ++++++++++++++++
 arch/mips/mm/tlb-r4k.c                        | 22 --------------
 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  | 18 ++++++------
 arch/powerpc/include/asm/book3s/64/radix.h    | 14 ++++-----
 arch/powerpc/mm/book3s64/hash_pgtable.c       |  8 ++---
 arch/s390/include/asm/pgtable.h               |  6 ++--
 arch/x86/include/asm/pgtable.h                | 12 ++++----
 drivers/dax/dax-private.h                     |  2 +-
 drivers/nvdimm/pfn_devs.c                     |  6 ++--
 include/linux/huge_mm.h                       |  7 -----
 include/linux/pgtable.h                       | 21 ++++++++++++--
 mm/debug_vm_pgtable.c                         | 20 ++++++-------
 mm/huge_memory.c                              | 27 ++++++++++++-----
 mm/memory.c                                   | 11 ++++++-
 mm/mm_init.c                                  |  1 +
 mm/shmem.c                                    | 29 ++++++++++++-------
 20 files changed, 144 insertions(+), 100 deletions(-)
Signature · 1 line
2.55.0