RFR: 8371948: TestStackOverflowDuringInit.java fails xss too small on linux-aarch64 [v5]

Andrew Haley aph at openjdk.org
Thu Dec 18 09:14:00 UTC 2025


On Thu, 11 Dec 2025 01:57:58 GMT, SendaoYan <syan at openjdk.org> wrote:

>> Hi all,
>> 
>> On linux-aarch64, when page size is 64K, the minimal java thread stack size is 448K. So this PR increase the minimal java thread stack size in test/hotspot/jtreg/runtime/ClassInitErrors/TestStackOverflowDuringInit.java. Below is the calculation formula for the minimal java thread stack size:
>> 
>> 
>> page_size = 64K
>> _java_thread_min_stack_allowed = 72K
>> _stack_red_zone_size = align_up(StackRedPages * unit, page_size) = align_up(1*4K, 64K) = 64K
>> _stack_yellow_zone_size = align_up(StackYellowPages * unit, page_size) = align_up(2*4K, 64K) = 64K
>> _stack_reserved_zone_size = align_up(StackReservedPages * unit, page_size) = align_up(1*4K, 64K) = 64K
>> _stack_shadow_zone_size = align_up(StackShadowPages * unit, page_size) = align_up(25*4K, 64K) = 128K
>> _java_thread_min_stack_allowed = align_up(72K+64K+64K+64K+128K, 64K) = 448K
>> 
>> 
>> This PR add whitebox API getMinimumJavaStackSize() to the the allowd minimum java stack size, and get the allowed minimum java stack size in test/hotspot/jtreg/runtime/ClassInitErrors/TestStackOverflowDuringInit.java
>> 
>> Change has been verified locally both on linux-x64 and linux-aarch64.
>
> SendaoYan has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - remove test/hotspot/jtreg/runtime/memory/ReadMinimumJavaStackSize.java
>  - Use get_minimum_java_stack_size instead of get_minimum_java_stack_sizes

Marked as reviewed by aph (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/28352#pullrequestreview-3591597450


More information about the hotspot-dev mailing list