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

Peter Krempa pkrempa at redhat.com
Tue Dec 19 13:21:49 UTC 2017


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, after this change, the 'topologycpus'
> variable doesn't need to have function scope, we move its
> declaration to the inner scope as well.
> 
> The comments around the check are modified in order to explain
> the different QEMU versions involved.
> 
> Signed-off-by: Andrea Bolognani <abologna at redhat.com>
> ---
>  src/qemu/qemu_domain.c | 17 +++++++++++------
>  1 file changed, 11 insertions(+), 6 deletions(-)

ACK
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20171219/d0b583f9/attachment-0001.sig>


More information about the libvir-list mailing list