vswitchd: Remove port from datapath if it becomes non-operational
authorAnsis Atteka <aatteka@nicira.com>
Sat, 7 Apr 2012 00:52:56 +0000 (17:52 -0700)
committerAnsis Atteka <aatteka@nicira.com>
Mon, 9 Apr 2012 17:19:42 +0000 (10:19 -0700)
If kernel module rejects config changes then vswitchd sets the ofport
column to -1, but does not remove the non-operational port from the
datapath. This patch fixes this problem.

ovs-vsctl add-br ovsbr
ovs-vsctl add-port ovsbr p1
ovs-vsctl add-port ovsbr p2
ovs-vsctl set Interface p1 options:remote_ip=2.1.1.1 options:key=123 type=gre
ovs-vsctl set Interface p2 options:remote_ip=1.1.1.1 options:key=123 type=gre
ovs-vsctl set Interface p2 options:remote_ip=2.1.1.1 options:key=123 type=gre
ovs-dpctl show #observe that p2 does not appear here anymore

Signed-off-by: Ansis Atteka <aatteka@nicira.com>
vswitchd/bridge.c

index 35d8723354ad2d9e3e321b6d9e4dde7653d08398..5aa5c89e3d002088446c5640324cf62400707512 100644 (file)
@@ -1208,6 +1208,11 @@ bridge_add_ofproto_ports(struct bridge *br)
                     /* We already reported a related error, don't bother
                      * duplicating it. */
                 }
+                if (!ofproto_port_query_by_name(br->ofproto, port->name,
+                                                &ofproto_port)) {
+                    ofproto_port_del(br->ofproto, ofproto_port.ofp_port);
+                    ofproto_port_destroy(&ofproto_port);
+                }
                 iface_clear_db_record(iface->cfg);
                 iface_destroy(iface);
             }