<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Please review the updated webrev which adds new VM flags to allow for the manual <div class="">selection of Heap size based on a percentage of total available memory.</div><div class=""><br class=""></div><div class="">This version deprecates the existing fractional options and allows the new % based flags</div><div class="">to override.<br class=""><div class=""><br class=""></div><div class=""><a href="http://cr.openjdk.java.net/~bobv/8186248/webrev.01/" class="">http://cr.openjdk.java.net/~bobv/8186248/webrev.01/</a></div><div class=""><br class=""></div><div class="">Bob.</div><div class=""><br class=""></div><div class=""><br class=""><div class=""><div><blockquote type="cite" class=""><div class="">On Aug 17, 2017, at 12:36 AM, David Holmes <<a href="mailto:David.Holmes@oracle.com" class="">David.Holmes@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">On 17/08/2017 1:29 PM, Bob Vandette wrote:<br class=""><blockquote type="cite" class="">I saw that but wasn't sure it needed the added flexibility since its probably ok that initial sizes are 50% or less.<br class=""></blockquote><br class="">I'd go for consistency.<br class=""><br class="">Also now you will need to guard against values < 1, I think.<br class=""><br class="">There may be an option checking test that will need updating as well.<br class=""><br class="">Cheers,<br class="">David<br class=""><br class=""><blockquote type="cite" class="">Bob.<br class=""><blockquote type="cite" class="">On Aug 16, 2017, at 5:04 PM, David Holmes <<a href="mailto:david.holmes@oracle.com" class="">david.holmes@oracle.com</a>> wrote:<br class=""><br class="">Hi Bob,<br class=""><br class=""><blockquote type="cite" class="">On 17/08/2017 3:32 AM, Bob Vandette wrote:<br class="">Please review this simple two line fix which allows more flexibility in selecting the % of system RAM<br class="">to be used by the Heap. This just changes two int variables to doubles.<br class="">RFE:<br class=""><a href="https://bugs.openjdk.java.net/browse/JDK-8186248" class="">https://bugs.openjdk.java.net/browse/JDK-8186248</a> <<a href="https://bugs.openjdk.java.net/browse/JDK-8186248" class="">https://bugs.openjdk.java.net/browse/JDK-8186248</a>><br class="">Webrev:<br class=""><a href="http://cr.openjdk.java.net/~bobv/8186248" class="">http://cr.openjdk.java.net/~bobv/8186248</a> <<a href="http://cr.openjdk.java.net/~bobv/8186248" class="">http://cr.openjdk.java.net/~bobv/8186248</a>><br class=""></blockquote><br class="">Wouldn't you also want/need to change the type of InitialRAMFraction?<br class=""><br class="">Note: jdk10/hs is currently closed to changes as we prepare to push up to jdk10/jdk10.<br class=""><br class="">Thanks,<br class="">David<br class=""><br class=""><blockquote type="cite" class="">Bob.<br class=""></blockquote></blockquote></blockquote></div></div></blockquote></div><br class=""></div></div></div></body></html>