RFR: JDK-8151912: guarantee(GCPauseIntervalMillis >= 1) failed: Constraint for GCPauseIntervalMillis should guarantee that value is >= 1

[Bengt Rutisson]

Hi Thomas,

On 2016-03-16 15:21, Thomas Schatzl wrote:
> Hi,
>
> On Wed, 2016-03-16 at 15:07 +0100, Bengt Rutisson wrote:
>> Hi all,
>>
>> Could I have a couple of reviews for this change?
>>
>> http://cr.openjdk.java.net/~brutisso/8151912/webrev.00/
>> https://bugs.openjdk.java.net/browse/JDK-8151912
>>
>> The problem is that we do this during setup of G1CollectorPolicy:
>>
>>       FLAG_SET_DEFAULT(GCPauseIntervalMillis, MaxGCPauseMillis + 1);
>>
>> If MaxGCPauseMillis is max uint we will overflow and
>> GCPauseIntervalMillis will be 0.
>>
>> Before JDK-8151604 we did not notice this overflow.
>>
>> The proposed change reduces the max value for MaxGCPauseMillis to
>> max_uint - 1. I think it can be argued that we should have a more
>> reasonable max value, but I don't have a good suggestion for what
>> that
>> value would be. So, for now, to fix the current failures in nightly
>> testing, I propose to just reduce it to the largest value that we can
>> actually handle.
>    Looks good.

Thanks!

Since this is a pretty simple fix and a blocker for the nightly testing 
I will be pushing this now without waiting 24h.

Thanks,
Begnt


>
> Thomas
>