RFR: JDK-8151674: STW phases at Concurrent GC should count in PerfCounter (original) (raw)

Jon Masamitsu jon.masamitsu at oracle.com
Wed Mar 16 18:13:04 UTC 2016

On 03/10/2016 07:59 PM, Yasumasa Suenaga wrote:

Hi all,

This review request continues from: http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2016-March/016774.html http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2016-March/016896.html I wonder that STW phases (Remark and Cleanup) at G1 are not counted in jstat FGC column. For example, Initial Mark and Remark at CMS are counted as FGC. We discussed on hotspot-gc-dev for this issue, and I proposed to add new PerfCounter for CGC STW phases. I uploaded webrev. Could you review it? hotspot: http://cr.openjdk.java.net/~ysuenaga/JDK-8151674/webrev.00/hotspot/ jdk: http://cr.openjdk.java.net/~ysuenaga/JDK-8151674/webrev.00/jdk/ For compatibility, this patch works the same as the current by default. If you set -XX:+EnableConcGCPerfCounter, CGC counter will work fine. (I want to set +EnableConcGCPerfCounter by default)

I would prefer not to have the flag (EnableConcGCPerfCounte) and just make the change for CMS. This is a change for a major release and I think we can accept the incompatibility.

http://cr.openjdk.java.net/~ysuenaga/JDK-8151674/webrev.00/hotspot/src/share/vm/gc/g1/g1MonitoringSupport.cpp.udiff.html

Is there a hard connection between these phases and a generational collector (i.e., should the "In a generational collector this would be the" be dropped in the comment)?

A comment such as

// name "collector.2". STW phases as part of a concurrent collection.

A set of JDK changes had been previously been reviewed (not by me). Are these the same?

http://cr.openjdk.java.net/~ysuenaga/JDK-8151674/webrev.00/jdk/

Jon

I cannot access JPRT. So I need a sponsor. Thanks, Yasumasa

More information about the hotspot-gc-dev mailing list