RFR: 8376666: Convert G1BlockOffsetTable to use Atomic<T> [v2]

Ivan Walulya iwalulya at openjdk.org
Wed Feb 4 13:13:03 UTC 2026


On Wed, 4 Feb 2026 09:35:37 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> Hi all,
>> 
>>   please review this conversion of the G1 block offset table to use `Atomic<T>`.
>> 
>> Testing: tier1-5, gha
>
> Thomas Schatzl has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
> 
>  - * only change G1 BOT for now
>  - Merge branch 'master' into 8376666-convert-bot
>  - 8376666
>    
>    Hi all,
>    
>      please review this conversion of the various block offset tables to use `Atomic<T>`. Since they are very similar, and mostly changes from `uint8*` to `Atomic<uint8>*` I merged the conversion together.
>    
>    It is not the goal to actually merge them into one class that all three collectors use.
>    
>    Testing: tier1-5, gha

Marked as reviewed by iwalulya (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/29482#pullrequestreview-3750989248


More information about the hotspot-gc-dev mailing list