RFR: 8328083: degrade virtual thread support for GetObjectMonitorUsage [v2]

Serguei Spitsyn sspitsyn at openjdk.org
Fri May 3 10:33:53 UTC 2024


On Fri, 3 May 2024 06:40:07 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> src/hotspot/share/prims/jvmti.xml line 8280:
>> 
>>> 8278:           <description>
>>> 8279:             The number of platform threads waiting to own this monitor, or <code>0</code>
>>> 8280:             if only virtual threads are waiting or no threads are waiting
>> 
>> This is now exactly the same as `waiter_count` above. I don't think this is what you intended.
>
> Indeed, looks like the description for waiter_count has been pasted in here in error.

Thank you.
Fixed as below:

diff --git a/src/hotspot/share/prims/jvmti.xml b/src/hotspot/share/prims/jvmti.xml
index d382a02178e..3bcf15466d7 100644
--- a/src/hotspot/share/prims/jvmti.xml
+++ b/src/hotspot/share/prims/jvmti.xml
@@ -8277,7 +8277,8 @@ class C2 extends C1 implements I2 {
           <jint/>
           <description>
             The number of platform threads waiting to own this monitor, or <code>0</code>
-            if only virtual threads are waiting or no threads are waiting
+            if only virtual threads are waiting to be notified or no threads are waiting
+            to be notified
           </description>
         </field>
         <field id="notify_waiters">

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19030#discussion_r1589031880


More information about the serviceability-dev mailing list