Hi Bengt,<div><br></div><div>The current factoring looks nice and uniform. Thanks :-)</div><div><br></div><div>But for most minor GCs and both CMS pause phases, the extra logging doesn't really give additional information.</div>
<div>Most minor GCs are going to say "Allocation Failure", and the two CMS phases would change from, e.g.</div><div><br></div><div>[GC [1 CMS-initial-mark</div><div><br></div><div>to something like</div><div><br>
</div><div>[GC (CMS Initial Mark) [1 CMS-initial-mark</div><div><br></div><div>which is probably reasonable given the scope of the change, but not really helpful.</div><div>The "real cause", such as which generation (or perhaps System.gc() with ExplicitGCInvokesConcurrent, or even GC locker) is triggering this collection cycle, may be more useful, but it's hard to fit into the current form.</div>
<div><br></div><div>- Kris<br><br><div class="gmail_quote">On Tue, May 8, 2012 at 10:18 PM, Bengt Rutisson <span dir="ltr"><<a href="mailto:bengt.rutisson@oracle.com" target="_blank">bengt.rutisson@oracle.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF" text="#000000">
    <br>
    Hi again everyone,<br>
    <br>
    It seems like the feedback on hotspot-gc-use is that we should add
    the GC cause to all collectors but also provide a switch to turn
    this logging off.<br>
    <br>
    Here is an updated webrev:<br>
    <a href="http://cr.openjdk.java.net/~brutisso/7166894/webrev.03/" target="_blank">http://cr.openjdk.java.net/~brutisso/7166894/webrev.03/</a><br>
    <br>
    Changes:<br>
    * GC cause logged for all collectors<br>
    * Added the flag -XX:-PrintGCCause to turn the new information off<br>
    * Refactored the string concatenation code into a helper class<br>
    <br>
    I guess I will also have to update the CR to now reflect the fact
    that this does not just concern full GCs anymore.<br>
    <br>
    Thanks,<br>
    Bengt<div><div class="h5"><br></div></div></div></blockquote></div></div>