<div dir="ltr">Hi Thomas,<div><br></div><div>From a parsing POV would be preferable "Pause Young Normal" would be preferable to Pause Young (Normal)".  I know it's a small change but it does make a difference. The rest of the code is fine (of course not an official review).</div><div><br></div><div>As for the change it's self, [73.077s][info ][gc,start     ] GC(262) Pause Initial Mark (G1 Humongous Allocation) is already perfectly clear to me. I'm not sure how</div><div><div>[73.077s][info ][gc,start     ] GC(262) Pause Young (Initial Mark)  (G1 Humongous Allocation) clarifies things. IMO, this change only adds to the already high level of noise in the GC logs. On that note, I have time scheduled in July to see what can be done to reduce redundancy in the logs.</div></div><div><br></div><div>Kind regards,</div><div>Kirk</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 14, 2018 at 4:02 PM, Thomas Schatzl <span dir="ltr"><<a href="mailto:thomas.schatzl@oracle.com" target="_blank">thomas.schatzl@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<br>
  another round of reviews after some more internal remarks :P<br>
<br>
I also changed the title of the CR.<br>
<br>
The set of "final" tags would be:<br>
<br>
Pause Young (Normal) ...<br>
Pause Young (Concurrent Start) ...<br>
Pause Young (Concurrent End) ...<br>
Pause Young (Mixed) ...<br>
<br>
I also adapted the strings in the GCVerifyType functionality.<br>
<br>
<a href="http://cr.openjdk.java.net/~tschatzl/8204082/webrev.1_to_2" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~<wbr>tschatzl/8204082/webrev.1_to_2</a> (diff)<br>
<a href="http://cr.openjdk.java.net/~tschatzl/8204082/webrev.2" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~<wbr>tschatzl/8204082/webrev.2</a> (full)<br>
<br>
Testing:<br>
running through all gc tests locally<br>
<br>
Thanks,<br>
  Thomas<br>
<br>
</blockquote></div><br></div>