[PATCH 1/2] qemu: Change return type of qemuMonitorGetGuestCPU()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



To allow returning more granullar errors, change the error type to an
integer.
---
 src/qemu/qemu_monitor.c      | 19 ++++++++++++-------
 src/qemu/qemu_monitor.h      |  5 +++--
 src/qemu/qemu_monitor_json.c | 26 ++++++++++++++++----------
 src/qemu/qemu_monitor_json.h |  4 +++-
 src/qemu/qemu_process.c      |  9 +++++++--
 tests/qemumonitorjsontest.c  |  5 +++--
 6 files changed, 44 insertions(+), 24 deletions(-)

diff --git a/src/qemu/qemu_monitor.c b/src/qemu/qemu_monitor.c
index 14e4e2d..30315b3 100644
--- a/src/qemu/qemu_monitor.c
+++ b/src/qemu/qemu_monitor.c
@@ -3933,28 +3933,33 @@ qemuMonitorSetDomainLog(qemuMonitorPtr mon, int logfd)
  * qemuMonitorJSONGetGuestCPU:
  * @mon: Pointer to the monitor
  * @arch: arch of the guest
+ * @data: returns the cpu data
  *
  * Retrieve the definition of the guest CPU from a running qemu instance.
  *
- * Returns the cpu definition object. On error returns NULL.
+ * Returns 0 on success, -2 if the operation is not supported by the guest,
+ * -1 on other errors.
  */
-virCPUDataPtr
+int
 qemuMonitorGetGuestCPU(qemuMonitorPtr mon,
-                       virArch arch)
+                       virArch arch,
+                       virCPUDataPtr *data)
 {
-    VIR_DEBUG("mon=%p, arch='%s'", mon, virArchToString(arch));
+    VIR_DEBUG("mon=%p, arch='%s' data='%p'", mon, virArchToString(arch), data);

     if (!mon) {
         virReportError(VIR_ERR_INVALID_ARG, "%s",
                        _("monitor must not be NULL"));
-        return NULL;
+        return -1;
     }

     if (!mon->json) {
         virReportError(VIR_ERR_OPERATION_UNSUPPORTED, "%s",
                        _("JSON monitor is required"));
-        return NULL;
+        return -1;
     }

-    return qemuMonitorJSONGetGuestCPU(mon, arch);
+    *data = NULL;
+
+    return qemuMonitorJSONGetGuestCPU(mon, arch, data);
 }
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monitor.h
index ecc6d7b..f893b1f 100644
--- a/src/qemu/qemu_monitor.h
+++ b/src/qemu/qemu_monitor.h
@@ -764,8 +764,9 @@ int qemuMonitorGetDeviceAliases(qemuMonitorPtr mon,

 int qemuMonitorSetDomainLog(qemuMonitorPtr mon, int logfd);

-virCPUDataPtr qemuMonitorGetGuestCPU(qemuMonitorPtr mon,
-                                     virArch arch);
+int qemuMonitorGetGuestCPU(qemuMonitorPtr mon,
+                           virArch arch,
+                           virCPUDataPtr *data);

 /**
  * When running two dd process and using <> redirection, we need a
diff --git a/src/qemu/qemu_monitor_json.c b/src/qemu/qemu_monitor_json.c
index 593c90f..87990e2 100644
--- a/src/qemu/qemu_monitor_json.c
+++ b/src/qemu/qemu_monitor_json.c
@@ -5503,9 +5503,10 @@ qemuMonitorJSONParseCPUx86FeatureWord(virJSONValuePtr data,
 }


-static virCPUDataPtr
+static int
 qemuMonitorJSONGetCPUx86Data(qemuMonitorPtr mon,
-                             const char *property)
+                             const char *property,
+                             virCPUDataPtr *cpudata)
 {
     virJSONValuePtr cmd;
     virJSONValuePtr reply = NULL;
@@ -5513,14 +5514,14 @@ qemuMonitorJSONGetCPUx86Data(qemuMonitorPtr mon,
     virCPUx86Data *x86Data = NULL;
     virCPUx86CPUID cpuid;
     size_t i;
-    virCPUDataPtr ret = NULL;
     int n;
+    int ret = -1;

     if (!(cmd = qemuMonitorJSONMakeCommand("qom-get",
                                            "s:path", QOM_CPU_PATH,
                                            "s:property", property,
                                            NULL)))
-        return NULL;
+        return -1;

     if (qemuMonitorJSONCommand(mon, cmd, &reply) < 0)
         goto cleanup;
@@ -5551,9 +5552,11 @@ qemuMonitorJSONGetCPUx86Data(qemuMonitorPtr mon,
             goto cleanup;
     }

-    if (!(ret = virCPUx86MakeData(VIR_ARCH_X86_64, &x86Data)))
+    if (!(*cpudata = virCPUx86MakeData(VIR_ARCH_X86_64, &x86Data)))
         goto cleanup;

+    ret = 0;
+
 cleanup:
     virJSONValueFree(cmd);
     virJSONValueFree(reply);
@@ -5566,24 +5569,27 @@ cleanup:
  * qemuMonitorJSONGetGuestCPU:
  * @mon: Pointer to the monitor
  * @arch: arch of the guest
+ * @data: returns the cpu data of the guest
  *
  * Retrieve the definition of the guest CPU from a running qemu instance.
  *
- * Returns the cpu definition object. On error returns NULL.
+ * Returns 0 on success, -2 if guest doesn't support this feature,
+ * -1 on other errors.
  */
-virCPUDataPtr
+int
 qemuMonitorJSONGetGuestCPU(qemuMonitorPtr mon,
-                           virArch arch)
+                           virArch arch,
+                           virCPUDataPtr *data)
 {
     switch (arch) {
     case VIR_ARCH_X86_64:
     case VIR_ARCH_I686:
-        return qemuMonitorJSONGetCPUx86Data(mon, "feature-words");
+        return qemuMonitorJSONGetCPUx86Data(mon, "feature-words", data);

     default:
         virReportError(VIR_ERR_INTERNAL_ERROR,
                        _("CPU definition retrieval isn't supported for '%s'"),
                        virArchToString(arch));
-        return NULL;
+        return -1;
     }
 }
diff --git a/src/qemu/qemu_monitor_json.h b/src/qemu/qemu_monitor_json.h
index 9e0a0c9..a93c51e 100644
--- a/src/qemu/qemu_monitor_json.h
+++ b/src/qemu/qemu_monitor_json.h
@@ -427,5 +427,7 @@ int qemuMonitorJSONDetachCharDev(qemuMonitorPtr mon,
 int qemuMonitorJSONGetDeviceAliases(qemuMonitorPtr mon,
                                     char ***aliases);

-virCPUDataPtr qemuMonitorJSONGetGuestCPU(qemuMonitorPtr mon, virArch arch);
+int qemuMonitorJSONGetGuestCPU(qemuMonitorPtr mon,
+                               virArch arch,
+                               virCPUDataPtr *data);
 #endif /* QEMU_MONITOR_JSON_H */
diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index fcceedd..e34f542 100644
--- a/src/qemu/qemu_process.c
+++ b/src/qemu/qemu_process.c
@@ -3472,17 +3472,22 @@ qemuProcessVerifyGuestCPU(virQEMUDriverPtr driver, virDomainObjPtr vm)
     virArch arch = def->os.arch;
     virCPUDataPtr guestcpu = NULL;
     qemuDomainObjPrivatePtr priv = vm->privateData;
+    int rc;
     bool ret = false;

     switch (arch) {
     case VIR_ARCH_I686:
     case VIR_ARCH_X86_64:
         qemuDomainObjEnterMonitor(driver, vm);
-        guestcpu = qemuMonitorGetGuestCPU(priv->mon, arch);
+        rc = qemuMonitorGetGuestCPU(priv->mon, arch, &guestcpu);
         qemuDomainObjExitMonitor(driver, vm);

-        if (!(guestcpu))
+        if (rc < 0) {
+            if (rc == -2)
+                break;
+
             goto cleanup;
+        }

         if (def->features[VIR_DOMAIN_FEATURE_PVSPINLOCK] == VIR_DOMAIN_FEATURE_STATE_ON) {
             if (!cpuHasFeature(guestcpu, VIR_CPU_x86_KVM_PV_UNHALT)) {
diff --git a/tests/qemumonitorjsontest.c b/tests/qemumonitorjsontest.c
index a6bd346..5636d95 100644
--- a/tests/qemumonitorjsontest.c
+++ b/tests/qemumonitorjsontest.c
@@ -1997,8 +1997,9 @@ testQemuMonitorJSONGetCPUData(const void *opaque)
     if (qemuMonitorTestAddItem(test, "qom-get", jsonStr) < 0)
         goto cleanup;

-    if (!(cpuData = qemuMonitorJSONGetGuestCPU(qemuMonitorTestGetMonitor(test),
-                                               VIR_ARCH_X86_64)))
+    if (qemuMonitorJSONGetGuestCPU(qemuMonitorTestGetMonitor(test),
+                                   VIR_ARCH_X86_64,
+                                   &cpuData) < 0)
         goto cleanup;

     if (!(actual = cpuDataFormat(cpuData)))
-- 
1.8.4.2

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]