RFR: 8140489: Introduce shorthand for average_time_ms in G1CollectorPolicy
Derek White
derek.white at oracle.com
Tue Oct 27 21:23:50 UTC 2015
This looks fine.
It would be really nice to factor out all of the "update_stats" work
from record_collection_pause_end(), so we can see the actually policy
decisions. But maybe you are headed in that direction.
- Derek
On 10/27/15 11:41 AM, Erik Helin wrote:
> Hi all,
>
> this small patch introduces the little helper method
> G1CollectorPolicy::average_tims_ms instead of calling
> g1_phases()->average_time_ms, making the code a bit shorter and easier
> to read.
>
> Enhancement:
> https://bugs.openjdk.java.net/browse/JDK-8140489
>
> Webrev:
> http://cr.openjdk.java.net/~ehelin/8140489/webrev.00/
>
> Testing:
> JPRT
>
> Thanks,
> Erik
More information about the hotspot-gc-dev
mailing list