On Thu, Feb 14, 2013 at 6:37 AM, Kyle Mestery <kmest...@cisco.com> wrote:
> In tnl_set_config(), when determining if a tunnel port
> already exists, make sure to also check the destination port. For VXLAN, this
> can be different and allows multiple VXLAN ports in the datapath to be
> created.
>
> Signed-off-by: Kyle Mestery <kmest...@cisco.com>

This seems problematic to me because when we lookup the appropriate
port on receive, we'll basically get a random one and not necessarily
the one associated with the UDP port that the packet arrived on.  This
should get much easier once we simplify the kernel tunnel code because
in the normal case each protocol will only have a single port
associated with it and can just return that directly without doing a
lookup.  I know that Pravin was doing some work to remove the old
code.  Pravin, do you have anything you are ready to send out?
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to