[master] RFR: OMWorld: reenable all platforms

Coleen Phillimore coleenp at openjdk.org
Wed Jun 12 18:57:30 UTC 2024


On Wed, 12 Jun 2024 14:53:54 GMT, Roman Kennke <rkennke at openjdk.org> wrote:

>> This reenables all platforms to use OMWorld, and by extension UseCompactObjectHeaders. 
>> 
>> This change simply calls the runtime if a lock is inflated, until port support for OMWorld cache lookup is added.
>> 
>> ARM (32-bit) required no changes as it already always called the runtime when a monitor is inflated.
>
> src/hotspot/share/runtime/lightweightSynchronizer.cpp line 1045:
> 
>> 1043:   const markWord mark = obj->mark();
>> 1044: 
>> 1045:   if (mark.is_unlocked()) {
> 
> Was that not needed before? Or why is this added now?

Also wondering this.  We come here if the object is locked via slow path, but can it become unlocked only for the platforms that come don't check the ObjectMonitor and go slow path?

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

PR Review Comment: https://git.openjdk.org/lilliput/pull/174#discussion_r1636943499


More information about the lilliput-dev mailing list