[ovs-dev] [PATCH 5/6] vxlan: Update tnl_set_config() to use UDP dst_port to distinguish VXLAN ports

Kyle Mestery (kmestery) kmestery at cisco.com
Fri Feb 15 17:20:43 UTC 2013


On Feb 15, 2013, at 10:58 AM, Jesse Gross <jesse at nicira.com> wrote:
> On Thu, Feb 14, 2013 at 6:37 AM, Kyle Mestery <kmestery at 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 <kmestery at 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?

Thanks Jesse. I think this explains some of the issues I'm seeing now. I
am keenly interested in Pravin's patches here. :)


More information about the dev mailing list