RFR: JDK-8310233: Linux: THP initialization is incorrect [v2]

[Thomas Stuefe]

> Today, if we use UseTransparentHugePages, we assume that the static hugepage detection we do is valid for THPs:
> - that THPs use the page size (in hotspot used as "default large page size") found in /proc/memlimit Hugepagesize)
> - that THPs are enabled if that page size is >0.
> 
> Both assumptions are incorrect. THPs are enabled depending on the mode in `/sys/kernel/mm/transparent_hugepage/enabled` (tri-state). And the pagesize used by khugepaged is the one set in `/sys/kernel/mm/transparent_hugepage/hpage_pmd_size`. The latter can differ from the default large page size on the system (e.g. static hugepage default size could be 1g, whereas THP hugepage size is 2m).
> 
> ------
> 
> About the patch:
> 
> This is a limited, minimally invasive patch to fix THP detection. The patch aims to be easy to downport. There is more work to do, which I will do in subsequent RFEs.
> 
> Functionally, for *static* (non-THP) pages nothing changes. THP-mode now correctly detects THP support in the OS, and uses the correct page size (see examples below).
> 
> -------------
> 
> Example 1: System has THPs disabled, but static hugepages (1g, 2m) configured:
> 
> 
> thomas at starfish $ cat /sys/kernel/mm/transparent_hugepage/enabled
> always madvise [never]
> thomas at starfish $ cat /proc/meminfo | grep Hugepage
> Hugepagesize:    1048576 kB
> 
> 
> Without patch, we incorrectly assume THPs are enabled, and that THP page size is 1G (!), which we then proceed and use as heap page size, causing the heap size to be rounded up from 512m -> 1G. But - even though it is printed as "1G page backed" in log output - in reality it will still 4K-page-backed: the `madvise(2)` we use to set the THP page size will be ignored by the system, since THPs are disabled.
> 
> 
> thomas at starfish $ ./images/jdk/bin/java -Xmx512m -XX:+UseLargePages -XX:+UseTransparentHugePages -Xlog:pagesize -version
> [0.001s][info][pagesize] Using the default large page size: 1G
> [0.001s][info][pagesize] Usable page sizes: 4k, 2M, 1G
> ...
> [0.016s][info][pagesize] Heap:  min=1G max=1G base=0x00000000c0000000 size=1G page_size=1G
> 
> 
> With patch, we correctly refuse to use large pages (and we log more info):
> 
> 
> thomas at starfish $ ./images/jdk/bin/java -Xmx512m -XX:+UseLargePages -XX:+UseTransparentHugePages -Xlog:pagesize -version
> [0.001s][info][pagesize] Static hugepage support: 2M, 1G (default)
> [0.001s][info][pagesize]   default pagesize: 1G
> [0.001s][info][pagesize] Transparent hugepage (THP) support:
> [0.001s][info][pagesize]   mode: never
> [0.001s][warning][pagesize] UseLargePages ...

Thomas Stuefe has updated the pull request incrementally with two additional commits since the last revision:

 - Merge branch 'JDK-8310233-Linux-THP-initialization-incorrect' of github.com:tstuefe/jdk into JDK-8310233-Linux-THP-initialization-incorrect
 - Add test case and modify log output slightly

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/14739/files
  - new: https://git.openjdk.org/jdk/pull/14739/files/1e4dfcef..a65c346b

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=14739&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=14739&range=00-01

  Stats: 343 lines in 4 files changed: 317 ins; 19 del; 7 mod
  Patch: https://git.openjdk.org/jdk/pull/14739.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/14739/head:pull/14739

PR: https://git.openjdk.org/jdk/pull/14739