RFR: 8376119: G1: Convert volatiles in G1CMMarkStack to Atomic<T>

Thomas Schatzl tschatzl at openjdk.org
Mon Jan 26 09:47:36 UTC 2026


On Fri, 23 Jan 2026 19:15:47 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

>> `
>>   struct TaskQueueEntryChunk {
>>     TaskQueueEntryChunk* next;
>>     G1TaskQueueEntry data[EntriesPerChunk];
>>   };
>> `
>> 
>> In this case, should we also have `TaskQueueEntryChunk* next;` as Atomic?
>
>> `struct TaskQueueEntryChunk { TaskQueueEntryChunk* next; G1TaskQueueEntry data[EntriesPerChunk]; };`
>> 
>> In this case, should we also have `TaskQueueEntryChunk* next;` as Atomic?
> 
> I only found uses of `next` while holding `G1MarkStackChunkList_lock`.

Thanks @kimbarrett @walulyai for your reviews

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

PR Comment: https://git.openjdk.org/jdk/pull/29365#issuecomment-3798691614


More information about the hotspot-gc-dev mailing list