On Fri, 3 May 2024 06:40:07 GMT, Alan Bateman <al...@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

Reply via email to