RFR: 8186248 - Allow more flexibility in selecting Heap % of available RAM

[Bob Vandette]

> On Aug 17, 2017, at 12:36 AM, David Holmes <David.Holmes at oracle.com> wrote:
> 
> On 17/08/2017 1:29 PM, Bob Vandette wrote:
>> I saw that but wasn't sure it needed the added flexibility since its probably ok that initial sizes are 50% or less.
> 
> I'd go for consistency.
I’m going to wait until the CSR decision is finalized.  I may be adding a new flag which won’t be
a float value.

> 
> Also now you will need to guard against values < 1, I think.
It looks like there’s a range option to our flags now.  I agree we need the range to be 1 to MAX_DBL.
> 
> There may be an option checking test that will need updating as well.

I looked at these and they don’t get impacted by the float change.

If I need to add a new flag, then I’ll have to add a test or 2.

Bob.

> 
> Cheers,
> David
> 
>> Bob.
>>> On Aug 16, 2017, at 5:04 PM, David Holmes <david.holmes at oracle.com> wrote:
>>> 
>>> Hi Bob,
>>> 
>>>> On 17/08/2017 3:32 AM, Bob Vandette wrote:
>>>> Please review this simple two line fix which allows more flexibility in selecting the % of system RAM
>>>> to be used by the Heap.  This just changes two int variables to doubles.
>>>> RFE:
>>>> https://bugs.openjdk.java.net/browse/JDK-8186248 <https://bugs.openjdk.java.net/browse/JDK-8186248>
>>>> Webrev:
>>>> http://cr.openjdk.java.net/~bobv/8186248 <http://cr.openjdk.java.net/~bobv/8186248>
>>> 
>>> Wouldn't you also want/need to change the type of InitialRAMFraction?
>>> 
>>> Note: jdk10/hs is currently closed to changes as we prepare to push up to jdk10/jdk10.
>>> 
>>> Thanks,
>>> David
>>> 
>>>> Bob.