RFR (XS): JDK-8027450 : Improve time reporting in G1 remark

Thomas Schatzl thomas.schatzl at oracle.com
Mon Sep 22 09:06:09 UTC 2014


Hi,

On Fri, 2014-09-19 at 11:25 -0400, Joseph Provino wrote:
> Hi all,
> 
> Can I have reviews for the following small change that logs more G1 
> Concurrent Mark
> Trace Times.  I also changed G1RemarkGCTraceTime() to G1CMTraceTime() 
> because it's
> used in places other than remark.
> 
> CR: https://bugs.openjdk.java.net/browse/JDK-8027450
> 
> Webrev:http://cr.openjdk.java.net/~jprovino/8027450/webrev.00/
> 
> Testing: jprt
> 
> I also need a sponsor to commit the changes.

Looks good. Maybe the strings could be changed to fit the existing
though, e.g. "GC ref-proc" -> "Reference processing", and "GC
aggregate-data" -> "Aggregate data".

Thanks,
  Thomas





More information about the hotspot-gc-dev mailing list