<p dir="ltr">Hi Jon,</p>
<p dir="ltr">2016/03/17 3:25 "Jon Masamitsu" <<a href="mailto:jon.masamitsu@oracle.com">jon.masamitsu@oracle.com</a>>:<br>
><br>
><br>
><br>
> On 03/10/2016 07:59 PM, Yasumasa Suenaga wrote:<br>
>><br>
>> Hi all,<br>
>><br>
>> This review request continues from:<br>
>>    <a href="http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2016-March/016774.html">http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2016-March/016774.html</a><br>
>>    <a href="http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2016-March/016896.html">http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2016-March/016896.html</a><br>
>><br>
>> I wonder that STW phases (Remark and Cleanup) at G1 are not counted in jstat FGC column.<br>
>> For example, Initial Mark and Remark at CMS are counted as FGC.<br>
>> We discussed on hotspot-gc-dev for this issue, and I proposed to add new PerfCounter<br>
>> for CGC STW phases.<br>
>><br>
>> I uploaded webrev. Could you review it?<br>
>>      hotspot: <a href="http://cr.openjdk.java.net/~ysuenaga/JDK-8151674/webrev.00/hotspot/">http://cr.openjdk.java.net/~ysuenaga/JDK-8151674/webrev.00/hotspot/</a><br>
>>          jdk: <a href="http://cr.openjdk.java.net/~ysuenaga/JDK-8151674/webrev.00/jdk/">http://cr.openjdk.java.net/~ysuenaga/JDK-8151674/webrev.00/jdk/</a><br>
>><br>
>> For compatibility, this patch works the same as the current by default.<br>
>> If you set -XX:+EnableConcGCPerfCounter, CGC counter will work fine.<br>
>> (I want to set +EnableConcGCPerfCounter by default)<br>
><br>
><br>
> I would prefer not to have the flag (EnableConcGCPerfCounte)<br>
> and just make the change for CMS.  This is a change for a major<br>
> release and I think we can accept the incompatibility.</p>
<p dir="ltr">Okay, I will remove this flag.</p>
<p dir="ltr">I added a comment to JBS about G1 initial mark.<br>
I will upload new webrev after discussing for it.<br></p>
<p dir="ltr">> <a href="http://cr.openjdk.java.net/~ysuenaga/JDK-8151674/webrev.00/hotspot/src/share/vm/gc/g1/g1MonitoringSupport.cpp.udiff.html">http://cr.openjdk.java.net/~ysuenaga/JDK-8151674/webrev.00/hotspot/src/share/vm/gc/g1/g1MonitoringSupport.cpp.udiff.html</a><br>
><br>
> + // name "collector.2". In a generational collector this would be the<br>
> + // STW phases in concurrent collection.<br>
> + _conc_collection_counters =<br>
> + new CollectorCounters("G1 stop-the-world phases", 2);<br>
><br>
><br>
> Is there a hard connection between these phases and a generational<br>
> collector (i.e., should the "In a generational collector this would be the"<br>
> be dropped in the comment)?<br>
><br>
> A comment such as<br>
><br>
> // name "collector.2". STW phases as part of a concurrent collection.</p>
<p dir="ltr">Thanks, I will fix.<br></p>
<p dir="ltr">> A set of JDK changes had been previously been reviewed (not by me). Are these<br>
> the same?<br>
><br>
> <a href="http://cr.openjdk.java.net/~ysuenaga/JDK-8151674/webrev.00/jdk/">http://cr.openjdk.java.net/~ysuenaga/JDK-8151674/webrev.00/jdk/</a></p>
<p dir="ltr">Yes, same patch.<br></p>
<p dir="ltr">Yasumasa<br></p>
<p dir="ltr">> Jon<br>
><br>
><br>
><br>
>><br>
>><br>
>> I cannot access JPRT.<br>
>> So I need a sponsor.<br>
>><br>
>><br>
>> Thanks,<br>
>><br>
>> Yasumasa<br>
><br>
><br>
</p>