From: Jiri Denemark <jdenemar@redhat.com>
Date: Tue, 27 Mar 2018 21:41:10 +0000 (+0200)
Subject: qemu: Drop qemuMigrationParamsSetCapability
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=0809f1bac436165560311a442bf5884162be9695;p=libvirt.git

qemu: Drop qemuMigrationParamsSetCapability

It's become only a tiny wrapper around virBitmapSetBit, which can easily
be called directly. We don't need to call virBitmapClearBit since
migParams->caps bitmap is initialized with zeros.

Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
---

diff --git a/src/qemu/qemu_migration_params.c b/src/qemu/qemu_migration_params.c
index e0cbdb1a38..262825972c 100644
--- a/src/qemu/qemu_migration_params.c
+++ b/src/qemu/qemu_migration_params.c
@@ -229,21 +229,6 @@ qemuMigrationParamsApply(virQEMUDriverPtr driver,
 }
 
 
-static int
-qemuMigrationParamsSetCapability(virDomainObjPtr vm ATTRIBUTE_UNUSED,
-                                 qemuMonitorMigrationCaps capability,
-                                 bool state,
-                                 qemuMigrationParamsPtr migParams)
-{
-    if (state)
-        ignore_value(virBitmapSetBit(migParams->caps, capability));
-    else
-        ignore_value(virBitmapClearBit(migParams->caps, capability));
-
-    return 0;
-}
-
-
 /* qemuMigrationParamsEnableTLS
  * @driver: pointer to qemu driver
  * @vm: domain object
@@ -358,23 +343,17 @@ qemuMigrationParamsDisableTLS(virDomainObjPtr vm,
 
 
 int
-qemuMigrationParamsSetCompression(virDomainObjPtr vm,
+qemuMigrationParamsSetCompression(virDomainObjPtr vm ATTRIBUTE_UNUSED,
                                   qemuMigrationCompressionPtr compression,
                                   qemuMigrationParamsPtr migParams)
 {
-    if (qemuMigrationParamsSetCapability(vm,
-                                         QEMU_MONITOR_MIGRATION_CAPS_XBZRLE,
-                                         compression->methods &
-                                         (1ULL << QEMU_MIGRATION_COMPRESS_XBZRLE),
-                                         migParams) < 0)
-        return -1;
+    if (compression->methods & (1ULL << QEMU_MIGRATION_COMPRESS_XBZRLE))
+        ignore_value(virBitmapSetBit(migParams->caps,
+                                     QEMU_MONITOR_MIGRATION_CAPS_XBZRLE));
 
-    if (qemuMigrationParamsSetCapability(vm,
-                                         QEMU_MONITOR_MIGRATION_CAPS_COMPRESS,
-                                         compression->methods &
-                                         (1ULL << QEMU_MIGRATION_COMPRESS_MT),
-                                         migParams) < 0)
-        return -1;
+    if (compression->methods & (1ULL << QEMU_MIGRATION_COMPRESS_MT))
+        ignore_value(virBitmapSetBit(migParams->caps,
+                                     QEMU_MONITOR_MIGRATION_CAPS_COMPRESS));
 
     migParams->params.compressLevel_set = compression->level_set;
     migParams->params.compressLevel = compression->level;