[ovs-dev] [PATCH 1/7] Add support for 802.1ad (QinQ tunneling)

Xiao Liang shaw.leon at gmail.com
Fri Mar 17 12:31:19 UTC 2017


On Fri, Mar 17, 2017 at 11:57 AM, Ben Pfaff <blp at ovn.org> wrote:
> On Thu, Mar 16, 2017 at 09:18:00PM -0400, Eric Garver wrote:
>> On Thu, Mar 16, 2017 at 03:23:38PM -0700, Ben Pfaff wrote:
>> > Thanks for all the iteration on this patch.
>> >
>> > I've folded in the following incremental and applied this to master.
>>
>> Great! Thanks for the feedback and clean ups.
>>
>> Do you intend to take the tests as well? The macro OVS_CHECK_8021AD()
>> probably needs adjusted due the incremental below.
>>
>> The dot1q-tunnel patches should stand on their own and can be re-spun if
>> needed.
>
> I'm working on the dot1q-tunnel patch now and expect to apply it
> tomorrow, or if not then at least review it.
>
> After that, I think that I'll probably ask for a respin of the remaining
> patches.
>
> I am so glad to finally get QinQ support in, and I imagine that goes
> double for the authors of this patch.

I am really happy to see this feature goes in. Thanks for the efforts
from Thomas, Ben, Andy, and especially Eric.

Thanks,
Xiao


More information about the dev mailing list