|
|
99c80f |
diff --git a/application/org.openjdk.jmc.browser.attach/src/main/java/org/openjdk/jmc/browser/attach/LocalJVMToolkit.java b/application/org.openjdk.jmc.browser.attach/src/main/java/org/openjdk/jmc/browser/attach/LocalJVMToolkit.java
|
|
|
99c80f |
--- a/application/org.openjdk.jmc.browser.attach/src/main/java/org/openjdk/jmc/browser/attach/LocalJVMToolkit.java
|
|
|
99c80f |
+++ b/application/org.openjdk.jmc.browser.attach/src/main/java/org/openjdk/jmc/browser/attach/LocalJVMToolkit.java
|
|
|
99c80f |
@@ -199,6 +199,7 @@
|
|
|
99c80f |
String address = null;
|
|
|
99c80f |
String version = null;
|
|
|
99c80f |
String jvmArgs = null;
|
|
|
99c80f |
+ String jvmName = null;
|
|
|
99c80f |
String jvmVendor = null;
|
|
|
99c80f |
|
|
|
99c80f |
try {
|
|
|
99c80f |
@@ -210,6 +211,7 @@
|
|
|
99c80f |
jvmArgs = MonitoredVmUtil.jvmArgs(mvm);
|
|
|
99c80f |
StringMonitor sm = (StringMonitor) mvm.findByName("java.property.java.vm.name"); //$NON-NLS-1$
|
|
|
99c80f |
if (sm != null) {
|
|
|
99c80f |
+ jvmName = sm.stringValue();
|
|
|
99c80f |
type = getJVMType(sm.stringValue());
|
|
|
99c80f |
}
|
|
|
99c80f |
|
|
|
99c80f |
@@ -268,7 +270,7 @@
|
|
|
99c80f |
} catch (Exception x) {
|
|
|
99c80f |
// ignore
|
|
|
99c80f |
}
|
|
|
99c80f |
- connDesc = createDescriptor(name, jvmArgs, jvmVendor, vmid, connectable, type, jvmArch, address, version, isDebug);
|
|
|
99c80f |
+ connDesc = createDescriptor(name, jvmArgs, jvmName, jvmVendor, vmid, connectable, type, jvmArch, address, version, isDebug);
|
|
|
99c80f |
return connDesc;
|
|
|
99c80f |
}
|
|
|
99c80f |
});
|
|
|
99c80f |
@@ -367,6 +369,7 @@
|
|
|
99c80f |
String javaArgs = null;
|
|
|
99c80f |
String jvmArgs = null;
|
|
|
99c80f |
String jvmVersion = null;
|
|
|
99c80f |
+ String jvmName = null;
|
|
|
99c80f |
String jvmVendor = null;
|
|
|
99c80f |
VirtualMachine vm = null;
|
|
|
99c80f |
try {
|
|
|
99c80f |
@@ -384,8 +387,8 @@
|
|
|
99c80f |
+ vmd + ": " + e.getMessage()); //$NON-NLS-1$
|
|
|
99c80f |
}
|
|
|
99c80f |
if (props != null) {
|
|
|
99c80f |
- String vmName = props.getProperty("java.vm.name"); //$NON-NLS-1$
|
|
|
99c80f |
- jvmType = getJVMType(vmName);
|
|
|
99c80f |
+ jvmName = props.getProperty("java.vm.name"); //$NON-NLS-1$
|
|
|
99c80f |
+ jvmType = getJVMType(jvmName);
|
|
|
99c80f |
version = props.getProperty("java.version"); //$NON-NLS-1$
|
|
|
99c80f |
jvmVersion = props.getProperty("java.vm.version"); //$NON-NLS-1$
|
|
|
99c80f |
jvmVendor = props.getProperty("java.vm.vendor");
|
|
|
99c80f |
@@ -406,7 +409,7 @@
|
|
|
99c80f |
}
|
|
|
99c80f |
}
|
|
|
99c80f |
if (connectable.isAttachable()) {
|
|
|
99c80f |
- connDesc = createDescriptor(javaArgs, jvmArgs, jvmVendor, Integer.parseInt(vmd.id()), connectable, jvmType, jvmArch,
|
|
|
99c80f |
+ connDesc = createDescriptor(javaArgs, jvmArgs, jvmName, jvmVendor, Integer.parseInt(vmd.id()), connectable, jvmType, jvmArch,
|
|
|
99c80f |
address, version, isDebug);
|
|
|
99c80f |
}
|
|
|
99c80f |
BrowserAttachPlugin.getPluginLogger().info("Done resolving PID " + vmd); //$NON-NLS-1$
|
|
|
99c80f |
@@ -471,10 +474,10 @@
|
|
|
99c80f |
}
|
|
|
99c80f |
|
|
|
99c80f |
private static DiscoveryEntry createDescriptor(
|
|
|
99c80f |
- String javaCommand, String jvmArgs, String jvmVendor, int pid, Connectable connectable, JVMType type,
|
|
|
99c80f |
- JVMArch arch, String address, String version, boolean isDebug) {
|
|
|
99c80f |
- JVMDescriptor jvmInfo = new JVMDescriptor(version, type, arch, javaCommand, jvmArgs, jvmVendor, pid, isDebug,
|
|
|
99c80f |
- connectable);
|
|
|
99c80f |
+ String javaCommand, String jvmArgs, String jvmName, String jvmVendor, int pid, Connectable connectable,
|
|
|
99c80f |
+ JVMType type, JVMArch arch, String address, String version, boolean isDebug) {
|
|
|
99c80f |
+ JVMDescriptor jvmInfo = new JVMDescriptor(version, type, arch, javaCommand, jvmArgs, jvmName, jvmVendor, pid,
|
|
|
99c80f |
+ isDebug, connectable);
|
|
|
99c80f |
LocalConnectionDescriptor lcd = new LocalConnectionDescriptor(pid, address, connectable == ATTACHABLE);
|
|
|
99c80f |
String guid = "Local-[PID:" + pid + ", seq:" + (SEQ_NUMBER++) + "]"; //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
|
|
|
99c80f |
IServerDescriptor sd = IServerDescriptor.create(guid, null, jvmInfo);
|
|
|
99c80f |
diff --git a/application/org.openjdk.jmc.browser.jdp/src/main/java/org/openjdk/jmc/browser/jdp/JDPDescriptorProvider.java b/application/org.openjdk.jmc.browser.jdp/src/main/java/org/openjdk/jmc/browser/jdp/JDPDescriptorProvider.java
|
|
|
99c80f |
--- a/application/org.openjdk.jmc.browser.jdp/src/main/java/org/openjdk/jmc/browser/jdp/JDPDescriptorProvider.java
|
|
|
99c80f |
+++ b/application/org.openjdk.jmc.browser.jdp/src/main/java/org/openjdk/jmc/browser/jdp/JDPDescriptorProvider.java
|
|
|
99c80f |
@@ -91,7 +91,7 @@
|
|
|
99c80f |
String pid = map.get(JMXDataKeys.KEY_PID);
|
|
|
99c80f |
// NOTE: We would like to have the JVM type, architecture and vendor included in the JDP payload. We should probably file an enhancement request on JDK for this.
|
|
|
99c80f |
JVMDescriptor jvmInfo = new JVMDescriptor(null, JVMType.UNKNOWN, JVMArch.UNKNOWN, commandLine, null,
|
|
|
99c80f |
- null, pid == null ? null : Integer.parseInt(pid), false, Connectable.MGMNT_AGENT_STARTED);
|
|
|
99c80f |
+ null, null, pid == null ? null : Integer.parseInt(pid), false, Connectable.MGMNT_AGENT_STARTED);
|
|
|
99c80f |
String path = null;
|
|
|
99c80f |
if (name == null) {
|
|
|
99c80f |
} else if (name.endsWith(PATH_SEPARATOR)) {
|
|
|
99c80f |
diff --git a/application/org.openjdk.jmc.rjmx.services.jfr/src/main/java/org/openjdk/jmc/rjmx/services/jfr/internal/FlightRecorderServiceV1.java b/application/org.openjdk.jmc.rjmx.services.jfr/src/main/java/org/openjdk/jmc/rjmx/services/jfr/internal/FlightRecorderServiceV1.java
|
|
|
99c80f |
--- a/application/org.openjdk.jmc.rjmx.services.jfr/src/main/java/org/openjdk/jmc/rjmx/services/jfr/internal/FlightRecorderServiceV1.java
|
|
|
99c80f |
+++ b/application/org.openjdk.jmc.rjmx.services.jfr/src/main/java/org/openjdk/jmc/rjmx/services/jfr/internal/FlightRecorderServiceV1.java
|
|
|
99c80f |
@@ -131,6 +131,9 @@
|
|
|
99c80f |
|
|
|
99c80f |
public FlightRecorderServiceV1(IConnectionHandle handle) throws ConnectionException, ServiceNotAvailableException {
|
|
|
99c80f |
cfs = handle.getServiceOrThrow(ICommercialFeaturesService.class);
|
|
|
99c80f |
+ if (!JVMSupportToolkit.hasFlightRecorder(handle)) {
|
|
|
99c80f |
+ throw new ServiceNotAvailableException(""); //$NON-NLS-1$
|
|
|
99c80f |
+ }
|
|
|
99c80f |
if (!isDynamicFlightRecorderSupported(handle) && isFlightRecorderDisabled(handle)) {
|
|
|
99c80f |
throw new ServiceNotAvailableException(""); //$NON-NLS-1$
|
|
|
99c80f |
}
|
|
|
99c80f |
diff --git a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/ConnectionToolkit.java b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/ConnectionToolkit.java
|
|
|
99c80f |
--- a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/ConnectionToolkit.java
|
|
|
99c80f |
+++ b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/ConnectionToolkit.java
|
|
|
99c80f |
@@ -377,24 +377,24 @@
|
|
|
99c80f |
public static boolean isOracle(IConnectionHandle handle) {
|
|
|
99c80f |
JVMDescriptor descriptor = handle.getServerDescriptor().getJvmInfo();
|
|
|
99c80f |
// This should normally not happen for discovered JVMs, but users can create custom connections
|
|
|
99c80f |
- String vendor = null;
|
|
|
99c80f |
+ String name = null;
|
|
|
99c80f |
if (descriptor != null) {
|
|
|
99c80f |
- vendor = descriptor.getJvmVendor();
|
|
|
99c80f |
+ name = descriptor.getJvmName();
|
|
|
99c80f |
} else {
|
|
|
99c80f |
// We try checking if connected
|
|
|
99c80f |
if (handle.isConnected()) {
|
|
|
99c80f |
MBeanServerConnection connection = handle.getServiceOrNull(MBeanServerConnection.class);
|
|
|
99c80f |
if (connection != null) {
|
|
|
99c80f |
try {
|
|
|
99c80f |
- vendor = getRuntimeBean(connection).getVmVendor();
|
|
|
99c80f |
+ name = getRuntimeBean(connection).getVmName();
|
|
|
99c80f |
} catch (IOException e) {
|
|
|
99c80f |
- // Worst case we classify JVM vendor wrong
|
|
|
99c80f |
+ // Worst case we classify JVM name wrong
|
|
|
99c80f |
RJMXPlugin.getDefault().getLogger().log(Level.WARNING, "Could not check if Oracle JVM", e);
|
|
|
99c80f |
}
|
|
|
99c80f |
}
|
|
|
99c80f |
}
|
|
|
99c80f |
}
|
|
|
99c80f |
- return vendor != null && vendor.contains("Oracle");
|
|
|
99c80f |
+ return name != null && (name.contains("Java HotSpot"));
|
|
|
99c80f |
}
|
|
|
99c80f |
|
|
|
99c80f |
/**
|
|
|
99c80f |
diff --git a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/JVMSupportToolkit.java b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/JVMSupportToolkit.java
|
|
|
99c80f |
--- a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/JVMSupportToolkit.java
|
|
|
99c80f |
+++ b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/JVMSupportToolkit.java
|
|
|
99c80f |
@@ -85,6 +85,27 @@
|
|
|
99c80f |
}
|
|
|
99c80f |
|
|
|
99c80f |
/**
|
|
|
99c80f |
+ * Checks if Flight Recorder is available for use
|
|
|
99c80f |
+ *
|
|
|
99c80f |
+ * @param connection
|
|
|
99c80f |
+ * @return If it is an Oracle JVM or there is a FlightRecorder VM option, then return true.
|
|
|
99c80f |
+ * Otherwise, return false. This is used for verifying JDK 8 JVMs that are not built
|
|
|
99c80f |
+ * with JFR enabled, e.g., OpenJDK 8
|
|
|
99c80f |
+ */
|
|
|
99c80f |
+ public static boolean hasFlightRecorder(IConnectionHandle connection) {
|
|
|
99c80f |
+ if (ConnectionToolkit.isOracle(connection)) {
|
|
|
99c80f |
+ return true;
|
|
|
99c80f |
+ }
|
|
|
99c80f |
+ MBeanServerConnection server = connection.getServiceOrNull(MBeanServerConnection.class);
|
|
|
99c80f |
+ try {
|
|
|
99c80f |
+ HotspotManagementToolkit.getVMOption(server, "FlightRecorder");
|
|
|
99c80f |
+ return true;
|
|
|
99c80f |
+ } catch (Exception e) { // RuntimeMBeanException thrown if FlightRecorder is not present
|
|
|
99c80f |
+ return false;
|
|
|
99c80f |
+ }
|
|
|
99c80f |
+ }
|
|
|
99c80f |
+
|
|
|
99c80f |
+ /**
|
|
|
99c80f |
* Checks if Flight Recorder is disabled.
|
|
|
99c80f |
*
|
|
|
99c80f |
* @param connection
|
|
|
99c80f |
diff --git a/application/org.openjdk.jmc.rjmx/src/main/resources/org/openjdk/jmc/rjmx/messages/internal/messages.properties b/application/org.openjdk.jmc.rjmx/src/main/resources/org/openjdk/jmc/rjmx/messages/internal/messages.properties
|
|
|
99c80f |
--- a/application/org.openjdk.jmc.rjmx/src/main/resources/org/openjdk/jmc/rjmx/messages/internal/messages.properties
|
|
|
99c80f |
+++ b/application/org.openjdk.jmc.rjmx/src/main/resources/org/openjdk/jmc/rjmx/messages/internal/messages.properties
|
|
|
99c80f |
@@ -54,7 +54,7 @@
|
|
|
99c80f |
JVMSupport_FLIGHT_RECORDER_NOT_FULLY_SUPPORTED_OLD_HOTSPOT_SHORT=Flight Recorder is not fully supported for JVMs with Java versions below 7u40
|
|
|
99c80f |
JVMSupport_FLIGHT_RECORDER_NOT_SUPPORTED_NOT_HOTSPOT=Flight Recorder is not supported for non HotSpot JVMs
|
|
|
99c80f |
JVMSupport_FLIGHT_RECORDER_NOT_SUPPORTED_NOT_HOTSPOT_SHORT=Flight Recorder is not supported for non HotSpot JVMs
|
|
|
99c80f |
-JVMSupport_FLIGHT_RECORDER_NOT_ENABLED=Flight Recorder features are not enabled. To enable this you need to use an Oracle JDK 7u4 or later JVM started with -XX:+UnlockCommercialFeatures -XX:+FlightRecorder or an OpenJDK 11+ JVM.
|
|
|
99c80f |
+JVMSupport_FLIGHT_RECORDER_NOT_ENABLED=Flight Recorder features are not enabled. To enable this you need to use an Oracle JDK 7u4 or later JVM started with -XX:+UnlockCommercialFeatures -XX:+FlightRecorder, an OpenJDK 8 JFR-enabled JVM, or an OpenJDK 11+ JVM.
|
|
|
99c80f |
JVMSupport_FLIGHT_RECORDER_NOT_ENABLED_SHORT=Flight Recorder features are not enabled
|
|
|
99c80f |
JVMSupport_FLIGHT_RECORDER_DISABLED=Flight Recorder is explicitly disabled. To enable the Flight Recorder, remove -XX:-FlightRecorder from the command line.
|
|
|
99c80f |
JVMSupport_FLIGHT_RECORDER_DISABLED_SHORT=Flight Recorder is explicitly disabled
|
|
|
99c80f |
diff --git a/application/org.openjdk.jmc.ui.common/src/main/java/org/openjdk/jmc/ui/common/jvm/JVMDescriptor.java b/application/org.openjdk.jmc.ui.common/src/main/java/org/openjdk/jmc/ui/common/jvm/JVMDescriptor.java
|
|
|
99c80f |
--- a/application/org.openjdk.jmc.ui.common/src/main/java/org/openjdk/jmc/ui/common/jvm/JVMDescriptor.java
|
|
|
99c80f |
+++ b/application/org.openjdk.jmc.ui.common/src/main/java/org/openjdk/jmc/ui/common/jvm/JVMDescriptor.java
|
|
|
99c80f |
@@ -38,6 +38,7 @@
|
|
|
99c80f |
public class JVMDescriptor {
|
|
|
99c80f |
private final String javaVersion;
|
|
|
99c80f |
private final JVMType jvmType;
|
|
|
99c80f |
+ private final String jvmName;
|
|
|
99c80f |
private final String jvmVendor;
|
|
|
99c80f |
private final JVMArch jvmArch;
|
|
|
99c80f |
private final String javaCommand;
|
|
|
99c80f |
@@ -47,13 +48,14 @@
|
|
|
99c80f |
private final Connectable connectable;
|
|
|
99c80f |
|
|
|
99c80f |
public JVMDescriptor(String javaVersion, JVMType jvmType, JVMArch jvmArch, String javaCommand, String jvmArguments,
|
|
|
99c80f |
- String jvmVendor, Integer pid, boolean debug, Connectable attachable) {
|
|
|
99c80f |
+ String jvmName, String jvmVendor, Integer pid, boolean debug, Connectable attachable) {
|
|
|
99c80f |
super();
|
|
|
99c80f |
this.javaVersion = javaVersion;
|
|
|
99c80f |
this.jvmType = jvmType;
|
|
|
99c80f |
this.jvmArch = jvmArch;
|
|
|
99c80f |
this.javaCommand = javaCommand;
|
|
|
99c80f |
this.jvmArguments = jvmArguments;
|
|
|
99c80f |
+ this.jvmName = jvmName;
|
|
|
99c80f |
this.jvmVendor = jvmVendor;
|
|
|
99c80f |
this.pid = pid;
|
|
|
99c80f |
this.debug = debug;
|
|
|
99c80f |
@@ -80,6 +82,10 @@
|
|
|
99c80f |
return jvmArguments;
|
|
|
99c80f |
}
|
|
|
99c80f |
|
|
|
99c80f |
+ public String getJvmName() {
|
|
|
99c80f |
+ return jvmName;
|
|
|
99c80f |
+ }
|
|
|
99c80f |
+
|
|
|
99c80f |
public String getJvmVendor() {
|
|
|
99c80f |
return jvmVendor;
|
|
|
99c80f |
}
|
|
|
99c80f |
diff --git a/application/tests/org.openjdk.jmc.rjmx.test/src/test/java/org/openjdk/jmc/rjmx/test/JVMSupportToolkitTest.java b/application/tests/org.openjdk.jmc.rjmx.test/src/test/java/org/openjdk/jmc/rjmx/test/JVMSupportToolkitTest.java
|
|
|
99c80f |
--- a/application/tests/org.openjdk.jmc.rjmx.test/src/test/java/org/openjdk/jmc/rjmx/test/JVMSupportToolkitTest.java
|
|
|
99c80f |
+++ b/application/tests/org.openjdk.jmc.rjmx.test/src/test/java/org/openjdk/jmc/rjmx/test/JVMSupportToolkitTest.java
|
|
|
99c80f |
@@ -47,8 +47,10 @@
|
|
|
99c80f |
|
|
|
99c80f |
@SuppressWarnings("nls")
|
|
|
99c80f |
public class JVMSupportToolkitTest {
|
|
|
99c80f |
- private static final String VENDOR_OPEN_JDK = "OpenJDK";
|
|
|
99c80f |
- private static final String VENDOR_ORACLE = "Oracle";
|
|
|
99c80f |
+ private static final String NAME_OPEN_JDK = "OpenJDK 64-Bit Server VM";
|
|
|
99c80f |
+ private static final String NAME_ORACLE = "Java HotSpot(TM) 64-Bit Server VM";
|
|
|
99c80f |
+ private static final String VENDOR_OPEN_JDK = "Oracle Corporation";
|
|
|
99c80f |
+ private static final String VENDOR_ORACLE = "Oracle Corporation";
|
|
|
99c80f |
// FIXME: Add tests for the methods that take IConnectionHandle as a parameter.
|
|
|
99c80f |
|
|
|
99c80f |
private static final String SUPPORTED_MESSAGE = null;
|
|
|
99c80f |
@@ -65,8 +67,8 @@
|
|
|
99c80f |
public void testJfr17U40HotSpotSupported() {
|
|
|
99c80f |
ServerHandle server = new ServerHandle(
|
|
|
99c80f |
new ServerDescriptor(null, null,
|
|
|
99c80f |
- new JVMDescriptor("1.7.0_40", JVMType.HOTSPOT, JVMArch.UNKNOWN, VENDOR_ORACLE, null, null, null,
|
|
|
99c80f |
- false, null)),
|
|
|
99c80f |
+ new JVMDescriptor("1.7.0_40", JVMType.HOTSPOT, JVMArch.UNKNOWN, null, null, NAME_ORACLE,
|
|
|
99c80f |
+ VENDOR_ORACLE, null, false, null)),
|
|
|
99c80f |
new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null);
|
|
|
99c80f |
String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false);
|
|
|
99c80f |
assertEquals(SUPPORTED_MESSAGE, errorMessage);
|
|
|
99c80f |
@@ -76,8 +78,8 @@
|
|
|
99c80f |
public void testJfr17U4HotSpotNotFullySupported() {
|
|
|
99c80f |
ServerHandle server = new ServerHandle(
|
|
|
99c80f |
new ServerDescriptor(null, null,
|
|
|
99c80f |
- new JVMDescriptor("1.7.0_04", JVMType.HOTSPOT, JVMArch.UNKNOWN, VENDOR_ORACLE, null, null, null,
|
|
|
99c80f |
- false, null)),
|
|
|
99c80f |
+ new JVMDescriptor("1.7.0_04", JVMType.HOTSPOT, JVMArch.UNKNOWN, null, null, NAME_ORACLE,
|
|
|
99c80f |
+ VENDOR_ORACLE, null, false, null)),
|
|
|
99c80f |
new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null);
|
|
|
99c80f |
String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false);
|
|
|
99c80f |
assertEquals(Messages.JVMSupport_FLIGHT_RECORDER_NOT_FULLY_SUPPORTED_OLD_HOTSPOT, errorMessage);
|
|
|
99c80f |
@@ -87,8 +89,8 @@
|
|
|
99c80f |
public void testJfr17HotSpotNotSupported() {
|
|
|
99c80f |
ServerHandle server = new ServerHandle(
|
|
|
99c80f |
new ServerDescriptor(null, null,
|
|
|
99c80f |
- new JVMDescriptor("1.7.0", JVMType.HOTSPOT, JVMArch.UNKNOWN, VENDOR_ORACLE, null, null, null,
|
|
|
99c80f |
- false, null)),
|
|
|
99c80f |
+ new JVMDescriptor("1.7.0", JVMType.HOTSPOT, JVMArch.UNKNOWN, null, null, NAME_ORACLE,
|
|
|
99c80f |
+ VENDOR_ORACLE, null, false, null)),
|
|
|
99c80f |
new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null);
|
|
|
99c80f |
String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false);
|
|
|
99c80f |
assertEquals(Messages.JVMSupport_FLIGHT_RECORDER_NOT_SUPPORTED_OLD_HOTSPOT, errorMessage);
|
|
|
99c80f |
@@ -98,8 +100,8 @@
|
|
|
99c80f |
public void testJfrJRockitNotSupported() {
|
|
|
99c80f |
ServerHandle server = new ServerHandle(
|
|
|
99c80f |
new ServerDescriptor(null, null,
|
|
|
99c80f |
- new JVMDescriptor("1.6", JVMType.JROCKIT, JVMArch.UNKNOWN, VENDOR_ORACLE, null, null, null,
|
|
|
99c80f |
- false, null)),
|
|
|
99c80f |
+ new JVMDescriptor("1.6", JVMType.JROCKIT, JVMArch.UNKNOWN, null, null, NAME_ORACLE,
|
|
|
99c80f |
+ VENDOR_ORACLE, null, false, null)),
|
|
|
99c80f |
new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null);
|
|
|
99c80f |
String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false);
|
|
|
99c80f |
assertEquals(Messages.JVMSupport_JROCKIT_NO_LONGER_SUPPORTED, errorMessage);
|
|
|
99c80f |
@@ -109,8 +111,8 @@
|
|
|
99c80f |
public void testJfrOldHotSpotNotSupported() {
|
|
|
99c80f |
ServerHandle server = new ServerHandle(
|
|
|
99c80f |
new ServerDescriptor(null, null,
|
|
|
99c80f |
- new JVMDescriptor("1.6", JVMType.HOTSPOT, JVMArch.UNKNOWN, VENDOR_ORACLE, null, null, null,
|
|
|
99c80f |
- false, null)),
|
|
|
99c80f |
+ new JVMDescriptor("1.6", JVMType.HOTSPOT, JVMArch.UNKNOWN, null, null, NAME_ORACLE,
|
|
|
99c80f |
+ VENDOR_ORACLE, null, false, null)),
|
|
|
99c80f |
new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null);
|
|
|
99c80f |
String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false);
|
|
|
99c80f |
assertEquals(Messages.JVMSupport_FLIGHT_RECORDER_NOT_SUPPORTED_OLD_HOTSPOT, errorMessage);
|
|
|
99c80f |
@@ -119,8 +121,9 @@
|
|
|
99c80f |
@Test
|
|
|
99c80f |
public void testJfrNonHotSpotNotSupported() {
|
|
|
99c80f |
ServerHandle server = new ServerHandle(
|
|
|
99c80f |
- new ServerDescriptor(null, null, new JVMDescriptor("1.7", JVMType.OTHER, JVMArch.UNKNOWN, VENDOR_ORACLE,
|
|
|
99c80f |
- null, null, null, false, null)),
|
|
|
99c80f |
+ new ServerDescriptor(null, null,
|
|
|
99c80f |
+ new JVMDescriptor("1.7", JVMType.OTHER, JVMArch.UNKNOWN, null, null, NAME_ORACLE, VENDOR_ORACLE,
|
|
|
99c80f |
+ null, false, null)),
|
|
|
99c80f |
new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null);
|
|
|
99c80f |
String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false);
|
|
|
99c80f |
assertEquals(Messages.JVMSupport_FLIGHT_RECORDER_NOT_SUPPORTED_NOT_HOTSPOT, errorMessage);
|
|
|
99c80f |
@@ -130,8 +133,8 @@
|
|
|
99c80f |
public void testJfrUnknownNoWarning() {
|
|
|
99c80f |
ServerHandle server = new ServerHandle(
|
|
|
99c80f |
new ServerDescriptor(null, null,
|
|
|
99c80f |
- new JVMDescriptor("1.7", JVMType.UNKNOWN, JVMArch.UNKNOWN, VENDOR_ORACLE, null, null, null,
|
|
|
99c80f |
- false, null)),
|
|
|
99c80f |
+ new JVMDescriptor("1.7", JVMType.UNKNOWN, JVMArch.UNKNOWN, null, null, NAME_ORACLE,
|
|
|
99c80f |
+ VENDOR_ORACLE, null, false, null)),
|
|
|
99c80f |
new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null);
|
|
|
99c80f |
String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false);
|
|
|
99c80f |
assertEquals(SUPPORTED_MESSAGE, errorMessage);
|
|
|
99c80f |
@@ -141,8 +144,8 @@
|
|
|
99c80f |
public void testJfr8HotSpotOpenJDKSupported() {
|
|
|
99c80f |
ServerHandle server = new ServerHandle(
|
|
|
99c80f |
new ServerDescriptor(null, null,
|
|
|
99c80f |
- new JVMDescriptor("1.8.0", JVMType.HOTSPOT, JVMArch.UNKNOWN, VENDOR_OPEN_JDK, null, null, null,
|
|
|
99c80f |
- false, null)),
|
|
|
99c80f |
+ new JVMDescriptor("1.8.0", JVMType.HOTSPOT, JVMArch.UNKNOWN, null, null, NAME_OPEN_JDK,
|
|
|
99c80f |
+ VENDOR_OPEN_JDK, null, false, null)),
|
|
|
99c80f |
new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null);
|
|
|
99c80f |
String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false);
|
|
|
99c80f |
assertEquals(SUPPORTED_MESSAGE, errorMessage);
|
|
|
99c80f |
@@ -152,8 +155,8 @@
|
|
|
99c80f |
public void testJdk7HotSpotOpenJDKNotSupported() {
|
|
|
99c80f |
ServerHandle server = new ServerHandle(
|
|
|
99c80f |
new ServerDescriptor(null, null,
|
|
|
99c80f |
- new JVMDescriptor("1.7.0", JVMType.HOTSPOT, JVMArch.UNKNOWN, VENDOR_OPEN_JDK, null, null, null,
|
|
|
99c80f |
- false, null)),
|
|
|
99c80f |
+ new JVMDescriptor("1.7.0", JVMType.HOTSPOT, JVMArch.UNKNOWN, null, null, NAME_OPEN_JDK,
|
|
|
99c80f |
+ VENDOR_OPEN_JDK, null, false, null)),
|
|
|
99c80f |
new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null);
|
|
|
99c80f |
String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false);
|
|
|
99c80f |
assertNotNull(errorMessage);
|