Integrated: 8290814: Hide G1RootRegions behind G1ConcurrentMark

Thomas Schatzl tschatzl at openjdk.org
Tue Aug 9 12:04:47 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

This pull request has now been integrated.

Changeset: 0ade2641
Author:    Thomas Schatzl <tschatzl at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/0ade2641f78c1ab890de658095b6b54e52f8d5ab
Stats:     37 lines in 4 files changed: 24 ins; 11 del; 2 mod

8290814: Hide G1RootRegions behind G1ConcurrentMark

Reviewed-by: sangheki, iwalulya

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

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



More information about the hotspot-gc-dev mailing list