[libvirt] [PATCH] qemu: Improve some qemu.conf error reporting

Cole Robinson crobinso at redhat.com
Wed Jun 30 20:38:36 UTC 2010


Log some info if we can't find a config file. Make parse failures
fatal, and actually raise an error message.

Signed-off-by: Cole Robinson <crobinso at redhat.com>
---
 src/qemu/qemu_conf.c |   13 +++++++++----
 1 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/src/qemu/qemu_conf.c b/src/qemu/qemu_conf.c
index ce42bd6..e2f7b31 100644
--- a/src/qemu/qemu_conf.c
+++ b/src/qemu/qemu_conf.c
@@ -133,16 +133,21 @@ int qemudLoadDriverConfig(struct qemud_driver *driver,
     /* Just check the file is readable before opening it, otherwise
      * libvirt emits an error.
      */
-    if (access (filename, R_OK) == -1) return 0;
+    if (access (filename, R_OK) == -1) {
+        VIR_INFO("Could not read qemu config file %s", filename);
+        return 0;
+    }
 
     conf = virConfReadFile (filename, 0);
-    if (!conf) return 0;
+    if (!conf) {
+        return -1;
+    }
 
 
 #define CHECK_TYPE(name,typ) if (p && p->type != (typ)) {               \
         qemuReportError(VIR_ERR_INTERNAL_ERROR,                         \
-                         "remoteReadConfigFile: %s: %s: expected type " #typ, \
-                         filename, (name));                             \
+                        "%s: %s: %s: expected type " #typ,              \
+                        __func__, filename, (name));                    \
         virConfFree(conf);                                              \
         return -1;                                                      \
     }
-- 
1.6.6.1




More information about the libvir-list mailing list