Re: [Xen-devel] [libvirt] [PATCH v2 5/7] virNetDevMacVLanTapSetup: Allow enabling of IFF_MULTI_QUEUE

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

On Mon, 2015-12-14 at 12:35 +0100, Michal Privoznik wrote:
> On 14.12.2015 11:23, Ian Campbell wrote:
> > Hello,
> > 
> > On Thu, 2015-12-10 at 08:38 +0100, Michal Privoznik wrote:
> > > Like we are doing for TUN/TAP devices, we should do the same for
> > > macvtaps. Although, it's not as critical as in that case, we
> > > should do it for the consistency.
> > > 
> > > Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
> > 
> > This has triggered a build failure on amd64+i386+armhf within the Xen
> > automated test framework (which uses Debian Wheezy as the build
> > environment), I doubt it is in any way Xen specific though:
> > 
> > util/virnetdevmacvlan.c: In function 'virNetDevMacVLanTapSetup':
> > util/virnetdevmacvlan.c:338:26: error: 'IFF_MULTI_QUEUE' undeclared
> > (first use in this function)
> > util/virnetdevmacvlan.c:338:26: note: each undeclared identifier is
> > reported only once for each function it appears in
> > 
> 
> this is supposed to be fixed by:

Ah, I somehow missed that commit in the logs, sorry.

The test run in question had picked upÂafe73ed46856 which was before the
fixup, the next one will pickup the newer version and be fine.

Thanks and sorry for the noise.

Ian.

> 
> 
> commit ec93cc25ecdad100a535cb52c08f7eaa3004b960
> Author:ÂÂÂÂÂMichal Privoznik <mprivozn@xxxxxxxxxx>
> AuthorDate: Sat Dec 12 08:05:17 2015 +0100
> Commit:ÂÂÂÂÂMichal Privoznik <mprivozn@xxxxxxxxxx>
> CommitDate: Sun Dec 13 08:35:46 2015 +0100
> 
> ÂÂÂÂvirNetDevMacVLanTapSetup: Work around older systems
> ÂÂÂÂ
> ÂÂÂÂSome older systems, e.g. RHEL-6 do not have IFF_MULTI_QUEUE flag
> ÂÂÂÂwhich we use to enable multiqueue feature. Therefore one gets the
> ÂÂÂÂfollowing compile error there:
> ÂÂÂÂ
> ÂÂÂÂÂÂCCÂÂÂÂÂutil/libvirt_util_la-virnetdevmacvlan.lo
> ÂÂÂÂutil/virnetdevmacvlan.c: In function 'virNetDevMacVLanTapSetup':
> ÂÂÂÂutil/virnetdevmacvlan.c:338: error: 'IFF_MULTI_QUEUE' undeclared
> (first use in this function)
> ÂÂÂÂutil/virnetdevmacvlan.c:338: error: (Each undeclared identifier is
> reported only once
> ÂÂÂÂutil/virnetdevmacvlan.c:338: error: for each function it appears in.)
> ÂÂÂÂmake[3]: *** [util/libvirt_util_la-virnetdevmacvlan.lo] Error 1
> ÂÂÂÂ
> ÂÂÂÂSo, whenever user wants us to enable the feature on such systems,
> ÂÂÂÂwe will just throw a runtime error instead.
> ÂÂÂÂ
> ÂÂÂÂSigned-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
> 
> 
> 
> Michal

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel