8186502: Assert when range testing G1RefProcDrainInterval on 64-bit systems
Reviewed-by: sjohanss, sangheki
--- a/src/hotspot/share/gc/g1/g1_globals.hpp Mon Nov 06 09:44:42 2017 +0100
+++ b/src/hotspot/share/gc/g1/g1_globals.hpp Mon Nov 06 09:41:54 2017 +0100
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2001, 2016, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2001, 2017, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -74,10 +74,10 @@
"in milliseconds.") \
range(1.0, DBL_MAX) \
\
- product(intx, G1RefProcDrainInterval, 10, \
+ product(int, G1RefProcDrainInterval, 10, \
"The number of discovered reference objects to process before " \
"draining concurrent marking work queues.") \
- range(1, max_intx) \
+ range(1, INT_MAX) \
\
experimental(double, G1LastPLABAverageOccupancy, 50.0, \
"The expected average occupancy of the last PLAB in " \
--- a/test/hotspot/jtreg/runtime/CommandLine/OptionsValidation/common/optionsvalidation/JVMOptionsUtils.java Mon Nov 06 09:44:42 2017 +0100
+++ b/test/hotspot/jtreg/runtime/CommandLine/OptionsValidation/common/optionsvalidation/JVMOptionsUtils.java Mon Nov 06 09:41:54 2017 +0100
@@ -216,6 +216,9 @@
case "CMSPrecleanDenominator":
option.addPrepend("-XX:CMSPrecleanNumerator=" + ((new Integer(option.getMin())) - 1));
break;
+ case "G1RefProcDrainInterval":
+ option.addPrepend("-XX:+ExplicitGCInvokesConcurrent");
+ break;
case "InitialTenuringThreshold":
option.addPrepend("-XX:MaxTenuringThreshold=" + option.getMax());
break;