On Sat, 13 May 2023 07:28:38 GMT, Serguei Spitsyn <sspit...@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 <sspit...@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
