]> git.ipfire.org Git - thirdparty/libvirt.git/commitdiff
qemu: simplify qemuProcessSetupRawIO
authorJán Tomko <jtomko@redhat.com>
Thu, 13 Jan 2022 17:26:14 +0000 (18:26 +0100)
committerJán Tomko <jtomko@redhat.com>
Wed, 19 Jan 2022 12:57:59 +0000 (13:57 +0100)
Remove the now unused 'driver' parameter, as well as the pointless
if (ret == 0) comparison which is always true after removing the
cleanup label.

Signed-off-by: Ján Tomko <jtomko@redhat.com>
Reviewed-by: Andrea Bolognani <abologna@redhat.com>
src/qemu/qemu_process.c

index 6203efef3a99cac5679d34d9d589f9a99bab1e35..7ff4dc1835e418020f8a715fb77330e1199c063b 100644 (file)
@@ -5022,8 +5022,7 @@ qemuProcessSetupGraphics(virQEMUDriver *driver,
 
 
 static int
-qemuProcessSetupRawIO(virQEMUDriver *driver G_GNUC_UNUSED,
-                      virDomainObj *vm,
+qemuProcessSetupRawIO(virDomainObj *vm,
                       virCommand *cmd G_GNUC_UNUSED)
 {
     bool rawio = false;
@@ -5062,8 +5061,7 @@ qemuProcessSetupRawIO(virQEMUDriver *driver G_GNUC_UNUSED,
 
     if (rawio) {
 #ifdef CAP_SYS_RAWIO
-        if (ret == 0)
-            virCommandAllowCap(cmd, CAP_SYS_RAWIO);
+        virCommandAllowCap(cmd, CAP_SYS_RAWIO);
 #else
         virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
                        _("Raw I/O is not supported on this platform"));
@@ -7423,7 +7421,7 @@ qemuProcessLaunch(virConnectPtr conn,
         goto cleanup;
 
     VIR_DEBUG("Setting up raw IO");
-    if (qemuProcessSetupRawIO(driver, vm, cmd) < 0)
+    if (qemuProcessSetupRawIO(vm, cmd) < 0)
         goto cleanup;
 
     virCommandSetPreExecHook(cmd, qemuProcessHook, &hookData);