]> xenbits.xensource.com Git - libvirt.git/commitdiff
qemu_driver: Fix order of arguments in qemuDomainGetStatsCpuProc()
authorMichal Privoznik <mprivozn@redhat.com>
Tue, 6 Sep 2022 15:24:38 +0000 (17:24 +0200)
committerMichal Privoznik <mprivozn@redhat.com>
Tue, 6 Sep 2022 15:24:38 +0000 (17:24 +0200)
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>
src/qemu/qemu_driver.c

index c7cca640011d9772c59c9d0980530a8ff331d777..a3f8d8baf5c97a671d7255eb1e9558eece98e301 100644 (file)
@@ -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;