On Thu, 28 Sep 2023 21:04:58 GMT, Leonid Mesnik <lmes...@openjdk.org> wrote:
>> The test fails because ThreadDeath is raised during class >> jdk.internal.misc.VirtualThreads initialization. The proposed fix is to >> pre-initialize this step to avoid such failures. See more details in the bug. >> I reproduced the original problem and verified that it is not reproduced >> after fix. >> Tested with tier5 and running nsk/jvmti tests with and without virtual test >> thread factory. >> >> I don't think that more complex fix is needed. There is a plan to review >> nsk/jvmti stopThread tests and see if >> ./serviceability/jvmti/vthread/StopThreadTest/StopThreadTest.java >> might be improved to cover them. > > Leonid Mesnik has updated the pull request incrementally with one additional > commit since the last revision: > > missed import added. The work around looks okay to me. I've notified Alan about this PR. Thanks, Serguei Marked as reviewed by sspitsyn (Reviewer). ------------- Marked as reviewed by sspitsyn (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/15966#pullrequestreview-1651636390 PR Review: https://git.openjdk.org/jdk/pull/15966#pullrequestreview-1651636434
