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)
commita65084987b743fbeaf9a5376ea0fe99454912186
treea985959cc39ea1bdc655f546076da84d31f23ddf
parent53509ad55c16816d7a88a5695853555c7ef3572e
vswitchd: Remove port from datapath if it becomes non-operational

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