From da8285c44a1db7b3ee85920a08dc80a133254c49 Mon Sep 17 00:00:00 2001 From: Peter Krempa Date: Wed, 15 Jun 2016 16:07:20 +0200 Subject: [PATCH] qemu: perf: Don't set state of first event for every other event A bug in the code used the value of the first perf event as state for all the mentioned one rather than extracting individual ones. Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1346730 --- src/qemu/qemu_driver.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 28859366ee..e918de5d48 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -9750,7 +9750,7 @@ qemuDomainSetPerfEvents(virDomainPtr dom, if (def) { for (i = 0; i < nparams; i++) { virTypedParameterPtr param = ¶ms[i]; - enabled = params->value.b; + enabled = param->value.b; type = virPerfEventTypeFromString(param->field); if (!enabled && virPerfEventDisable(priv->perf, type) < 0) @@ -9769,7 +9769,7 @@ qemuDomainSetPerfEvents(virDomainPtr dom, if (persistentDef) { for (i = 0; i < nparams; i++) { virTypedParameterPtr param = ¶ms[i]; - enabled = params->value.b; + enabled = param->value.b; type = virPerfEventTypeFromString(param->field); persistentDef->perf->events[type] = enabled ?