[libvirt] [PATCH] qemu: Don't fail without emulatorpin or cpumask

Eric Blake eblake at redhat.com
Thu Oct 18 22:35:31 UTC 2012


On 10/18/2012 04:32 PM, Guido Günther wrote:
> This unbreaks qemu:///session that got broken by
> 
> ba63d8f7d843461f77a8206c1ef9da38388713e5.
> ---
>  src/qemu/qemu_process.c |    4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
> index e08ec67..293ffe9 100644
> --- a/src/qemu/qemu_process.c
> +++ b/src/qemu/qemu_process.c
> @@ -2041,8 +2041,10 @@ qemuProcessSetEmulatorAffinites(virConnectPtr conn,
>          cpumask = def->cputune.emulatorpin->cpumask;
>      else if (def->cpumask)
>          cpumask = def->cpumask;
> -    else
> +    else {

Almost.  Our coding style says that if you have {} on one side of
'else', then you need it on the other side as well.  ACK if you update
the patch to add {} to the rest of the if/else chain.

-- 
Eric Blake   eblake at redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 617 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20121018/6a5c7bc7/attachment-0001.sig>


More information about the libvir-list mailing list