<br><br><div class="gmail_quote">On Fri, Mar 26, 2010 at 4:59 PM, Jon Masamitsu <span dir="ltr"><<a href="mailto:Jon.Masamitsu@sun.com">Jon.Masamitsu@sun.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im"><br>
On Mar 24, 2010, at 5:01 PM, Hiroshi Yamauchi wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Jon,<br>
<br>
You're right that adjust_for_throughput() will not shrink the heap.<br>
Another part of the policy is that the heap is shrunk if the<br>
throughput goal and the pause time goal are being<br>
met.   It's a slower process than just enforcing MaxHeapFreeRatio<br>
and requires a number of GC's (that is, the heap would be<br>
shrunk down a percentage at each GC).<br>
<br>
GC usually happens only as needed. So, after the application goes idle, GC does not probably happen. The next GC happens only after the application goes active again. Assuming that, there aren't many chances to shrink the heap (maybe only one). If the application calls System.gc() right before going idle, that may be the only chance. In such cases, the slower process may not work very well.<br>

</blockquote>
<br></div>
Agreed.  The option to use MaxHeapFreeRatio/MinHeapFreeRation on a<br>
System.gc() is needed.  As you say we probably only get the System.gc()<br>
to do anything. </blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
We would<br>
need to change the UseAdaptiveSizePolicy to do something like<br>
<br>
<br>
if ( average-pause > pause-goal)<br>
         shrink heap<br>
else if (average-throughput < throughput-goal) &&<br>
           ! (UseFreeRatioForParallelGC && heap too empty))  <<<< new<br>
        grow heap<br>
else<br>
        shrink heap<br>
<br>
<br>
Or maybe something like<br>
<br>
<br>
if ( average-pause > pause-goal)<br>
         shrink heap<br>
else if (average-throughput < throughput-goal)<br>
         if  (UseFreeRatioForParallelGC && heap too empty)    <<<< new<br>
                shrink heap (policy-to-be-determined)         <<<<<br>
         else                                                 <<<<<br>
                grow heap<br>
else if (average-throughput < throughput-goal)<br>
        grow heap to reduce gc overhead<br>
else<br>
        shrink heap<br>
<br>
<br>
OK. I think the latter may be able to shrink the heap more aggressively.<br>
<br>
Do you think the free ratio-based shrinking should happen only when the throughput goal is met? Using the above scenario, when the application is working hard (and perhaps GC is working hard as well) and goes idle next moment, the average throughput at the time of the phase change may not be good enough for the goal to be met (hence no shrinking).<br>

</blockquote>
<br>
<br></div>
I don't believe we can handle that case (application suddenly going idle and GC does<br>
the right thing).   As you say earlier we might not even have a GC.   And whose to know<br>
if the going idle isn't to be followed almost immediately by furious activity again.  The application has to<br>
make that call.   What we add here is the requirement to limit the amount of free space in<br>
the heap even if the throughput goal is not being met.<br></blockquote><div><br></div><div>I think we agree that it's a good idea to try to shrink the heap in response to System.gc() in such a scenario/app. </div><div>
<br></div><div>Since we are talking about a setting where the free ratio flags takes precedence over the psAdaptiveSizePolicy's throughput goals, the suggested logic in the original webrev is perhaps not so terrible? If so, I suppose the logic for UseFreeRatioOnlyInSystemGCForParallelGC does help with that because it can shrink the heap regardless of the throughput goal. It may make sense to remove UseFreeRatioForParallelGC and keep UseFreeRatioOnlyInSystemGCForParallelGC only. If you'd like to see it happen inside psAdaptiveSizePolicy instead, the current webrev does not work.</div>
<div><br></div><div>Thanks.</div><div><br></div><div>Hiroshi</div><div><br></div></div>