Blame SOURCES/12-6692.patch

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);