From: Andrew Jones Date: Tue, 21 Jun 2016 16:34:04 +0000 (+0200) Subject: vl: smp_parse: fix regression X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=66f37d360b3fc91275ab6ca3de81f0d356c24b4a;p=qemu.git vl: smp_parse: fix regression Commit 0544edd88a "vl: smp_parse: cleanups" regressed any -smp config that left either cores or threads unspecified, and specified a topology supporting more cpus than the given online cpus. The correct way to calculate the missing parameter would be to use maxcpus, but it's too late to change that now. Restore the old way, which is to calculate it with the online cpus (as is still done), but then, if the result is zero, just set it to one. Signed-off-by: Andrew Jones Message-Id: <1466526844-29245-1-git-send-email-drjones@redhat.com> Signed-off-by: Paolo Bonzini --- diff --git a/vl.c b/vl.c index 64cfa968c9..0367647204 100644 --- a/vl.c +++ b/vl.c @@ -1234,8 +1234,10 @@ static void smp_parse(QemuOpts *opts) } else if (cores == 0) { threads = threads > 0 ? threads : 1; cores = cpus / (sockets * threads); + cores = cores > 0 ? cores : 1; } else if (threads == 0) { threads = cpus / (cores * sockets); + threads = threads > 0 ? threads : 1; } else if (sockets * cores * threads < cpus) { error_report("cpu topology: " "sockets (%u) * cores (%u) * threads (%u) < "