[ovs-git] [openvswitch/ovs] c59b72: datapath: backport: openvswitch: do not ignore net...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: c59b72b590a7e4b5787d359a57bd0c2da1c752da
      https://github.com/openvswitch/ovs/commit/c59b72b590a7e4b5787d359a57bd0c2da1c752da
  Author: Pravin B Shelar <pshelar at ovn.org>
  Date:   2016-08-15 (Mon, 15 Aug 2016)

  Changed paths:
    M datapath/linux/compat/dev-openvswitch.c
    M datapath/vport-geneve.c
    M datapath/vport-gre.c
    M datapath/vport-lisp.c
    M datapath/vport-stt.c
    M datapath/vport-vxlan.c

  Log Message:
  -----------
  datapath: backport: openvswitch: do not ignore netdev errors when creating tunnel vports

Upstream commit:
    commit 4b5b9ba553f9aa5f484ab972fc9b58061885ceca
    Author: Martynas Pumputis <martynas at weave.works>
    Date:   Tue Aug 9 16:24:50 2016 +0100

    openvswitch: do not ignore netdev errors when creating tunnel vports

    The creation of a tunnel vport (geneve, gre, vxlan) brings up a
    corresponding netdev, a multi-step operation which can fail.

    For example, changing a vxlan vport's netdev state to 'up' binds the
    vport's socket to a UDP port - if the binding fails (e.g. due to the
    port being in use), the error is currently ignored giving the
    appearance that the tunnel vport creation completed successfully.

    Signed-off-by: Martynas Pumputis <martynas at weave.works>
    Acked-by: Pravin B Shelar <pshelar at ovn.org>
    Signed-off-by: David S. Miller <davem at davemloft.net>

Signed-off-by: Pravin B Shelar <pshelar at ovn.org>
Acked-by: Jesse Gross <jesse at kernel.org>