RFR: 8273000: Remove WeakReference-based class initialisation barrier implementation [v2]

Paul Sandoz psandoz at openjdk.java.net
Wed Sep 1 18:33:32 UTC 2021


On Wed, 1 Sep 2021 16:16:38 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

>> Get rid of WeakReference-based logic in DirectMethodHandle::checkInitialized() and reimplement it with `Unsafe::ensureClassInitialized()`/`shouldBeInitialized()`. 
>> 
>> The key observation is that `Unsafe::ensureClassInitialized()` does not block the initializing thread. 
>> 
>> Also, removed `Unsafe::shouldBeInitialized()` in `DMH::shouldBeInitialized(MemberName)` to save on calling into the VM.
>> `Unsafe::ensureClassInitialized()` already has a fast-path check which checks whether the class is fully initialized or not.
>> 
>> Testing: tier1 - tier6
>
> Vladimir Ivanov has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Improve comments

Looks good, just a minor suggestion up to you to accept or not.

src/java.base/share/classes/jdk/internal/misc/Unsafe.java line 1152:

> 1150:      * The call returns when either class {@code c} is fully initialized or
> 1151:      * class {@code c} is being initialized and the call is performed from
> 1152:      * the initializing thread.

Suggestion:

     * The call returns when either class {@code c} is fully initialized or
     * class {@code c} is being initialized and the call is performed from
     * the initializing thread. In the latter case a subsequent call to
     * {@link #shouldBeInitialized}, from the calling thread of this call,
     * will return {@code true}.

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

Marked as reviewed by psandoz (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/5258


More information about the core-libs-dev mailing list