RFR: 8168904: Initialize and reset G1 phase times to zero

Stefan Johansson stefan.johansson at oracle.com
Fri Nov 4 09:36:21 UTC 2016


Hi Erik,

On 2016-10-28 16:39, Erik Helin wrote:
> Hi all,
>
> this small patch initializes all fields of G1GCPhaseTimes to 0.0
> (or 0 for counts) and also resets them at the start of each GC. When
> working on "8167679 - G1 phase logging is messy", I ran into problems
> when summarizing the values (because some fields where uninitialized).
> Instead of using if statements to determine which fields are valid to
> summarize, it is easier to just initialize and reset them all correctly.
>
> Bug:
> https://bugs.openjdk.java.net/browse/JDK-8168904
>
> Webrev:
> http://cr.openjdk.java.net/~ehelin/8168904/00/
Looks good.
StefanJ
> Testing:
> - Built on Linux x86-64
> - make test-hotspot-gtest
> - make test TEST=hotspot_gc
>
> Thanks,
> Erik




More information about the hotspot-gc-dev mailing list