8213448: [TESTBUG] enhance jfr/jvm/TestDumpOnCrash

Added 2 more crash scenarios, removed dumponexit, and more

Reviewed-by: egahlin
This commit is contained in:
Mikhailo Seledtsov 2019-03-07 12:15:48 -08:00
parent 939d40e1a8
commit f391d9141b

View File

@ -1,5 +1,5 @@
/*
* Copyright (c) 2016, 2018, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2016, 2019, 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
@ -24,6 +24,7 @@
*/
package jdk.jfr.jvm;
import java.io.IOException;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;
@ -54,48 +55,66 @@ public class TestDumpOnCrash {
private static final CharSequence LOG_FILE_EXTENSION = ".log";
private static final CharSequence JFR_FILE_EXTENSION = ".jfr";
static class Crasher {
static class CrasherIllegalAccess {
public static void main(String[] args) {
Unsafe.getUnsafe().putInt(0L, 0);
}
}
public static void main(String[] args) throws Exception {
processOutput(runProcess());
static class CrasherHalt {
public static void main(String[] args) {
System.out.println("Running Runtime.getRuntime.halt");
Runtime.getRuntime().halt(17);
}
}
private static OutputAnalyzer runProcess() throws Exception {
return new OutputAnalyzer(
ProcessTools.createJavaProcessBuilder(true,
static class CrasherSig {
public static void main(String[] args) throws Exception {
String signalName = args[0];
System.out.println("Sending SIG" + signalName + " to process " + ProcessHandle.current().pid());
Runtime.getRuntime().exec("kill -" + signalName + " " + ProcessHandle.current().pid()).waitFor();
}
}
public static void main(String[] args) throws Exception {
verify(runProcess(CrasherIllegalAccess.class.getName(), ""));
verify(runProcess(CrasherHalt.class.getName(), ""));
// Verification is excluded for the test case below until 8219680 is fixed
long pid = runProcess(CrasherSig.class.getName(), "FPE");
// @ignore 8219680
// verify(pid);
}
private static long runProcess(String crasher, String signal) throws Exception {
System.out.println("Test case for crasher " + crasher);
Process p = ProcessTools.createJavaProcessBuilder(true,
"-Xmx64m",
"-Xint",
"-XX:-CreateCoredumpOnCrash",
"--add-exports=java.base/jdk.internal.misc=ALL-UNNAMED",
"-XX:StartFlightRecording=dumponexit=true",
Crasher.class.getName()).start());
"-XX:StartFlightRecording",
crasher,
signal)
.start();
OutputAnalyzer output = new OutputAnalyzer(p);
System.out.println("========== Crasher process output:");
System.out.println(output.getOutput());
System.out.println("==================================");
return p.pid();
}
private static void processOutput(OutputAnalyzer output) throws Exception {
output.shouldContain("CreateCoredumpOnCrash turned off, no core file dumped");
private static void verify(long pid) throws IOException {
String fileName = "hs_err_pid" + pid + ".jfr";
Path file = Paths.get(fileName).toAbsolutePath().normalize();
final Path jfrEmergencyFilePath = getHsErrJfrPath(output);
Asserts.assertTrue(Files.exists(jfrEmergencyFilePath), "No emergency jfr recording file " + jfrEmergencyFilePath + " exists");
Asserts.assertNotEquals(Files.size(jfrEmergencyFilePath), 0L, "File length 0. Should at least be some bytes");
System.out.printf("File size=%d%n", Files.size(jfrEmergencyFilePath));
Asserts.assertTrue(Files.exists(file), "No emergency jfr recording file " + file + " exists");
Asserts.assertNotEquals(Files.size(file), 0L, "File length 0. Should at least be some bytes");
System.out.printf("File size=%d%n", Files.size(file));
List<RecordedEvent> events = RecordingFile.readAllEvents(jfrEmergencyFilePath);
List<RecordedEvent> events = RecordingFile.readAllEvents(file);
Asserts.assertFalse(events.isEmpty(), "No event found");
System.out.printf("Found event %s%n", events.get(0).getEventType().getName());
}
private static Path getHsErrJfrPath(OutputAnalyzer output) throws Exception {
// extract to find hs-err_pid log file location
final String hs_err_pid_log_file = output.firstMatch("# *(\\S*hs_err_pid\\d+\\.log)", 1);
if (hs_err_pid_log_file == null) {
throw new RuntimeException("Did not find hs_err_pid.log file in output.\n");
}
// the dumped out jfr file should have the same name and location but with a .jfr extension
final String hs_err_pid_jfr_file = hs_err_pid_log_file.replace(LOG_FILE_EXTENSION, JFR_FILE_EXTENSION);
return Paths.get(hs_err_pid_jfr_file);
}
}