[PATCH v3 06/13] libxl: turn libxlMakeDomainOSCaps() to void

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

 



libxlMakeDomainOSCaps() always returns 0.

Signed-off-by: Dmitry Frolov <frolov@xxxxxxxxx>
---
 src/libxl/libxl_capabilities.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/libxl/libxl_capabilities.c b/src/libxl/libxl_capabilities.c
index 8853ba9d88..defbf9d223 100644
--- a/src/libxl/libxl_capabilities.c
+++ b/src/libxl/libxl_capabilities.c
@@ -508,7 +508,7 @@ libxlCapsInitGuests(libxl_ctx *ctx, virCaps *caps)
     return 0;
 }
 
-static int
+static void
 libxlMakeDomainOSCaps(const char *machine,
                       virDomainCapsOS *os,
                       virFirmware **firmwares,
@@ -523,7 +523,7 @@ libxlMakeDomainOSCaps(const char *machine,
     capsLoader->readonly.report = true;
 
     if (STREQ(machine, "xenpv") || STREQ(machine, "xenpvh"))
-        return 0;
+        return;
 
     capsLoader->supported = VIR_TRISTATE_BOOL_YES;
     capsLoader->values.values = g_new0(char *, nfirmwares);
@@ -539,7 +539,7 @@ libxlMakeDomainOSCaps(const char *machine,
     VIR_DOMAIN_CAPS_ENUM_SET(capsLoader->readonly,
                              VIR_TRISTATE_BOOL_YES);
 
-    return 0;
+    return;
 }
 
 static int
@@ -675,8 +675,8 @@ libxlMakeDomainCapabilities(virDomainCaps *domCaps,
     else
         domCaps->maxvcpus = PV_MAX_VCPUS;
 
-    if (libxlMakeDomainOSCaps(domCaps->machine, os, firmwares, nfirmwares) < 0 ||
-        libxlMakeDomainDeviceDiskCaps(disk) < 0 ||
+    libxlMakeDomainOSCaps(domCaps->machine, os, firmwares, nfirmwares);
+    if (libxlMakeDomainDeviceDiskCaps(disk) < 0 ||
         libxlMakeDomainDeviceGraphicsCaps(graphics) < 0 ||
         libxlMakeDomainDeviceVideoCaps(video) < 0)
         return -1;
-- 
2.34.1
_______________________________________________
Devel mailing list -- devel@xxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxx




[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]

  Powered by Linux