RFR: dcubed - misc cleanups of jvm.cpp and JVM/TI files.
Serguei Spitsyn
sspitsyn at openjdk.java.net
Fri Apr 8 01:51:18 UTC 2022
On Thu, 7 Apr 2022 18:32:56 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:
> Misc cleanups of jvm.cpp and JVM/TI files.
src/hotspot/share/prims/jvmtiThreadState.cpp line 124:
> 122: // Set this as the state for the JavaThread or mounted virtual thread
> 123: // only if thread_oop is current thread->jvmti_vthread().
> 124: thread->set_jvmti_thread_state(this);
This update has the same problem as previous one that I've commented. We set the thread->jvmti_thread_state() value in the JavaThread when it is bound to a normal/carrier or mounted virtual thread. There is no JavaThread only for unmounted virtual thread.
src/hotspot/share/prims/jvmtiThreadState.cpp line 217:
> 215: //
> 216:
> 217: // VTMT cannot be disabled while this counter is positive
A dot at the end is also needed to follow your comment style.
-------------
PR: https://git.openjdk.java.net/loom/pull/140
More information about the loom-dev
mailing list