[libvirt] [PATCH] qemu: Align virQEMUCapsFlags members

Andrea Bolognani abologna at redhat.com
Wed Oct 7 08:25:57 UTC 2015


On Mon, 2015-10-05 at 17:02 +0200, Jiri Denemark wrote:
> > Would removing the values altogether cause any trouble with
> > the following values, that would have to remain defined
> > explicitly?
> > 
> >   QEMU_CAPS_NET_NAME     = QEMU_CAPS_0_10
> >   QEMU_CAPS_HOST_NET_ADD = QEMU_CAPS_0_10
> 
> Oh, that's a real mess. Normally, if some caps are implied by another
> one, we handle that in the corresponding probing function. But in
> this
> case these are not separate capabilities, we still have just one
> capability, but there are three symbolic names leading to it. I think
> the simplest solution of this mess is just
> 
>     #define QEMU_CAPS_NET_NAME QEMU_CAPS_0_10
>     #define QEMU_CAPS_HOST_NET_ADD QEMU_CAPS_0_10
> 
> and only keep QEMU_CAPS_0_10 in the enum.

I did just that and sent a v2 to the list.

Cheers.

-- 
Andrea Bolognani
Software Engineer - Virtualization Team




More information about the libvir-list mailing list