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

Thomas Schatzl tschatzl at openjdk.org
Thu Jan 29 09:14:19 UTC 2026


On Wed, 28 Jan 2026 21:15:50 GMT, David Holmes <dholmes at openjdk.org> wrote:

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

Since other changes also place this toplevel type at the same location to avoid future merge issues, I would like to wait finding a more appropriate place until later, filed https://bugs.openjdk.org/browse/JDK-8376664 and assigned it to me.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29427#discussion_r2740651055


More information about the hotspot-dev mailing list