RFR(S) 8154467: Cleanup initialization of GCPolicyCounters
Stefan Johansson
stefan.johansson at oracle.com
Wed Apr 20 12:26:35 UTC 2016
Hi Mikael,
On 2016-04-20 13:17, Mikael Gerdin wrote:
> Hi all,
>
> Please review this small cleanup to how GCPolicyCounters are
> initialized and managed.
>
> GCPolicyCounters* counters() is part of the CollectorPolicy API but in
> fact it is not necessary to expose it at such a general level.
> G1 has only used the object internally in the policy code and managing
> the policy counters object can be safely moved inside the G1Policy.
>
> ParalellGC has ignored initializing the field in the CollectorPolicy at
> all, since it does not really use the CollectorPolicy API.
>
> The Gen collectors can be adapted to access the counters() accessor
> through the GenCollectorPolicy class instead.
>
> Testing: RBT GC Testing
> Webrev: http://cr.openjdk.java.net/~mgerdin/8154467/webrev.0/
> Bug: https://bugs.openjdk.java.net/browse/JDK-8154467
>
Looks good,
StefanJ
> /Mikael
More information about the hotspot-gc-dev
mailing list