Integrated: 8315795: runtime/Safepoint/TestAbortVMOnSafepointTimeout.java fails after JDK-8305507

[Patricio Chilano Mateo]

On Thu, 7 Sep 2023 16:02:55 GMT, Patricio Chilano Mateo <pchilanomate at openjdk.org> wrote:

> Please review this small fix to test TestAbortVMOnSafepointTimeout.java. With the current value of AbortVMOnSafepointTimeoutDelay we get cases where the spawned child VM exits with a safepoint timeout but without the main thread being able to print the PRE_STALL_TEXT text. I added a more detailed analysis in the bug comments.
> To fix it I increased the value of AbortVMOnSafepointTimeoutDelay to be about twice as much as the longest time it could take to print the PRE_STALL_TEXT based on the measurements I have done in the failing Windows machines.
> Without the fix I can reproduce the issue in about 10% of the runs running tier1_runtime. With the fix I run ~400 runs of tier1_runtime without failures.
> 
> Thanks,
> Patricio

This pull request has now been integrated.

Changeset: 4c6d7fc2
Author:    Patricio Chilano Mateo <pchilanomate at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/4c6d7fc2584ce969649d06a366fb4b8688a3aaa9
Stats:     2 lines in 1 file changed: 0 ins; 0 del; 2 mod

8315795: runtime/Safepoint/TestAbortVMOnSafepointTimeout.java fails after JDK-8305507

Reviewed-by: dcubed

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

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