qemuDomainGetStatsCpuProc: Don't fetch stats for inactive VMs

CPU stats for inactive VM make no sense. In this case it's especially
misleading because 'vm->pid' of an inactive VM is '0' so
virProcessGetStat returns stats for virtqemud itself.

Fixes: 044b8744d65f8571038f85685b3c4b241162977b
Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index d227ac58cdb4780687f54c013c6f2de8e51b5e21..529e9fe3bec1928091cd30f0215148eba7e87b15 100644 (file)
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -17156,6 +17156,9 @@ qemuDomainGetStatsCpuProc(virDomainObj *vm,
     unsigned long long userTime = 0;
     unsigned long long sysTime = 0;
 
+    if (!virDomainObjIsActive(vm))
+        return;
+
     if (virProcessGetStatInfo(&cpuTime, &userTime, &sysTime,
                               NULL, NULL, vm->pid, 0) < 0) {
         /* ignore error */