RFR: 8168904: Initialize and reset G1 phase times to zero
Joseph Provino
joseph.provino at oracle.com
Fri Oct 28 16:31:56 UTC 2016
Hi Erik, this looks good to me.
joe
On 10/28/2016 10:39 AM, 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/
>
> 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