qga: guest-get-memory-blocks shouldn't fail for unexposed memory blocks
authorMichael Roth <mdroth@linux.vnet.ibm.com>
Mon, 19 Oct 2015 21:38:25 +0000 (16:38 -0500)
committerMichael Roth <mdroth@linux.vnet.ibm.com>
Mon, 19 Oct 2015 23:28:07 +0000 (18:28 -0500)
Some guests don't expose memory blocks via sysfs at all. This
shouldn't be a failure, instead just return an empty list. For
other access failures we still report an error.

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
qga/commands-posix.c

index a932809..ff0ba62 100644 (file)
@@ -2213,8 +2213,14 @@ GuestMemoryBlockList *qmp_guest_get_memory_blocks(Error **errp)
 
     dp = opendir("/sys/devices/system/memory/");
     if (!dp) {
-        error_setg_errno(errp, errno, "Can't open directory"
-                         "\"/sys/devices/system/memory/\"\n");
+        /* it's ok if this happens to be a system that doesn't expose
+         * memory blocks via sysfs, but otherwise we should report
+         * an error
+         */
+        if (errno != ENOENT) {
+            error_setg_errno(errp, errno, "Can't open directory"
+                             "\"/sys/devices/system/memory/\"\n");
+        }
         return NULL;
     }