[ovs-dev] [PATCH 1/2] Add support for tun_key to OVS datapath

Jesse Gross jesse at nicira.com
Fri Oct 5 17:47:41 UTC 2012


On Mon, Oct 1, 2012 at 8:52 AM, Kyle Mestery <kmestery at cisco.com> wrote:
> This is a first pass at providing a tun_key which can be
> used as the basis for flow-based tunnelling. The
> tun_key includes and replaces the tun_id in both struct
> ovs_skb_cb and struct sw_tun_key.
>
> This patch allows all existing tun_id behaviour to still work. Existing
> users of tun_id are redirected to tun_key->tun_id to retain compatibility.
> However, when the userspace code is updated to make use of the new tun_key,
> the old behaviour will be deprecated and removed.
>
> NOTE: With these changes, the tunneling code no longer assumes input and
> output keys are symmetric.  If they are not, PMTUD needs to be disabled
> for tunneling to work.
>
> Signed-off-by: Kyle Mestery <kmestery at cisco.com>
> Cc: Simon Horman <horms at verge.net.au>
> Cc: Jesse Gross <jesse at nicira.com>

One thought that I had was to try to get the kernel portions in one
release and the userspace portions that actually use them in the next.
 That way we would give people a window to smoothly upgrade because
the kernel module would support both mechanisms for that release.

We're currently planning to branch for the next release in two weeks
(although I'm going to be traveling for the second one and unable to
review code).  I think we're getting pretty close here, so do you
think my plan is doable?



More information about the dev mailing list