RFR: 8371956: Convert OopStorage to use Atomic<T>

[Kim Barrett]

On Mon, 17 Nov 2025 10:32:16 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

>> Please review this change to OopStorage to use Atomic<T> rather than directly
>> applying AtomicAccess to volatile members.
>> 
>> Testing: mach5 tier1-5
>
> test/hotspot/gtest/gc/shared/test_oopStorage.cpp line 50:
> 
>> 48: 
>> 49:   static ActiveArray& active_array(const OopStorage& storage) {
>> 50:     return *storage._active_array.load_acquire();
> 
> Is there a reason why this was changed to `load_acquire` instead of `load_relaxed`?

I think my fingers automatically typed "acquire" instead of "relaxed".  Will be fixed in next push.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28338#discussion_r2533778771