From ed557ebb162ff86891c645e5bcb70c2aa8f29b5e Mon Sep 17 00:00:00 2001 From: Ralf Schmelter Date: Thu, 23 Jul 2026 17:32:14 +0200 Subject: [PATCH] Add logging to himem args parsing test too. --- .../Vitals/TestHiMemReportArgParsing.java | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/test/hotspot/jtreg/runtime/Vitals/TestHiMemReportArgParsing.java b/test/hotspot/jtreg/runtime/Vitals/TestHiMemReportArgParsing.java index 96b3ba3a8d5a..7f993b99fffa 100644 --- a/test/hotspot/jtreg/runtime/Vitals/TestHiMemReportArgParsing.java +++ b/test/hotspot/jtreg/runtime/Vitals/TestHiMemReportArgParsing.java @@ -78,8 +78,8 @@ static void testValidNonExistingReportDir() throws IOException { File subdir = VitalsUtils.createSubTestDir("test-outputdir-1", false); VitalsUtils.fileShouldNotExist(subdir); ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder( - "-XX:+HiMemReport", "-XX:HiMemReportDir=" + subdir.getAbsolutePath(), "-Xlog:vitals", - "-Xmx64m", "-version"); + "-XX:+HiMemReport", "-XX:HiMemReportDir=" + subdir.getAbsolutePath(), + "-Xlog:vitals=trace,vitals+os=trace", "-Xmx64m", "-version"); OutputAnalyzer output = new OutputAnalyzer(pb.start()); output.reportDiagnosticSummary(); output.shouldHaveExitValue(0); @@ -97,8 +97,8 @@ static void testValidExistingReportDir() throws IOException { File subdir = VitalsUtils.createSubTestDir("test-outputdir-2", true); VitalsUtils.fileShouldExist(subdir); ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder( - "-XX:+HiMemReport", "-XX:HiMemReportDir=" + subdir.getAbsolutePath(), "-Xlog:vitals", - "-Xmx64m", "-version"); + "-XX:+HiMemReport", "-XX:HiMemReportDir=" + subdir.getAbsolutePath(), + "-Xlog:vitals=trace,vitals+os=trace", "-Xmx64m", "-version"); OutputAnalyzer output = new OutputAnalyzer(pb.start()); output.reportDiagnosticSummary(); output.shouldHaveExitValue(0); @@ -117,8 +117,8 @@ static void testInValidReportDir() throws IOException { File f = new File("/tmp/gibsnicht/gibsnicht/gibsnicht"); VitalsUtils.fileShouldNotExist(f); ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder( - "-XX:+HiMemReport", "-XX:HiMemReportDir=" + f.getAbsolutePath(), "-Xlog:vitals", - "-Xmx64m", "-version"); + "-XX:+HiMemReport", "-XX:HiMemReportDir=" + f.getAbsolutePath(), + "-Xlog:vitals=trace,vitals+os=trace", "-Xmx64m", "-version"); OutputAnalyzer output = new OutputAnalyzer(pb.start()); output.reportDiagnosticSummary(); output.shouldNotHaveExitValue(0); @@ -134,7 +134,7 @@ static void testInValidReportDir() throws IOException { */ static void testHiMemReportOn() throws IOException { ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder( - "-XX:+HiMemReport", "-Xlog:vitals", + "-XX:+HiMemReport", "-Xlog:vitals=trace,vitals+os=trace", "-Xmx64m", "-version"); OutputAnalyzer output = new OutputAnalyzer(pb.start()); output.reportDiagnosticSummary(); @@ -150,7 +150,7 @@ static void testHiMemReportOn() throws IOException { */ static void testHiMemReportOff() throws IOException { ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder( - "-XX:-HiMemReport", "-Xlog:vitals", + "-XX:-HiMemReport", "-Xlog:vitals=trace,vitals+os=trace", "-Xmx64m", "-version"); OutputAnalyzer output = new OutputAnalyzer(pb.start()); output.reportDiagnosticSummary(); @@ -163,7 +163,7 @@ static void testHiMemReportOff() throws IOException { */ static void testHiMemReportOffByDefault() throws IOException { ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder( - "-Xlog:vitals", + "-Xlog:vitals=trace,vitals+os=trace", "-Xmx64m", "-version"); OutputAnalyzer output = new OutputAnalyzer(pb.start()); output.reportDiagnosticSummary();