RFR: 8376357: Parallel: Convert MutableSpace classes to use Atomic<T>

David Holmes dholmes at openjdk.org
Wed Jan 28 21:20:27 UTC 2026


On Mon, 26 Jan 2026 17:14:07 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

> Hi all,
> 
>   please review these changes that convert `MutableSpace` classes to use `Atomic<T>`.
> 
> Testing: gha, tier1-5
> 
> Thanks,
>   Thomas

Overall looks good to me, but one nit.

Thanks

src/hotspot/share/runtime/vmStructs.cpp line 904:

> 902:                                                                           \
> 903:   declare_toplevel_type(void*)                                            \
> 904:   declare_toplevel_type(Atomic<HeapWord*>)                                \

This placement (under "C primitive pointer types ") seems inappropriate.

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

Marked as reviewed by dholmes (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/29427#pullrequestreview-3719192589
PR Review Comment: https://git.openjdk.org/jdk/pull/29427#discussion_r2738641833


More information about the hotspot-dev mailing list