RFR: 8365190: Remove LockingMode related code from share

Coleen Phillimore coleenp at openjdk.org
Thu Sep 4 11:21:55 UTC 2025


On Thu, 4 Sep 2025 02:26:26 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> src/hotspot/share/runtime/lightweightSynchronizer.cpp line 769:
>> 
>>> 767: 
>>> 768: // LightweightSynchronizer::inflate_locked_or_imse is used to to get an inflated
>>> 769: // ObjectMonitor* when lightweight locking is used. It is used from contexts
>> 
>> I guess you don't need the phrase "when lightweight locking is used".
>
> Even calling it "lightweight locking" is no longer needed.

I think the name "lightweight locking" is used for the file name and class name, so the name is okay.  It does help if you're trying to understand the history of the locking algorithm.

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

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


More information about the hotspot-dev mailing list