RFR: 8365190: Remove LockingMode related code from share [v3]

Fredrik Bredberg fbredberg at openjdk.org
Mon Sep 8 07:08:32 UTC 2025


On Mon, 8 Sep 2025 05:33:05 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Fredrik Bredberg has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   New version for Coleen
>
> src/hotspot/share/runtime/lightweightSynchronizer.cpp line 768:
> 
>> 766: }
>> 767: 
>> 768: // LightweightSynchronizer::inflate_locked_or_imse is used to to get an
> 
> Suggestion:
> 
> // LightweightSynchronizer::inflate_locked_or_imse is used to get an

Fixed

> src/hotspot/share/runtime/lightweightSynchronizer.cpp line 822:
> 
>> 820:   // The JavaThread* locking parameter requires that the
>> 821:   // locking_thread == JavaThread::current, or is suspended throughout
>> 822:   // the call by some other mechanism.
> 
> Suggestion:
> 
>   // The JavaThread* locking parameter requires that the locking_thread == JavaThread::current, 
>   // or is suspended throughout the call by some other mechanism.

Fixed

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27041#discussion_r2329325202
PR Review Comment: https://git.openjdk.org/jdk/pull/27041#discussion_r2329325682


More information about the serviceability-dev mailing list