<div dir="ltr">Hi Jon,<div><br></div><div>Just a comment on comment:</div><div><br></div><div><pre style="color:rgb(0,0,0);word-wrap:break-word;white-space:pre-wrap">        // Swap the survivor spaces if from_space is empty. The
        // resize_young_gen() called below is normally used after
        // a successful young GC and expects the survivor spaces
        // have been swapped.</pre></div><div>-></div><div><pre style="color:rgb(0,0,0);word-wrap:break-word;white-space:pre-wrap">        // Swap the survivor spaces if from_space is empty. The
        // resize_young_gen() called below is normally used after
        // a successful young GC and swapping of survivor spaces;
        // otherwise, it won't succeed to resize young gen under
        // the current mechanism</pre></div><div>Is it clearer?</div><div><br></div><div>Thanks.</div><div>Tao</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 26, 2014 at 1:27 PM, Jon Masamitsu <span dir="ltr"><<a href="mailto:jon.masamitsu@oracle.com" target="_blank">jon.masamitsu@oracle.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">7098155: Resize young gen at full collections for UseParallelGC<br>
<br>
This change adds the call to resize_young_gen() to resize the young generation<br>
 after a full collection.<br>
<br>
<a href="http://cr.openjdk.java.net/~jmasa/7098155/webrev.00/" target="_blank">http://cr.openjdk.java.net/~<u></u>jmasa/7098155/webrev.00/</a><br>
<br>
This is the CR I inherited from Tao.  I changed his original patch<br>
to remove the flag that originally guarded the change.  The flag<br>
was put in because we originally thought this would go into<br>
the jdk8 release rather late and considered it safer to guard<br>
the change.<span class="HOEnZb"><font color="#888888"><br>
<br>
Jon<br>
<br>
<br>
</font></span></blockquote></div><br></div>