archive/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2025-09-18 18:14:38 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

8245723: Remove PrintCompressedOopsMode and change gc+heap+coops info log to debug level

Browse source 
Reviewed-by: dholmes, coleenp, tschatzl
This commit is contained in:
Stefan Johansson 2020-05-27 11:48:13 +02:00
parent d0294c3845
commit 2ff952bf32
4 changed files with 11 additions and 18 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/hotspot/share/oops/compressedOops.cpp
View file

@ -66,7 +66,7 @@ void CompressedOops::initialize(const ReservedHeapSpace& heap_space) {
_heap_address_range = heap_space.region(); _heap_address_range = heap_space.region();
LogTarget(Info, gc, heap, coops) lt; LogTarget(Debug, gc, heap, coops) lt;
if (lt.is_enabled()) { if (lt.is_enabled()) {
ResourceMark rm; ResourceMark rm;
LogStream ls(lt); LogStream ls(lt);

1
src/hotspot/share/runtime/arguments.cpp
View file

@ -602,7 +602,6 @@ static AliasedFlag const aliased_jvm_flags[] = {
// NOTE: A compatibility request will be necessary for each alias to be removed. // NOTE: A compatibility request will be necessary for each alias to be removed.
static AliasedLoggingFlag const aliased_logging_flags[] = { static AliasedLoggingFlag const aliased_logging_flags[] = {
{ "PrintCompressedOopsMode", LogLevel::Info, true, LOG_TAGS(gc, heap, coops) },
{ "PrintSharedSpaces", LogLevel::Info, true, LOG_TAGS(cds) }, { "PrintSharedSpaces", LogLevel::Info, true, LOG_TAGS(cds) },
{ "TraceBiasedLocking", LogLevel::Info, true, LOG_TAGS(biasedlocking) }, { "TraceBiasedLocking", LogLevel::Info, true, LOG_TAGS(biasedlocking) },
{ "TraceClassLoading", LogLevel::Info, true, LOG_TAGS(class, load) }, { "TraceClassLoading", LogLevel::Info, true, LOG_TAGS(class, load) },

12
test/hotspot/jtreg/runtime/CompressedOops/CompressedClassPointers.java
View file

@ -154,9 +154,9 @@ public class CompressedClassPointers {
"-XX:SharedArchiveFile=./CompressedClassPointers.jsa", "-XX:SharedArchiveFile=./CompressedClassPointers.jsa",
"-Xmx128m", "-Xmx128m",
"-XX:SharedBaseAddress=8g", "-XX:SharedBaseAddress=8g",
"-XX:+PrintCompressedOopsMode",
"-XX:+VerifyBeforeGC", "-XX:+VerifyBeforeGC",
"-Xshare:dump", "-Xlog:cds"); "-Xshare:dump",
"-Xlog:cds,gc+heap+coops=debug");
OutputAnalyzer output = new OutputAnalyzer(pb.start()); OutputAnalyzer output = new OutputAnalyzer(pb.start());
if (output.firstMatch("Shared spaces are not supported in this VM") != null) { if (output.firstMatch("Shared spaces are not supported in this VM") != null) {
return; return;
@ -170,7 +170,7 @@ public class CompressedClassPointers {
"-XX:SharedArchiveFile=./CompressedClassPointers.jsa", "-XX:SharedArchiveFile=./CompressedClassPointers.jsa",
"-Xmx128m", "-Xmx128m",
"-XX:SharedBaseAddress=8g", "-XX:SharedBaseAddress=8g",
"-XX:+PrintCompressedOopsMode", "-Xlog:gc+heap+coops=debug",
"-Xshare:on", "-Xshare:on",
"-version"); "-version");
output = new OutputAnalyzer(pb.start()); output = new OutputAnalyzer(pb.start());
@ -269,9 +269,9 @@ public class CompressedClassPointers {
"-XX:SharedArchiveFile=./CompressedClassPointers.jsa", "-XX:SharedArchiveFile=./CompressedClassPointers.jsa",
"-Xmx128m", "-Xmx128m",
"-XX:SharedBaseAddress=8g", "-XX:SharedBaseAddress=8g",
"-XX:+PrintCompressedOopsMode",
"-XX:+VerifyBeforeGC", "-XX:+VerifyBeforeGC",
"-Xshare:dump", "-Xlog:cds"); "-Xshare:dump",
"-Xlog:cds,gc+heap+coops=debug");
OutputAnalyzer output = new OutputAnalyzer(pb.start()); OutputAnalyzer output = new OutputAnalyzer(pb.start());
if (output.firstMatch("Shared spaces are not supported in this VM") != null) { if (output.firstMatch("Shared spaces are not supported in this VM") != null) {
return; return;
@ -287,7 +287,7 @@ public class CompressedClassPointers {
"-XX:SharedArchiveFile=./CompressedClassPointers.jsa", "-XX:SharedArchiveFile=./CompressedClassPointers.jsa",
"-Xmx128m", "-Xmx128m",
"-XX:SharedBaseAddress=8g", "-XX:SharedBaseAddress=8g",
"-XX:+PrintCompressedOopsMode", "-Xlog:gc+heap+coops=debug",
"-Xshare:on", "-Xshare:on",
"-version"); "-version");
output = new OutputAnalyzer(pb.start()); output = new OutputAnalyzer(pb.start());

14
test/hotspot/jtreg/runtime/logging/CompressedOopsTest.java
View file

@ -51,24 +51,18 @@ public class CompressedOopsTest {
public static void main(String[] args) throws Exception { public static void main(String[] args) throws Exception {
ProcessBuilder pb = ProcessTools.createJavaProcessBuilder("-XX:+UseCompressedOops", ProcessBuilder pb = ProcessTools.createJavaProcessBuilder("-XX:+UseCompressedOops",
"-Xlog:gc+heap+coops=info", "-Xlog:gc+heap+coops=debug",
InnerClass.class.getName()); InnerClass.class.getName());
analyzeOutputOn(pb); analyzeOutputOn(pb);
pb = ProcessTools.createJavaProcessBuilder("-XX:+UseCompressedOops", pb = ProcessTools.createJavaProcessBuilder("-XX:+UseCompressedOops",
"-XX:+PrintCompressedOopsMode", "-Xlog:gc+heap+coops",
InnerClass.class.getName());
analyzeOutputOn(pb);
pb = ProcessTools.createJavaProcessBuilder("-XX:+UseCompressedOops",
"-XX:+PrintCompressedOopsMode",
"-Xlog:gc+heap+coops=off",
InnerClass.class.getName()); InnerClass.class.getName());
// No coops logging on info level.
analyzeOutputOff(pb); analyzeOutputOff(pb);
pb = ProcessTools.createJavaProcessBuilder("-XX:+UseCompressedOops", pb = ProcessTools.createJavaProcessBuilder("-XX:+UseCompressedOops",
"-Xlog:gc+heap+coops=info", "-Xlog:gc+heap+coops=off",
"-XX:-PrintCompressedOopsMode",
InnerClass.class.getName()); InnerClass.class.getName());
analyzeOutputOff(pb); analyzeOutputOff(pb);
} }