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

[Coleen Phillimore]

On Fri, 29 Apr 2022 17:35:57 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

> 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.

This pull request has now been integrated.

Changeset: be67acdf
Author:    Coleen Phillimore <coleenp at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/be67acdf5cb962deb657045c4c07592984a789c9
Stats:     304 lines in 21 files changed: 134 ins; 125 del; 45 mod

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

Reviewed-by: dholmes, fparain, iklam

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

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