On Thu, 30 Jul 2026 18:49:09 GMT, Patricio Chilano Mateo
<[email protected]> wrote:
>> Coleen Phillimore has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Update and clean up the test.
>
> src/hotspot/share/services/threadService.cpp line 1028:
>
>> 1026: st->print_cr("%s \"%s\"", owner_desc, currentThread->name());
>> 1027: } else {
>> 1028: st->print_cr(",\n which has now been released");
>
> Preexisting: Looks like this print statement is reversed with the `non-Java
> thread=` one below.
Yes, the print statements are reversed. good catch!
> test/hotspot/jtreg/serviceability/dcmd/thread/PrintRawMonitorLockTest.java
> line 59:
>
>> 57: static native int rawMonitorEnter(int id);
>> 58:
>> 59: static {
>
> We are already passing `-agentlib:PrintRawMonitorLockTest` so we could remove
> this block.
Okay, this is already loaded. Thanks for noticing that.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/32092#discussion_r3685498358
PR Review Comment: https://git.openjdk.org/jdk/pull/32092#discussion_r3685502396
