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