RFR: 8319773: Avoid inflating monitors when installing hash codes for LM_LIGHTWEIGHT [v4]

Coleen Phillimore coleenp at openjdk.org
Thu Nov 16 13:40:49 UTC 2023


On Thu, 16 Nov 2023 10:03:07 GMT, Axel Boldt-Christmas <aboldtch at openjdk.org> wrote:

>> Also I think the comment before this function is wrong.  We can inflate for deoptimization.
>
>> but you could move it down to before 'inflate'
> 
> Done.
> 
>> Also I think the comment before this function is wrong. We can inflate for deoptimization.
> 
> The comment on `is_lock_owned`? For relock_objects `current` is the deoptee's thread which is a JavaThread.

Yes that comment.  If we don't inflate for hashcode anymore for lightweight locking mode, why would we inflate with a non-JavaThread? Is it impossible?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16603#discussion_r1395703162


More information about the hotspot-dev mailing list