RFR: 8335217: Fix memory ordering in ClassLoaderData::ChunkedHandleList [v4]

Stefan Karlsson stefank at openjdk.org
Fri Jun 28 15:10:20 UTC 2024


On Fri, 28 Jun 2024 13:14:32 GMT, Zhengyu Gu <zgu at openjdk.org> wrote:

>> ClassLoaderData::ChunkedHandleList's head is installed via Atomic::release_store(). Therefore, readers need Atomic::Atomic::load_acquire() barrier to access its members safely.
>
> Zhengyu Gu has updated the pull request incrementally with one additional commit since the last revision:
> 
>   stefank's comments

I'm approving this, but really would prefer if you reverted the line in the destructor.

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

Marked as reviewed by stefank (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/19919#pullrequestreview-2148269307


More information about the hotspot-runtime-dev mailing list