Integrated: 8307968: serviceability/jvmti/vthread/StopThreadTest/StopThreadTest.java timed out

[Serguei Spitsyn]

On Sat, 13 May 2023 07:28:38 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:

> This is newly integrated test times out because it has a race in in the Test #A.1 and #A.2.
> The main root cause is a print statement which can case target virtual thread to unpark and unmount.
> This causes that the `StopThreads` unexpectedly fails with the `JVMTI_ERROR_OPAQUE_FRAME` error code.
> The target thread can be in some other unexpected states if JVMTI `StopThread`
> is called before the target thread method `A()` reached the synchronized statement.
> 
> The fix is to replace the `ensureStarted()` with the `ensureAtPointA()`.
> The fix also includes some simplifications related to clearing the target thread interrupt status.
> 
> Testing:
> Hundreds of mach5 runs of `serviceability/jvmti/vthread` tests which include the fixed `StopThreadTest`.
> TBD: To run mach5 tiers1-3.
> 
> The test does not fail with this fix anymore.

This pull request has now been integrated.

Changeset: c2ef3024
Author:    Serguei Spitsyn <sspitsyn at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/c2ef3024689d60c79320ab3ef9a2fdeccb4cde72
Stats:     22 lines in 1 file changed: 4 ins; 10 del; 8 mod

8307968: serviceability/jvmti/vthread/StopThreadTest/StopThreadTest.java timed out

Reviewed-by: cjplummer, lmesnik

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

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