On Fri, 3 May 2024 06:40:07 GMT, Alan Bateman <[email protected]> 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