changeset 8665:935cacbb8fab

8190442: Backout changes for JDK-8087291 from 8u-dev as it didn't use main CR id Summary: Backed out changeset 62c8545a9b6f Reviewed-by: asaha
author poonam
date Tue, 31 Oct 2017 20:29:34 +0000
parents 62c8545a9b6f
children ac8450bdd81c
files src/share/vm/memory/metaspace.cpp test/runtime/Metaspace/MaxMetaspaceSizeTest.java
diffstat 2 files changed, 1 insertions(+), 66 deletions(-) [+]
line wrap: on
line diff
--- a/src/share/vm/memory/metaspace.cpp	Tue Oct 24 22:45:26 2017 +0900
+++ b/src/share/vm/memory/metaspace.cpp	Tue Oct 31 20:29:34 2017 +0000
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011, 2017, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2011, 2014, 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
@@ -3135,24 +3135,6 @@
 
   CompressedClassSpaceSize = align_size_down_bounded(CompressedClassSpaceSize, _reserve_alignment);
   set_compressed_class_space_size(CompressedClassSpaceSize);
-
-  // Initial virtual space size will be calculated at global_initialize()
-  uintx min_metaspace_sz =
-      VIRTUALSPACEMULTIPLIER * InitialBootClassLoaderMetaspaceSize;
-  if (UseCompressedClassPointers) {
-    if ((min_metaspace_sz + CompressedClassSpaceSize) >  MaxMetaspaceSize) {
-      if (min_metaspace_sz >= MaxMetaspaceSize) {
-        vm_exit_during_initialization("MaxMetaspaceSize is too small.");
-      } else {
-        FLAG_SET_ERGO(uintx, CompressedClassSpaceSize,
-                      MaxMetaspaceSize - min_metaspace_sz);
-      }
-    }
-  } else if (min_metaspace_sz >= MaxMetaspaceSize) {
-    FLAG_SET_ERGO(uintx, InitialBootClassLoaderMetaspaceSize,
-                  min_metaspace_sz);
-  }
-
 }
 
 void Metaspace::global_initialize() {
--- a/test/runtime/Metaspace/MaxMetaspaceSizeTest.java	Tue Oct 24 22:45:26 2017 +0900
+++ /dev/null	Thu Jan 01 00:00:00 1970 +0000
@@ -1,47 +0,0 @@
-/*
- * Copyright (c) 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
- * under the terms of the GNU General Public License version 2 only, as
- * published by the Free Software Foundation.
- *
- * This code is distributed in the hope that it will be useful, but WITHOUT
- * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
- * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
- * version 2 for more details (a copy is included in the LICENSE file that
- * accompanied this code).
- *
- * You should have received a copy of the GNU General Public License version
- * 2 along with this work; if not, write to the Free Software Foundation,
- * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
- *
- * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
- * or visit www.oracle.com if you need additional information or have any
- * questions.
- */
-
-import com.oracle.java.testlibrary.ProcessTools;
-import com.oracle.java.testlibrary.OutputAnalyzer;
-
-/*
- * @test MaxMetaspaceSizeTest
- * @requires vm.bits == "64"
- * @bug 8087291
- * @library /testlibrary
- * @run main/othervm MaxMetaspaceSizeTest
- */
-
-public class MaxMetaspaceSizeTest {
-    public static void main(String... args) throws Exception {
-        ProcessBuilder pb = ProcessTools.createJavaProcessBuilder(
-            "-Xmx1g",
-            "-XX:InitialBootClassLoaderMetaspaceSize=4195328",
-            "-XX:MaxMetaspaceSize=4195328",
-            "-XX:+UseCompressedClassPointers",
-            "-XX:CompressedClassSpaceSize=1g",
-            "-version");
-        OutputAnalyzer output = new OutputAnalyzer(pb.start());
-        output.shouldContain("MaxMetaspaceSize is too small.");
-    }
-}