<div dir="ltr">Hi Thomas,<div><br></div><div style>Could the same kind of update be done for OldSize and SurvivorRatio?</div><div style>For OldSize, if it's not explicitly set then it'll always show a default value which is like 5MB or something, regardless of the collector used.</div>
<div style>For SurvivorRatio, when using UseParallelGC or UseParallelOldGC, if it's not explicitly set then InitialSurvivorRatio is the flag that actually takes effect. It's better if SurvivorRatio could be updated accordingly too.</div>
<div style><br></div><div style>Thanks,</div><div style>Kris</div><div style><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Apr 18, 2013 at 1:53 AM, 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>
  could I have some reviews for the change for CR 7057939?<br>
<br>
The problem presented in that CR is that if MaxNewSize is not specified on the command line, it will never be updated by the internal sizing logic.<br>
<br>
So programs like jmap that read MaxNewSize always display uintx_max.<br>
<br>
The patch simply updates the MaxNewSize global after the sizing logic<br>
has been applied for all heaps (in GenCollectorPolicy and<br>
G1CollectorPolicy) as this problem is not only specific to parallel gc.<br>
<br>
The change for G1CollectorPolicy is slightly more complicated because there are multiple exits in the constructor of G1YoungGenSizer (where this<br>
calculation happens).<br>
Also depending on the sizing goals for G1, a different calculation for MaxNewSize has to be applied.<br>
<br>
In particular, if NewRatio is defined, the MaxNewSize is MaxHeapSize divided by NewRatio, otherwise it is dependent on G1MaxNewSizePercent.<br>
<br>
I also moved some asserts from G1YoungGenSizer to checks in argument processing as the asserts are actually mandatory (range checking).<br>
<br>
Bugs.sun:<br>
<a href="http://bugs.sun.com/view_bug.do?bug_id=7057939" target="_blank">http://bugs.sun.com/view_bug.do?bug_id=7057939</a><br>
<br>
JIRA:<br>
<a href="https://jbs.oracle.com/bugs/browse/JDK-7057939" target="_blank">https://jbs.oracle.com/bugs/browse/JDK-7057939</a><br>
<br>
Webrev:<br>
<a href="http://cr.openjdk.java.net/~tschatzl/7057939/webrev/" target="_blank">http://cr.openjdk.java.net/~tschatzl/7057939/webrev/</a><br>
<br>
Testing:<br>
jprt, jtreg tests<br>
<br>
Thanks,<br>
Thomas<br>
</blockquote></div><br></div>