detect libgraal using UseJVMCINativeLibrary VM option; report zero usage until libgraal MX Bean is registred

Author: thurka

visualvm/graalvm/src/org/graalvm/visualvm/graalvm/libgraal/MemoryModel.java

@@ -27,6 +27,7 @@
 import java.io.File;
 import java.io.FileInputStream;
 import java.io.FileOutputStream;
+import java.io.IOException;
 import java.io.InputStream;
 import java.io.OutputStream;
 import java.lang.management.MemoryUsage;
@@ -94,6 +95,7 @@ final class MemoryModel {
     private Jvm jvm;
     private ObjectName libgraalName;
     private MBeanServerConnection connection;
+    private boolean isAlreadyRegistered;
 
     private int chartCache = -1;
 
@@ -311,13 +313,22 @@ public void run() {
         }
     }
 
+    private boolean isLibgraalRegistered() throws IOException {
+        if (!isAlreadyRegistered) {
+            isAlreadyRegistered = connection.isRegistered(libgraalName);
+        }
+        return isAlreadyRegistered;
+    }
+
     private Object[] getAttributes(String... names) {
         try {
-            List<Attribute> attrs = connection.getAttributes(libgraalName, names).asList();
-            Object[] values = new Object[attrs.size()];
-            
-            for (int i = 0; i < values.length; i++) {
-                values[i] = attrs.get(i).getValue();
+            Object[] values = new Object[names.length];
+            if (isLibgraalRegistered()) {
+                List<Attribute> attrs = connection.getAttributes(libgraalName, names).asList();
+
+                for (int i = 0; i < values.length; i++) {
+                    values[i] = attrs.get(i).getValue();
+                }
             }
             return values;
         } catch (Exception ex) {
@@ -342,13 +353,18 @@ private void updateValues(final long time, Object[] values) {
         if (values != null) {
             CompositeData usageData = (CompositeData) values[0];
             CompositeData peakData = (CompositeData) values[1];
+
+            timestamp = time;
             if (usageData != null && peakData != null) {
                 MemoryUsage mem = MemoryUsage.from(usageData);
                 MemoryUsage peak = MemoryUsage.from(peakData);
                 heapUsed = mem.getUsed();
                 heapCapacity = peak.getUsed();
                 maxHeap = mem.getMax();
-                timestamp = time;
+            } else {
+                heapUsed = 0;
+                heapCapacity = 0;
+                maxHeap = 0;
             }
         }
     }

visualvm/graalvm/src/org/graalvm/visualvm/graalvm/libgraal/MemoryViewPluginProvider.java

@@ -24,9 +24,7 @@
  */
 package org.graalvm.visualvm.graalvm.libgraal;
 
-import java.io.IOException;
 import java.lang.management.ManagementFactory;
-import javax.management.MBeanServerConnection;
 import javax.management.MalformedObjectNameException;
 import javax.management.ObjectName;
 import org.graalvm.visualvm.application.Application;
@@ -44,6 +42,7 @@
  */
 public class MemoryViewPluginProvider extends DataSourceViewPluginProvider<Application> {
 
+    private static final String JVMCINativeLibraryFlag = "UseJVMCINativeLibrary";    // NOI18N
     static final String LIBGRAAL_HEAP = "Libgraal";  // NOI18N
 
     private final ObjectName libgraalName;
@@ -90,21 +89,11 @@ private static ObjectName getLibgraalName() {
         }
     }
 
-    private static MBeanServerConnection getConnection(Application app) {
-        JmxModel jmxModel = JmxModelFactory.getJmxModelFor(app);
-        if (jmxModel != null && jmxModel.getConnectionState() == JmxModel.ConnectionState.CONNECTED) {
-            return jmxModel.getMBeanServerConnection();
-        }
-        return null;
-    }
-
     private boolean isSupported(Application app) {
-        try {
-            MBeanServerConnection conn = getConnection(app);
-            if (conn != null) {
-                return conn.isRegistered(libgraalName);
-            }
-        } catch (IOException ex) {
+        JmxModel jmxModel = JmxModelFactory.getJmxModelFor(app);
+        if (jmxModel != null) {
+            String val = jmxModel.getFlagValue(JVMCINativeLibraryFlag);
+            return Boolean.valueOf(val);
         }
         return false;
     }