]> xenbits.xensource.com Git - libvirt.git/commitdiff
qemuBuildNumaCommandLine: Fix @masterInitiator check
authorMichal Privoznik <mprivozn@redhat.com>
Wed, 8 Jul 2020 13:35:08 +0000 (15:35 +0200)
committerMichal Privoznik <mprivozn@redhat.com>
Wed, 8 Jul 2020 13:35:08 +0000 (15:35 +0200)
A few commits ago, in aeecbc87b73, I've implemented command line
generation for ACPI HMAT. For this, we need to know if at least
one guest NUMA node has vCPUs. This is tracked in
@masterInitiator variable, which is initialized to -1, then we
iterate through guest NUMA nodes and break the loop if we find a
node with a vCPU. After the loop, if masterInitiator is still
negative then no NUMA node has a vCPU and we error out. But this
exact check was missing comparison for negativeness.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
src/qemu/qemu_command.c

index 7f215b4cc647d93aa84530f88c6b234c8825fe3f..f06a2f27549db102838a3ad03b5eb81cbe7fce7c 100644 (file)
@@ -7312,7 +7312,7 @@ qemuBuildNumaCommandLine(virQEMUDriverConfigPtr cfg,
         }
     }
 
-    if (masterInitiator) {
+    if (masterInitiator < 0) {
         virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
                        _("At least one NUMA node has to have CPUs"));
         goto cleanup;