[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-hotspot-runtime-dev
Subject:    Re: RFR: 8323724: Remove potential re-inflation from FastHashCode under LM_LIGHTWEIGHT
From:       Axel Boldt-Christmas <aboldtch () openjdk ! org>
Date:       2024-04-25 7:57:27
Message-ID: GbKMmDyzdKVIrAogMpYwEdO5ZXKeC3yVDKf8mW18p8s=.4bc9f2dc-2983-4596-b490-8127c06bb715 () github ! com
[Download RAW message or body]

On Wed, 24 Apr 2024 16:18:25 GMT, Coleen Phillimore <coleenp@openjdk.org> wrote:

> > This enhancement will remove the last code path which can inflate an \
> > ObjectMonitor from a non java thread, which in turn will simplify the \
> > is_lock_owned logic and allow for stronger assertions.
> 
> Is there an RFE for this? It looks like a good cleanup.

`is_lock_owned` was removed in #17626 . But the all the `nullptr` checks in \
`inflate_impl` can be removed and ` current->is_Java_thread()` can be asserted when \
`LockingMode == LM_LIGHTWEIGHT`. 

I do not believe there is an RFE for this.

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

PR Comment: https://git.openjdk.org/jdk/pull/18876#issuecomment-2076590227


[prev in list] [next in list] [prev in thread] [next in thread]