RFR: 8247972: incorrect implementation of JVM TI GetObjectMonitorUsage [v21]

Serguei Spitsyn sspitsyn at openjdk.org
Mon Mar 4 11:01:46 UTC 2024


On Mon, 4 Mar 2024 03:28:28 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Serguei Spitsyn has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   rename after merge: jvmti_common.h to jvmti_common.hpp
>
> test/hotspot/jtreg/serviceability/jvmti/ObjectMonitorUsage/ObjectMonitorUsage.java line 40:
> 
>> 38:  *       - owned object with N waitings to be notified
>> 39:  *       - owned object with N waitings to enter, from 0 to N waitings to re-enter,
>> 40:  *         from N to 0 waitings to be notified
> 
> "waitings" is not grammatically valid. Suggestion: s/waitings/threads waiting/

Thanks, fixed now.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17680#discussion_r1510976777


More information about the hotspot-dev mailing list