RFR: 8290814: Hide G1RootRegions behind G1ConcurrentMark

Tue Aug 2 21:58:41 UTC 2022

On Thu, 21 Jul 2022 14:38:20 GMT, Thomas Schatzl <tschatzl 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.

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

Marked as reviewed by sangheki (Reviewer).

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