RFR: 8256314: JVM TI GetCurrentContendedMonitor is implemented incorrectly [v4]

Serguei Spitsyn sspitsyn at openjdk.org
Tue Mar 5 07:43:46 UTC 2024


On Mon, 4 Mar 2024 09:51:32 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:

>> src/hotspot/share/prims/jvmtiEnvBase.cpp line 941:
>> 
>>> 939:     bool is_virtual = java_lang_VirtualThread::is_instance(thread_oop);
>>> 940:     jint state = is_virtual ? JvmtiEnvBase::get_vthread_state(thread_oop, java_thread)
>>> 941:                             : JvmtiEnvBase::get_thread_state(thread_oop, java_thread);
>> 
>> I've seen this in a couple of your PRs now. It would be nice to have a utility function to get the JVMTI thread state of any java.lang.Thread instance.
>
> Good suggestion, thanks. Will add.

Added now.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/17944#discussion_r1512266627


More information about the hotspot-dev mailing list