<br><br><div class="gmail_quote">On Wed, Mar 10, 2010 at 11:10 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="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im"><br>
On Mar 10, 2010, at 4:04 PM, Jon Masamitsu wrote:<br>
<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hiroshi,<br>
<br>
Have you signed a contributor agreement?  The gentleman<br>
who used to take care of those isn't in the group any longer<br>
and I'll have to go hunt it down before I look at the<br>
code.<br>
</blockquote>
<br></div>
I got confirmation that you're eligible to contribute.</blockquote><div><br>Good. <br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
I'm going to be out of the office doing a<br>
presentation tomorrow so won't be able to pursue<br>
this until Friday at the earliest.<br>
<br>
But just in general when UseAdativeSizePolicy is<br>
off the UseParallelGC collector doesn't do any resizing.<br>
Can you just make this the sizing policy when<br>
UseAdativeSizePolicy is off (instead of adding the<br>
new flags)?<br>
</blockquote>
<br></div>
Sorry, I misread your mail.  I can see the reason for the<br>
new flags.   I'll take a look at the  patch in the next<br>
couple of days.</blockquote><div><br>No problem. <br><br>Thanks,<br>Hiroshi<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div>
<div></div><div class="h5"><br>
<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
Jon<br>
<br>
<br>
On 03/10/10 15:17, Hiroshi Yamauchi wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hi,<br>
I'd like to have this patch<br>
 <a href="http://cr.openjdk.java.net/%7Erasbold/69XXXXX/webrev.00/" target="_blank">http://cr.openjdk.java.net/~rasbold/69XXXXX/webrev.00/</a><br>
contributed, if appropriate.<br>
It implements a simple heap shrinking logic based on the free ratio (MaxHeapFreeRatio) in the parallel collector. The way it works is the following: If the free ratio, (1.0 - <used> / <capacity>) * 100 > MaxHeapFreeRatio, this logic kicks in. Otherwise, the (default) adaptive size policy runs the show (as before). This feature is turned off by default. There are two new flags to turn it on:<br>

 UseFreeRatioForParallelGC - this enables it.<br>
 UseFreeRatioOnlyInSystemGCForParallelGC - this enables it only in explicit System.gc() calls. This is more useful in apps that call System.gc() when it's idle and don't want shrinking to happen at all other times.<br>

In our tests, it appears to work well in a simple test and to help a real app reduce its footprint (RSS) when it's idle.<br>
Thanks to Chuck Rasbold who uploaded the webrev on my behalf.<br>
Thanks,<br>
Hiroshi<br>
</blockquote></blockquote>
<br>
</div></div></blockquote></div><br>