]> xenbits.xensource.com Git - libvirt.git/commitdiff
qemu: Clean up usage of qemuDomainUpdateCurrentMemorySize
authorPeter Krempa <pkrempa@redhat.com>
Thu, 7 Feb 2019 09:46:20 +0000 (10:46 +0100)
committerPeter Krempa <pkrempa@redhat.com>
Fri, 8 Feb 2019 09:11:20 +0000 (10:11 +0100)
Remove the uneeded attribute and return value.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
src/qemu/qemu_domain.c
src/qemu/qemu_domain.h
src/qemu/qemu_driver.c

index 1769419604600b7273a3ef7b30f9e5f975c2dde6..db626d5c6f8e13184019f9b8d4b76b454e429f29 100644 (file)
@@ -10264,25 +10264,18 @@ qemuDomainMachineHasBuiltinIDE(const char *machine)
  *
  * In case when the balloon is not present for the domain, the function
  * recalculates the maximum size to reflect possible changes.
- *
- * Returns 0 on success and updates vm->def->mem.cur_balloon if necessary.
  */
-int
-qemuDomainUpdateCurrentMemorySize(virQEMUDriverPtr driver ATTRIBUTE_UNUSED,
-                                  virDomainObjPtr vm)
+void
+qemuDomainUpdateCurrentMemorySize(virDomainObjPtr vm)
 {
     /* inactive domain doesn't need size update */
     if (!virDomainObjIsActive(vm))
-        return 0;
+        return;
 
     /* if no balloning is available, the current size equals to the current
      * full memory size */
-    if (!virDomainDefHasMemballoon(vm->def)) {
+    if (!virDomainDefHasMemballoon(vm->def))
         vm->def->mem.cur_balloon = virDomainDefGetMemoryTotal(vm->def);
-        return 0;
-    }
-
-    return 0;
 }
 
 
index fe474170dc4ff3a7f80df2e27c529df68fe6eb0c..6d800a6c9da032522d03ea4fac0cfe68450b77d9 100644 (file)
@@ -851,8 +851,7 @@ bool qemuDomainMachineIsPSeries(const char *machine,
                                 const virArch arch);
 bool qemuDomainMachineHasBuiltinIDE(const char *machine);
 
-int qemuDomainUpdateCurrentMemorySize(virQEMUDriverPtr driver,
-                                      virDomainObjPtr vm);
+void qemuDomainUpdateCurrentMemorySize(virDomainObjPtr vm);
 
 unsigned long long qemuDomainGetMemLockLimitBytes(virDomainDefPtr def);
 int qemuDomainAdjustMaxMemLock(virDomainObjPtr vm);
index e1c1450488aae5e4b136dd83cb5689e5f21611a1..18222487491ecc3f5f31f63f0c6f883ea3c6118e 100644 (file)
@@ -2585,7 +2585,6 @@ qemuDomainGetInfo(virDomainPtr dom,
                   virDomainInfoPtr info)
 {
     unsigned long long maxmem;
-    virQEMUDriverPtr driver = dom->conn->privateData;
     virDomainObjPtr vm;
     int ret = -1;
 
@@ -2595,8 +2594,7 @@ qemuDomainGetInfo(virDomainPtr dom,
     if (virDomainGetInfoEnsureACL(dom->conn, vm->def) < 0)
         goto cleanup;
 
-    if (qemuDomainUpdateCurrentMemorySize(driver, vm) < 0)
-        goto cleanup;
+    qemuDomainUpdateCurrentMemorySize(vm);
 
     memset(info, 0, sizeof(*info));
 
@@ -7352,8 +7350,7 @@ static char
     if (virDomainGetXMLDescEnsureACL(dom->conn, vm->def, flags) < 0)
         goto cleanup;
 
-    if (qemuDomainUpdateCurrentMemorySize(driver, vm) < 0)
-        goto cleanup;
+    qemuDomainUpdateCurrentMemorySize(vm);
 
     if ((flags & VIR_DOMAIN_XML_MIGRATABLE))
         flags |= QEMU_DOMAIN_FORMAT_LIVE_FLAGS;