8139900: [TESTBUG] Remove G1UpdateBufferSize and InitialBootClassLoaderMetaspaceSize from TestOptionsWithRanges
Reviewed-by: gziemski, sangheki, ctornqvi
--- a/hotspot/test/runtime/CommandLine/OptionsValidation/TestOptionsWithRanges.java Wed Oct 14 14:51:10 2015 +0200
+++ b/hotspot/test/runtime/CommandLine/OptionsValidation/TestOptionsWithRanges.java Tue Oct 27 14:33:29 2015 +0300
@@ -29,7 +29,7 @@
* java.management
* jdk.attach
* jdk.management/sun.tools.attach
- * @run main/othervm/timeout=600 TestOptionsWithRanges
+ * @run main/othervm/timeout=780 TestOptionsWithRanges
*/
import java.util.ArrayList;
@@ -70,6 +70,8 @@
allOptionsAsMap.remove("G1ConcRefinementThreads");
allOptionsAsMap.remove("G1RSetRegionEntries");
allOptionsAsMap.remove("G1RSetSparseRegionEntries");
+ allOptionsAsMap.remove("G1UpdateBufferSize");
+ allOptionsAsMap.remove("InitialBootClassLoaderMetaspaceSize");
/*
* Remove parameters controlling the code cache. As these
--- a/hotspot/test/runtime/CommandLine/OptionsValidation/common/optionsvalidation/JVMOptionsUtils.java Wed Oct 14 14:51:10 2015 +0200
+++ b/hotspot/test/runtime/CommandLine/OptionsValidation/common/optionsvalidation/JVMOptionsUtils.java Tue Oct 27 14:33:29 2015 +0300
@@ -116,20 +116,6 @@
}
/**
- * Add dependency for option depending on it's type. E.g. run the JVM in
- * compilation mode for compiler options.
- *
- * @param option option
- * @param type type of the option
- */
- private static void addTypeDependency(JVMOption option, String type) {
- if (type.contains("C1") || type.contains("C2")) {
- /* Run in compiler mode for compiler flags */
- option.addPrepend("-Xcomp");
- }
- }
-
- /**
* Parse JVM Options. Get input from "inputReader". Parse using
* "-XX:+PrintFlagsRanges" output format.
*
@@ -214,7 +200,6 @@
token = token.substring(1, token.indexOf("}"));
if (acceptOrigin.test(token)) {
- addTypeDependency(option, token);
addNameDependency(option);
allOptions.put(name, option);