On Sun, Jul 24, 2016 at 01:07:27PM -0300, Thadeu Lima de Souza Cascardo wrote:
> When the route table is reset, the addresses list may be out of date, as we 
> race
> for the many netlink socket notifications.
> 
> A quick fix for this is flushing the addresses list, before dumping the routes
> and gathering source addresses for them.
> 
> That way, instead of using invalid source addresses or preventing an entry to 
> be
> added because of missing source addresses, repeated tests showed the correct
> entry is always added.
> 
> Signed-off-by: Thadeu Lima de Souza Cascardo <casca...@redhat.com>

Forgot to add...

Fixes: a8704b502785 ("tunneling: Handle multiple ip address for given device.")

> ---
>  lib/route-table.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/lib/route-table.c b/lib/route-table.c
> index 58e7f62..21df602 100644
> --- a/lib/route-table.c
> +++ b/lib/route-table.c
> @@ -151,6 +151,7 @@ route_table_reset(void)
>      struct ofpbuf request, reply, buf;
>  
>      route_map_clear();
> +    netdev_get_addrs_list_flush();
>      route_table_valid = true;
>      rt_change_seq++;
>  
> -- 
> 2.7.4
> 
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to