RFR: 8290814: Hide G1RootRegions behind G1ConcurrentMark

Thomas Schatzl tschatzl at openjdk.org
Tue Aug 9 12:02:48 UTC 2022


On Tue, 2 Aug 2022 21:56:38 GMT, Sangheon Kim <sangheki at openjdk.org> wrote:

>> Hi all,
>> 
>>   please review this cleanup that better encapsulates the `G1RootRegions` member of `G1ConcurrentMark` to be not directly accessible by non-concurrent mark related code. This is a preparatory change for [JDK-8289822](https://bugs.openjdk.org/browse/JDK-8289822) in that since the TAMSes are then owned by `G1ConcurrentMark`, there is the possibility to avoid allowing write access to these to non-G1ConcurrentMark classes as well.
>> 
>> [here](https://github.com/openjdk/jdk/compare/master...tschatzl:jdk:submit/8289822-move-tams?expand=1) you can see how that change could look like (also includes [JDK-8290357](https://bugs.openjdk.org/browse/JDK-8290357) that is also out for review).
>> 
>> Testing: tier1-3
>> 
>> Thanks,
>>   Thomas
>
> Looks good.

Thanks @sangheon @walulyai for your reviews

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

PR: https://git.openjdk.org/jdk/pull/9593



More information about the hotspot-gc-dev mailing list