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