RFR: 8365191: Cleanup after removing LockingMode related code [v2]

Coleen Phillimore coleenp at openjdk.org
Wed Sep 24 12:50:50 UTC 2025


On Wed, 24 Sep 2025 11:44:58 GMT, Fredrik Bredberg <fbredberg at openjdk.org> wrote:

>> src/hotspot/share/runtime/vmStructs.cpp line 685:
>> 
>>> 683:   volatile_nonstatic_field(ObjectMonitor,      _owner,                                        int64_t)                               \
>>> 684:   volatile_nonstatic_field(ObjectMonitor,      _next_om,                                      ObjectMonitor*)                        \
>>> 685:   volatile_nonstatic_field(BasicLock,          _monitor,                                      ObjectMonitor*)                        \
>> 
>> Since nothing now refers to this, you can delete it from vmStructs.
>
> According to @mur47x111, they still need this line for their fast locking implementation.

Oh you were supposed to leave the field in vmStructs_jvmci.cpp and remove it from this one.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27448#discussion_r2375681060


More information about the serviceability-dev mailing list