Skip to content
This repository has been archived by the owner on Feb 2, 2023. It is now read-only.
/ jdk15u-dev Public archive

Commit

Permalink
8260349: Cannot programmatically retrieve Metaspace max set via JAVA_…
Browse files Browse the repository at this point in the history
…TOOL_OPTIONS

Backport-of: b6a736738ae025604d86924298fdd04a7851b85f
  • Loading branch information
Yuri Nesterenko committed May 6, 2021
1 parent 5626db3 commit f0f89d2
Show file tree
Hide file tree
Showing 2 changed files with 123 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/hotspot/share/services/memoryPool.cpp
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2003, 2020, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2003, 2021, 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
Expand Down Expand Up @@ -206,8 +206,8 @@ size_t MetaspacePool::used_in_bytes() {
}

size_t MetaspacePool::calculate_max_size() const {
return FLAG_IS_CMDLINE(MaxMetaspaceSize) ? MaxMetaspaceSize :
MemoryUsage::undefined_size();
return !FLAG_IS_DEFAULT(MaxMetaspaceSize) ? MaxMetaspaceSize :
MemoryUsage::undefined_size();
}

CompressedKlassSpacePool::CompressedKlassSpacePool() :
Expand Down
120 changes: 120 additions & 0 deletions test/hotspot/jtreg/runtime/Metaspace/MaxMetaspaceSizeEnvVarTest.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,120 @@
/*
* Copyright (c) 2021, 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.
*/

/*
* @test
* @bug 8260349
* @summary test that setting via the env-var and options file shows up as expected
* @library /test/lib
* @run driver MaxMetaspaceSizeEnvVarTest
*/

import java.io.PrintWriter;
import java.lang.management.ManagementFactory;
import java.lang.management.MemoryPoolMXBean;

import jdk.test.lib.process.ProcessTools;
import jdk.test.lib.process.OutputAnalyzer;

public class MaxMetaspaceSizeEnvVarTest {

// This is the test class we exec, passing the MaxMetaspaceSize flag
// by different mechanisms.
static class Main {
public static void main(String[] args) throws Exception {
long expected = Long.parseLong(args[0]);
MemoryPoolMXBean metaspaceMemoryPool =
ManagementFactory.getPlatformMXBeans(MemoryPoolMXBean.class)
.stream()
.filter(pool -> "Metaspace".equals(pool.getName()))
.findFirst()
.orElseThrow();
long max = metaspaceMemoryPool.getUsage().getMax();
System.out.println("Metaspace max usage is " + max);
if (max != expected) {
throw new RuntimeException("Metaspace max " + max +
" != " + expected);
}
}
}

static void report(String msg) {
System.out.println(msg);
System.err.println(msg);
}

public static void main(String... args) throws Exception {
final String max = String.valueOf(9 * 1024 * 1024); // 9 MB
final String flagRaw = "MaxMetaspaceSize=" + max;
final String flag = "-XX:" + flagRaw;
final String main = "MaxMetaspaceSizeEnvVarTest$Main";

ProcessBuilder pb = null;
OutputAnalyzer output = null;

int test = 1;
report("Test " + test + ": flag not set");

Main.main(new String[] { "-1" }); // -1 == undefined size
report("------ end Test " + test);
test++;

report("Test " + test + ": normal command-line flag");
pb = ProcessTools.createJavaProcessBuilder(flag, main, max);
output = new OutputAnalyzer(pb.start());
output.shouldHaveExitValue(0);
output.reportDiagnosticSummary();
report("------ end Test " + test);
test++;

String[] envVars = {
"JDK_JAVA_OPTIONS",
"_JAVA_OPTIONS",
"JAVA_TOOL_OPTIONS"
};

for (String envVar : envVars) {
report("Test " + test + ": " + envVar + " env-var");
pb = ProcessTools.createJavaProcessBuilder(main, max);
pb.environment().put(envVar, flag);
output = new OutputAnalyzer(pb.start());
output.shouldHaveExitValue(0);
output.reportDiagnosticSummary();
report("------ end Test " + test);
test++;
}

report("Test " + test + ": .hotspotrc file");
final String rcFile = ".hotspotrc";
final String rcFileFlag = "-XX:Flags=" + rcFile;

PrintWriter pw = new PrintWriter(rcFile);
pw.println(flagRaw);
pw.close();
pb = ProcessTools.createJavaProcessBuilder(rcFileFlag, main, max);
output = new OutputAnalyzer(pb.start());
output.shouldHaveExitValue(0);
output.reportDiagnosticSummary();
report("------ end Test " + test);
}
}

1 comment on commit f0f89d2

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.