RFR: 8285832: runtime/Thread/TooSmallStackSize.java failed "assert(k->is_initialized()) failed: need to increase java_thread_min_stack_allowed calculation"

Coleen Phillimore coleenp at openjdk.java.net
Fri Apr 29 22:43:58 UTC 2022


The minimum stack size check for ThreadStackSize, CompilerStackSize and VMStackSize should also be done for windows. I moved this to shared code with platform specific constants.
Windows platform was checking the option against the current thread's stack size, and not checking ThreadStackSize, and was setting the stack size against that.  It worked until you increase StackShadowPages, and then the calculation is wrong.
For Windows, the minimum thread stack size with current StackShadowPages of 9 in debug is now 92K rather than 188K.
The ShadowPageSize is too low for the loom repository and is on the bounds of current mainline, so will be increased when Loom is integrated.  When increased to 11 StackShadowPages, the smallest stack size is 100K.
Tested with tier1-4 on windows-x64-debug.

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

Commit messages:
 - 8285832: runtime/Thread/TooSmallStackSize.java failed "assert(k->is_initialized()) failed: need to increase java_thread_min_stack_allowed calculation"
 - 8285832: runtime/Thread/TooSmallStackSize.java failed "assert(k->is_initialized()) failed: need to increase java_thread_min_stack_allowed calculation"

Changes: https://git.openjdk.java.net/jdk/pull/8471/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=8471&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8285832
  Stats: 292 lines in 21 files changed: 123 ins; 119 del; 50 mod
  Patch: https://git.openjdk.java.net/jdk/pull/8471.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/8471/head:pull/8471

PR: https://git.openjdk.java.net/jdk/pull/8471


More information about the hotspot-runtime-dev mailing list