On Fri, May 02, 2014 at 11:15:46AM -0700, Alex Wang wrote: > Commit 3e912ffcbb (netdev: Add 'change_seq' back to netdev.) added per- > netdev change number for indicating status change. Future commits used > this change number to optimize the netdev status update to database. > However, the work also introduced the bug in the following scenario: > > - assume interface eth0 has address 1.2.3.4, eth1 has adddress 10.0.0.1. > - assume tunnel port p1 is set with remote_ip=10.0.0.5. > - after setup, 'ovs-vsctl list interface p1 status' should show the > 'tunnel_egress_iface="eth1"'. > - now if the address of eth1 is change to 0 via 'ifconfig eth1 0'. > - expectedly, after change, 'ovs-vsctl list interface p1 status' should > show the 'tunnel_egress_iface="eth0"' > > However, 'tunnel_egress_iface' will not be updated on current master. > This is in that, the 'netdev-vport' module corresponding to p1 does > not react to routing related changes. > > To fix the bug, this commit adds a change sequence number in the route- > table module and makes netdev-vport check the sequence number for > tunnel status update. > > Bug #1240626 > > Signed-off-by: Alex Wang <al...@nicira.com>
Assuming you tested it and it fixed the problem, Acked-by: Ben Pfaff <b...@nicira.com> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
