Re: [libvirt] [PATCH v2 1/2] qemu: Invert condition nesting in qemuDomainDefValidate()

2017-12-19 Thread Peter Krempa
On Fri, Dec 15, 2017 at 15:48:27 +0100, Andrea Bolognani wrote: > While at the moment we're only performing a single check that is > connected to vCPU hotplugging, we're going to introduce a second > one soon. Move the topology check underneath the capability check > to make that easier; since, aft

[libvirt] [PATCH v2 1/2] qemu: Invert condition nesting in qemuDomainDefValidate()

2017-12-15 Thread Andrea Bolognani
While at the moment we're only performing a single check that is connected to vCPU hotplugging, we're going to introduce a second one soon. Move the topology check underneath the capability check to make that easier; since, after this change, the 'topologycpus' variable doesn't need to have functio