RFR: 8334085: Test crash: assert(thread->held_monitor_count() == 0) failed: Must be [v3]

Serguei Spitsyn sspitsyn at openjdk.org
Wed Jul 24 21:50:01 UTC 2024


On Wed, 24 Jul 2024 21:38:28 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:

>> test/hotspot/jtreg/serviceability/jvmti/GetOwnedMonitorInfo/GetOwnedMonitorInfoTest.java line 56:
>> 
>>> 54: 
>>> 55:     private static void jniMonitorEnterAndLetObjectDie() {
>>> 56:         // The monitor iterator used by GetOwnedMonitorInfo to
>> 
>> The original was correct "used to assert"
>
> The original was: "used by GetOwnedMonitorInfo used to assert".
> The word "used" is repeated twice.
> Changed to: "used by GetOwnedMonitorInfo to assert".
> Do I miss anything?

Okay, thanks.  It is a little bit confusing. Restored the original comment.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20294#discussion_r1690493967


More information about the serviceability-dev mailing list