[ovs-dev] [PATCH 1/1] Add support for tun_key to OVS datapath
Kyle Mestery (kmestery)
kmestery at cisco.com
Wed Aug 29 14:01:56 UTC 2012
On Aug 28, 2012, at 9:08 PM, Jesse Gross wrote:
> On Sun, Aug 26, 2012 at 6:55 PM, Kyle Mestery (kmestery)
> <kmestery at cisco.com> wrote:
>>
>> On Aug 21, 2012, at 2:05 AM, Simon Horman wrote:
>>
>>> On Mon, Aug 13, 2012 at 12:23:51PM -0400, Kyle Mestery 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.
>>>>
>>>> In ovs_skb_cb tun_key is a pointer as it is envisaged that it will grow
>>>> when support for IPv6 to an extent that inlining the structure will result
>>>> in ovs_skb_cb being larger than the 48 bytes available in skb->cb.
>>>>
>>>> As OVS does not support IPv6 as the outer transport protocol for tunnels
>>>> the IPv6 portions of this change, which appeared in the previous revision,
>>>> have been dropped in order to limit the scope and size of this patch.
>>>>
>>>> This patch allows all existing tun_id behaviour to still work. However,
>>>> when the userspace code is updated to make use of the new tun_key, the
>>>> old behaviour will be deprecated and removed.
>>>
>>> Hi Kyle,
>>>
>>> I apologise for not responding earlier, I was on holidays last week.
>>>
>> Hi Simon,
>>
>> I was on holidays last week myself.
>>
>>> I haven't manually compared this patch to the original patches that I posted
>>> but this patch looks good to me.
>>>
>> Thanks for reviewing this one! I hope Jesse can look at it. The main goal here
>> was to take your first two patches, collapse them together, and allow for the
>> existing tunnel configuration to remain in place. Then we can work on the next
>> few patches in this series after these go up.
>
> If you're planning on sending a new version soon then I'll wait until
> you send it out but then I'll make sure that it's the next thing that
> I look at.
Just sent a revised version with Simon's comments folded in.
Thanks,
Kyle
More information about the dev
mailing list