RFR: JDK-8151912: guarantee(GCPauseIntervalMillis >= 1) failed: Constraint for GCPauseIntervalMillis should guarantee that value is >= 1 (original) (raw)
Bengt Rutisson bengt.rutisson at oracle.com
Wed Mar 16 14:20:18 UTC 2016
- Previous message (by thread): RFR: JDK-8151912: guarantee(GCPauseIntervalMillis >= 1) failed: Constraint for GCPauseIntervalMillis should guarantee that value is >= 1
- Next message (by thread): RFR: JDK-8076995: gc/ergonomics/TestDynamicNumberOfGCThreads.java failed with java.lang.RuntimeException: 'new_active_workers' missing from stdout/stderr
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
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: FLAGSETDEFAULT(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 maxuint - 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
- Previous message (by thread): RFR: JDK-8151912: guarantee(GCPauseIntervalMillis >= 1) failed: Constraint for GCPauseIntervalMillis should guarantee that value is >= 1
- Next message (by thread): RFR: JDK-8076995: gc/ergonomics/TestDynamicNumberOfGCThreads.java failed with java.lang.RuntimeException: 'new_active_workers' missing from stdout/stderr
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]