Hi Bengt --<br><br>The change looks reasonable, but I have a comment and a follow-up question.<br><br>Not your change, but I'd elide the "half the real survivor size" since it's really a configurable parameter based on TargetSurvivorRatio with default half.<br>
I'd leave the comment as "set the new tenuring threshold and desired survivor size".<br><br>I'm curious though, as to what performance data prompted this change, and whether it might make sense, upon a promotion failure to do something about the tenuring threshold for the next scavenge (i.e. for example make the tenuring threshold half of its current value as a reaction to the fact that promotion failed). Is it currently left at its previous value or is it asjusted back to the default max value (which latter may be the wrong thing to do) or something else?<br>
<br>-- ramki<br><br><div class="gmail_quote">On Thu, Jan 10, 2013 at 1:30 AM, 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"><br>
Hi everyone,<br>
<br>
Could I have a couple of reviews for this small change to make DefNew and ParNew be more consistent in the way they treat the tenuring threshold:<br>
<br>
<a href="http://cr.openjdk.java.net/%7Ebrutisso/8005972/webrev.00/" target="_blank">http://cr.openjdk.java.net/~<u></u>brutisso/8005972/webrev.00/</a><br>
<br>
Thanks,<br>
Bengt<br>
</blockquote></div><br>