On Tue, 18 Jul 2023 00:48:30 GMT, Alex Menkov <amen...@openjdk.org> wrote:
>> The change fixes handling of "suspended" bit in VT state. >> The code looks very strange. >> java_lang_VirtualThread::RUNNING == 3, so line 803 clears >> JVMTI_THREAD_STATE_ALIVE(1) and JVMTI_THREAD_STATE_TERMINATED(2) >> Per log this code came from loom repo with VT integration. >> >> Testing: tier1-4, updated GetThreadStateMountedTest.java > > Alex Menkov has updated the pull request incrementally with one additional > commit since the last revision: > > check that VT is alive before applying suspend/interrupt bits Seems fine to me, assuming all tests pass. Thanks. ------------- Marked as reviewed by dholmes (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/14878#pullrequestreview-1533917232
