From: Michal Privoznik <mprivozn@redhat.com>
Date: Tue, 6 Sep 2022 15:24:38 +0000 (+0200)
Subject: qemu_driver: Fix order of arguments in qemuDomainGetStatsCpuProc()
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=e4f577b25ef30d4bde661dd6935559486167681a;p=libvirt.git

qemu_driver: Fix order of arguments in qemuDomainGetStatsCpuProc()

Just before pushing my earlier commit I've switch order of two
arguments of virProcessGetStatInfo() (as suggested in review).
However, I forgot to swap the arguments in
qemuDomainGetStatsCpuProc() which leads to userTime and sysTime
being swapped.

Fixes: 044b8744d65f8571038f85685b3c4b241162977b
Reported-by: Daniel P. Berrangé <berrange@redhat.com>
Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
---

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index c7cca64001..a3f8d8baf5 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -17771,10 +17771,10 @@ qemuDomainGetStatsCpuProc(virDomainObj *vm,
                           virTypedParamList *params)
 {
     unsigned long long cpuTime = 0;
-    unsigned long long sysTime = 0;
     unsigned long long userTime = 0;
+    unsigned long long sysTime = 0;
 
-    if (virProcessGetStatInfo(&cpuTime, &sysTime, &userTime,
+    if (virProcessGetStatInfo(&cpuTime, &userTime, &sysTime,
                               NULL, NULL, vm->pid, 0) < 0) {
         /* ignore error */
         return 0;