On Thu, Jul 28, 2016 at 11:26:19PM -0700, Justin Pettit wrote:
> Every IPv6-enabled interface is supposed to have a link-local address
> available to it.  This commit adds a link local interface to each router
> port and scopes link-local routes to the ingress port that received the
> packet.
> 
> Signed-off-by: Justin Pettit <jpet...@ovn.org>

In add_route() in ovn-northd.c, I don't think that it's ideal to use a
string comparison to compare an IPv6 address:
+    struct ds match = DS_EMPTY_INITIALIZER;
+    if (!strcmp("fe80::", network_s) && plen == 64) {

Acked-by: Ben Pfaff <b...@ovn.org>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to