RFR: JDK-8313782: Add user-facing warning if THPs are enabled but cannot be used [v2]
Thomas Stuefe
stuefe at openjdk.org
Mon Aug 7 09:41:20 UTC 2023
> We don't warn the user if `+UseTransparentHugePages ` were specified (and nothing else, but this implies `+UseLargePages`), but the system does not support THPs.
>
> We used to have warnings in the code, but I checked and they never really worked (I tried older versions of the JDK). The reason is that THP detection had been broken. The broken detection always reported success even if THPs are disabled, so the user would never have seen this warning even when we had it.
>
> THP detection has since been fixed with [JDK-8310233](https://bugs.openjdk.org/browse/JDK-8310233), but we still don't warn.
>
> The patch re-introduces the warning, with the same text as it used to have, and using the warning level of UL with the "pagesize" tag - that's what we do for static hugepage warnings too.
>
> Before:
>
> thomas at starfish$ ./images/jdk/bin/java -XX:+UseTransparentHugePages -version
> openjdk version "20-ea" 2023-03-21
> OpenJDK Runtime Environment SapMachine (build 20-ea+32)
> OpenJDK 64-Bit Server VM SapMachine (build 20-ea+32, mixed mode, sharing)
>
>
> Now:
>
> thomas at starfish$ ./images/jdk/bin/java -XX:+UseTransparentHugePages -version
> [0.001s][warning][pagesize] TransparentHugePages is not supported by the operating system.
> openjdk version "22-internal" 2024-03-19
> OpenJDK Runtime Environment (fastdebug build 22-internal-adhoc.thomas.source)
> OpenJDK 64-Bit Server VM (fastdebug build 22-internal-adhoc.thomas.source, mixed mode, sharing)
Thomas Stuefe has updated the pull request incrementally with one additional commit since the last revision:
Davids feedback
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/15158/files
- new: https://git.openjdk.org/jdk/pull/15158/files/c4e4ded6..763a0167
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=15158&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=15158&range=00-01
Stats: 3 lines in 1 file changed: 0 ins; 0 del; 3 mod
Patch: https://git.openjdk.org/jdk/pull/15158.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/15158/head:pull/15158
PR: https://git.openjdk.org/jdk/pull/15158
More information about the hotspot-runtime-dev
mailing list