<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">

<html>

<head>

  <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">

</head>

<body bgcolor="#ffffff" text="#000000">

Hi Kirk,<br>

<br>

I thought CMS only used PrintGC and PrintGCDetails - I can't find

CMSLogLevel mentioned in the source.<br>

<br>

If you're referring to the JEP that was recently published - then I

think the intent is to normalize them.<br>

<br>

Cheers,<br>

<br>

JohnC<br>

<br>

On 12/04/12 09:21, Kirk Pepperdine wrote:

<blockquote cite="mid:EFE058E1-E9D0-4DC0-B1AC-266B6EF8FD71@kodewerk.com"

 type="cite">

  <pre wrap="">Would it be worthwhile to explore normalizing the G1LogLevel with CMSLogLevel values?

Regards,

Kirk

On 2012-12-03, at 6:52 PM, John Cuthbertson <a class="moz-txt-link-rfc2396E" href="mailto:john.cuthbertson@oracle.com"><john.cuthbertson@oracle.com></a> wrote:

  </pre>

  <blockquote type="cite">

    <pre wrap="">Hi Everyone,

Can I have a couple of volunteers look over the changes for this CR - the webrev can be found at: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~johnc/8004170/webrev.0/">http://cr.openjdk.java.net/~johnc/8004170/webrev.0/</a>

Summary:

When we changed the G1 evacuation pause to not use instances of the TraceTime class we forgot to flush the GC log file (which happened during the TraceTime destructor). The fix is to add a flush call to G1CollectedHeap::log_gc_footer(). I also noticed that TraceCPUTime destructor does not flush the specified log file and, since it only seems to be instantiated from GC code, it probably should.

Testing:

GCOld with -XX:G1LogLevel=fine,finer,finest and -Xloggc. The submitter of the CR has also verified the fix.

Thanks,

JohnC

    </pre>

  </blockquote>

  <pre wrap=""><!---->

  </pre>

</blockquote>

<br>

</body>

</html>