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>
Tue, 10 Apr 2012 18:37:02 +0000 (11:37 -0700)
commit613bded455dfed2755c4f734baf2f8ecb046c9ae
treecd837ad071f6ae69beeb426b0ba050cfbd4dddc1
parenta3edfc39a77445f18c69347dc51f8e7c722fa6e5
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