]> git.proxmox.com Git - ovs.git/commitdiff
ofproto-dpif: Check for EBUSY as well
authorGreg Rose <gvrose8192@gmail.com>
Fri, 24 Aug 2018 20:36:29 +0000 (13:36 -0700)
committerBen Pfaff <blp@ovn.org>
Mon, 27 Aug 2018 15:58:39 +0000 (08:58 -0700)
Guru reported that we can't create more than one geneve tunnel.
Sometimes a driver will return EBUSY as well as EEXIST for some
duplicate configurations.  Check for EBUSY too.

Fixes: 7521e0cf9e ("ofproto-dpif: Let the dpif report when a ...")
Reported-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2018-August/047214.html
Reported-by: Guru Shetty <guru@ovn.org>
Signed-off-by: Greg Rose <gvrose8192@gmail.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
ofproto/ofproto-dpif.c

index e3abda571d3182c6ec53dbd384a10c8ff0b3267c..0a0c69a7aea019a06e74f8ad04b0170f4aab6443 100644 (file)
@@ -3687,7 +3687,7 @@ port_add(struct ofproto *ofproto_, struct netdev *netdev)
 
     odp_port_t port_no = ODPP_NONE;
     int error = dpif_port_add(ofproto->backer->dpif, netdev, &port_no);
-    if (error != EEXIST) {
+    if (error != EEXIST && error != EBUSY) {
         if (error) {
             return error;
         }