← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1523083] Re: launch a vm with macvtap port is not working with kernel < 3.13

 

Reviewed:  https://review.openstack.org/255916
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=d7604169988726d121cdc9727accfeb6e29f4aed
Submitter: Jenkins
Branch:    master

commit d7604169988726d121cdc9727accfeb6e29f4aed
Author: Moshe Levi <moshele@xxxxxxxxxxxx>
Date:   Tue Dec 8 13:11:33 2015 +0200

    SR-IOV: Fix macvtap assigned vf check when kernel < 3.13
    
    when creating macvtap passthrough to SR-IOV VF in Kernel >= 3.13 an
    upper_macvtap symbolic link is created. For Kernel < 3.13 the only
    way to know it is by parsing the ip link show output and look for
    macvtap[0-9]+@<vf ifname>.
    
    This patch used the ip link show command to detects
    macvtap assigned vf so that detection of macvtap assigned vf
    will work on all kernels
    
    Closes-Bug: #1523083
    Change-Id: Icbc8d6af5c00d1453095e04dd779210dc7244c7f


** Changed in: neutron
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1523083

Title:
  launch a vm with  macvtap port is not working with kernel < 3.13

Status in neutron:
  Fix Released

Bug description:
  the sriov agent check if vf is assigned to macvtap by the exists of
  upper_macvtap symbolic

  https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/mech_sriov/agent/eswitch_manager.py#L85-L86

  The upper_macvtap symbolic exists only in kernel 3.13 and above
  see http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/drivers/net/macvlan.c?id=5831d66e8097aedfa3bc35941cf265ada2352317
  "net: create sysfs symlinks for neighbour devices".

  This is a problem when using rhel 7.1 which comes with kernel 3.10

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1523083/+subscriptions


References