<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<br>
<div class="moz-cite-prefix">On 2/26/2014 1:51 PM, Tao Mao wrote:<br>
</div>
<blockquote
cite="mid:CANrGW1z=ZPHfFCKbeXJZk9ukqcwdqci1_0U0OdnPc7-oZy+HOA@mail.gmail.com"
type="cite">
<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>
</blockquote>
<br>
Yes, I will add it. Thanks.<br>
<br>
Jon<br>
<br>
<blockquote
cite="mid:CANrGW1z=ZPHfFCKbeXJZk9ukqcwdqci1_0U0OdnPc7-oZy+HOA@mail.gmail.com"
type="cite">
<div dir="ltr">
<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 moz-do-not-send="true"
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 moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Ejmasa/7098155/webrev.00/"
target="_blank">http://cr.openjdk.java.net/~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>
</blockquote>
<br>
</body>
</html>