[libvirt] [libvirt PATCH] Port-profile ID support using IFLA_VF_PORT_PROFILE netlink msg

[Scott Feldman]

On 5/10/10 2:26 AM, "Gerhard Stenzel" <gstenzel at linux.vnet.ibm.com> wrote:

> I tried the latest libvirt (from git) to compile with this patch:
> 
> util/macvtap.c:618: warning: 'struct ifla_vf_port_profile' declared
> inside parameter list
> util/macvtap.c:618: warning: its scope is only this definition or
> declaration, which is probably not what you want
> util/macvtap.c: In function 'sendPortProfileMulticastMsg':
> util/macvtap.c:648: error: 'IFLA_VF_PORT_PROFILE' undeclared (first use
> in this function)
> util/macvtap.c:648: error: (Each undeclared identifier is reported only
> once
> util/macvtap.c:648: error: for each function it appears in.)
> util/macvtap.c:649: error: dereferencing pointer to incomplete type
> util/macvtap.c: In function 'unsetPortProfileId':
> util/macvtap.c:709: error: storage size of 'ivp' isn't known
> util/macvtap.c:711: error: invalid application of 'sizeof' to incomplete
> type 'struct ifla_vf_port_profile'
> util/macvtap.c:709: warning: unused variable 'ivp' [-Wunused-variable]
> util/macvtap.c: In function 'setPortProfileId':
> util/macvtap.c:733: error: storage size of 'ivp' isn't known
> util/macvtap.c:734: error: 'IFLA_VF_UUID_MAX' undeclared (first use in
> this function)
> util/macvtap.c:739: error: invalid application of 'sizeof' to incomplete
> type 'struct ifla_vf_port_profile'
> util/macvtap.c:734: warning: unused variable
> 'host_uuid' [-Wunused-variable]
> util/macvtap.c:733: warning: unused variable 'ivp' [-Wunused-variable]
> make[3]: *** [libvirt_util_la-macvtap.lo] Error 1
> 
> As of 2.6.34-rc7, 'IFLA_VF_PORT_PROFILE' is not yet in the kernel. Any
> estimate when this might change?

Correct, IFLA_VF_PORT_PROFILE is not in the kernel yet.  The kernel patch is
be discussed on the kernel netdev mailing list.  The most recent discussion
is on ways to merge VDP and CDCP into what I've posted for
IFLA_VF_PORT_PROFILE.

I'll send out another libvirt patch once the kernel discussions are final.

-scott