<br><br><div class="gmail_quote">On Wed, Dec 19, 2012 at 12:46 PM, Joel Buckley <span dir="ltr"><<a href="mailto:joel.buckley@oracle.com" target="_blank">joel.buckley@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><div>Hey Bengt,</div><div><br></div><div>Code review looks simple enough.</div><div><br></div><div>One nit on naming consistency:</div><div>Should "<span style="color:rgb(0,0,255);font-family:monospace;font-weight:bold;white-space:pre-wrap">CMSIncrementalMode</span>" be "<span style="color:rgb(0,0,255);font-family:monospace;font-weight:bold;white-space:pre-wrap">UseCMSIncrementalGC</span>"</div>
<div>to be consistent with other flags?</div></div></blockquote><div><br>Historical name, probably not worth renaming especially since that mode is being deprecated now.<br><br></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div style="word-wrap:break-word"><div><br></div><div>A couple more general questions:</div><div><br></div><div>What GC combinations are non-deprecated and what are their</div><div>intended use cases?</div><div><br></div>
<div>For example, what GC combinations are intended for the following?:</div><div>* 4-100GB Heap (e.g. 64bit);  High Priority: 200ms max pause due to GC; Env: >50% Heap long lived (>12hour)</div><div>* 4-100GB Heap (e.g. 64bit);  High Priority: 500ms max pause due to GC; Env: ~40% Heap short lived (<1minute) & ~40% Heap long lived (>12hour)</div>
<div>* 1-3.5GB Heap (e.g. 32bit); High Priority: 200ms max pause due to GC; Env: ~90% Heap short lived (<1minute)</div><div>* 100-1024MB Heap (e.g. 32bit/embedded); High Priority: 500ms max pause due to GC; Env: varying short/long lived data profiles over time</div>
</div></blockquote><div><br>guesstimate: Probably CMS and G1 for all these cases except for embedded, where serial or parallel would probably work. #2 and #3 could probably be tuned for parallel old.<br><br>-- ramki<br>  <br>
</div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word"><div><br></div><div>Thanks,</div><div>Joel.</div><div><br></div>
Date: Wed, 19 Dec 2012 13:40:18 +0100<br>From: Bengt Rutisson <<a href="mailto:bengt.rutisson@oracle.com" target="_blank">bengt.rutisson@oracle.com</a>><br>Subject: Request for review (XS): 8003822: Deprecate the incremental<br>
<span style="white-space:pre-wrap">     </span>mode<span style="white-space:pre-wrap">    </span>of CMS<br>To: <a href="mailto:hotspot-gc-dev@openjdk.java.net" target="_blank">hotspot-gc-dev@openjdk.java.net</a><br>Message-ID: <<a href="mailto:50D1B5B2.4010709@oracle.com" target="_blank">50D1B5B2.4010709@oracle.com</a>><br>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed<div class="im"><br><br><br>Hi all,<br><br>Can I have a couple of reviews for this change to deprecate iCMS?<br><br><a href="http://cr.openjdk.java.net/%7Ebrutisso/8003822/webrev.00/" target="_blank">http://cr.openjdk.java.net/~brutisso/8003822/webrev.00/</a><br>
<br>This is part of the work for JEP 173:<br><br>JEP 173: Retire Some Rarely-Used GC Combinations<br><a href="http://openjdk.java.net/jeps/173" target="_blank">http://openjdk.java.net/jeps/173</a><br><br>The webrev is based on the the earlier webrev that I sent out to <br>
deprecate the DefNew + CMS and ParNew + SerialOld GC combinations:<br><br><a href="http://cr.openjdk.java.net/%7Ebrutisso/8003820/webrev.00/" target="_blank">http://cr.openjdk.java.net/~brutisso/8003820/webrev.00/</a><br>
<br>Thanks,<br>Bengt<br></div></div></blockquote></div><br>