[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH 0/3] Properly deal with multiple netdevs per PCI device when converting VF <-> PF



On 08/04/2017 05:36 AM, Laine Stump wrote:
> The commit log of Patch 1 explains the majority of the reason for
> these patches. In short, they disambiguate the multiple netdevs per
> PCI device on the SRIOV PF and VFs of a Mellanox dual port NIC *when
> converting from a VF netdev to a PF netdev and vice versa*. This
> permits us to set the vlan tag and MAC address for the correct VF
> netdev (and detect the online status of the correct PF netdev for that
> VF) when using a VF in macvtap passthrough mode. (in other words, with
> these patches in place, it is possible to use *all* the VF netdevs on
> both ports of a dual port mellanox NIC for macvtap passthrough.)
> 
> These patches *do not* solve the problem of saving/setting the mac
> address/vlan tag for both ports when using a dual port VF for PCI
> device assignment with VFIO (see my RFC email from yesterday). That is
> a much more complex problem. (These patches are a prerequisite to
> anything that might come out of that RFC though).
> 
> Laine Stump (3):
>   util: new function virNetDevGetPhysPortID()
>   util: support matching a phys_port_id in virPCIGetNetName()
>   util: match phys_port_id when converting PF-netdev to/from VF-netdev
> 
>  src/libvirt_private.syms |  1 +
>  src/util/virhostdev.c    |  2 +-
>  src/util/virnetdev.c     | 84 +++++++++++++++++++++++++++++++++++++++++++++---
>  src/util/virnetdev.h     |  5 +++
>  src/util/virpci.c        | 49 ++++++++++++++++++++++------
>  src/util/virpci.h        |  4 ++-
>  6 files changed, 129 insertions(+), 16 deletions(-)
> 

ACK

Michal


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]