<div dir="ltr">Looks good.<div><br></div><div>Thanks.</div><div>Tao</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Mar 20, 2016 at 7:43 PM, Jon Masamitsu <span dir="ltr"><<a href="mailto:jon.masamitsu@oracle.com" target="_blank">jon.masamitsu@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The averages reported for phase times (for example "Ext Root Scanning") were<br>
incorrect.  Not all the per thread values were included in the sum and the<br>
average value was incorrect (this with build 9-ea+1100)<br>
<br>
[0.366s][debug][gc,phases            ] GC(2)     Ext Root Scanning (ms):   Min:  0.3, Avg:  0.2, Max:  0.4, Diff:  0.0, Sum:  0.3<br>
[0.366s][trace][gc,phases,task       ] GC(2)                                0.4  0.3<br>
<br>
With the fix all values are included in the sum and the average is correct.<br>
<br>
[2.830s][debug][gc,phases            ] GC(0)     Ext Root Scanning (ms):   Min:  5.7, Avg:  7.3, Max:  8.9, Diff:  3.1, Sum: 14.6<br>
[2.830s][trace][gc,phases,task       ] GC(0)                                8.9  5.7<br>
<br>
<a href="https://bugs.openjdk.java.net/browse/JDK-8152208" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8152208</a><br>
<a href="http://cr.openjdk.java.net/~jmasa/8152208/webrev.00/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~jmasa/8152208/webrev.00/</a><br>
<br>
Thanks.<span class="HOEnZb"><font color="#888888"><br>
<br>
Jon<br>
</font></span></blockquote></div><br></div>