Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-03-02 Thread Jarno Rajahalme
and resubmit them based on your tree. >> >> -Original Message- >> From: Joe Stringer [mailto:j...@ovn.org] >> Sent: Thursday, March 2, 2017 11:43 AM >> To: Yang, Yi Y >> Cc: ovs dev ; Jarno Rajahalme >> Subject: Re: [ovs-dev] [PATCH v3 00/16] port Ji

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-03-02 Thread Joe Stringer
r tree. > > -Original Message- > From: Joe Stringer [mailto:j...@ovn.org] > Sent: Thursday, March 2, 2017 11:43 AM > To: Yang, Yi Y > Cc: ovs dev ; Jarno Rajahalme > Subject: Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs > > On 6 February 2017 a

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-03-01 Thread Yang, Yi Y
space l3 patches and vxlangpe patches and resubmit them based on your tree. -Original Message- From: Joe Stringer [mailto:j...@ovn.org] Sent: Thursday, March 2, 2017 11:43 AM To: Yang, Yi Y Cc: ovs dev ; Jarno Rajahalme Subject: Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 p

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-03-01 Thread Joe Stringer
On 6 February 2017 at 05:04, Yi Yang wrote: > This patch set just ports Jiri Benc's L3 8 support patches for layer 3 > encapsulated packets from net-next to current ovs, it also includes Jiri > Benc's 3 userspace patches, Jarno Rajahalme and Pravin Shelar's vlan fix > patches for L3 patchset as

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-13 Thread Valentine Sinitsyn
Hi Jan, On 10.02.2017 04:14, Jan Scheurich wrote: Hi Valentine, On 2017-02-09 08:58, Valentine Sinitsyn wrote: This L3 patchset looks similar to what we did internally with OVS 2.6 to add support for IPv6 tunnels. Could you please confirm that ovs-dpctl reports correct statistics with this pa

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-12 Thread Yang, Yi Y
Joe, got it, thanks a lot. -Original Message- From: Joe Stringer [mailto:j...@ovn.org] Sent: Saturday, February 11, 2017 5:38 AM To: Yang, Yi Y Cc: Multanen, Eric W ; ovs dev ; Yi-Hung Wei Subject: Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs On 9 February

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-12 Thread Yang, Yi Y
t: Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs On Fri, Feb 10, 2017 at 01:33:04PM -0800, Joe Stringer wrote: > On 10 February 2017 at 12:25, Eric Garver wrote: > > On Fri, Feb 10, 2017 at 11:24:58AM -0500, Eric Garver wrote: > >> On Tue, Feb 07, 2

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-11 Thread Eric Garver
On Fri, Feb 10, 2017 at 01:33:04PM -0800, Joe Stringer wrote: > On 10 February 2017 at 12:25, Eric Garver wrote: > > On Fri, Feb 10, 2017 at 11:24:58AM -0500, Eric Garver wrote: > >> On Tue, Feb 07, 2017 at 02:30:46PM -0800, Joe Stringer wrote: > >> > On 7 February 2017 at 09:44, Joe Stringer wro

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-10 Thread Joe Stringer
On 9 February 2017 at 17:45, Yang, Yi Y wrote: > Joe, thank you for explanation, let us push 802.1ad backport patches first, > Eric Garver has acked two of them, I'll rework another one patch he > commented. What Valentine mentioned has no business with 802.1 ad, it is what > we want to conside

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-10 Thread Joe Stringer
On 10 February 2017 at 12:25, Eric Garver wrote: > On Fri, Feb 10, 2017 at 11:24:58AM -0500, Eric Garver wrote: >> On Tue, Feb 07, 2017 at 02:30:46PM -0800, Joe Stringer wrote: >> > On 7 February 2017 at 09:44, Joe Stringer wrote: >> > > On 6 February 2017 at 16:46, Yang, Yi Y wrote: >> > >> Joe

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-10 Thread Eric Garver
On Fri, Feb 10, 2017 at 11:24:58AM -0500, Eric Garver wrote: > On Tue, Feb 07, 2017 at 02:30:46PM -0800, Joe Stringer wrote: > > On 7 February 2017 at 09:44, Joe Stringer wrote: > > > On 6 February 2017 at 16:46, Yang, Yi Y wrote: > > >> Joe, I checked current ovs and net-next kernel, obviously s

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-10 Thread Eric Garver
On Tue, Feb 07, 2017 at 02:30:46PM -0800, Joe Stringer wrote: > On 7 February 2017 at 09:44, Joe Stringer wrote: > > On 6 February 2017 at 16:46, Yang, Yi Y wrote: > >> Joe, I checked current ovs and net-next kernel, obviously some patches > >> from net-next are selectively backported to ovs, bu

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-09 Thread Yang, Yi Y
point out if I missed something. -Original Message- From: Joe Stringer [mailto:j...@ovn.org] Sent: Friday, February 10, 2017 8:01 AM To: Yang, Yi Y Cc: ovs dev ; Yi-Hung Wei Subject: Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs On 8 February 2017 at 18:50, Yang,

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-09 Thread Yang, Yi Y
27; Cc: d...@openvswitch.org Subject: Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs Hi Yi, I suggest we include (adapted versions of) your vxlan-gpe user space patches into the new L3 tunneling user-space series when we respin v2 with the review comments. Thus we get rid of one

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-09 Thread Yang, Yi Y
r posted https://mail.openvswitch.org/pipermail/ovs-dev/2017-January/327749.html, -Original Message- From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev-boun...@openvswitch.org] On Behalf Of Valentine Sinitsyn Sent: Thursday, February 9, 2017 3:58 PM To: ovs-dev@openvswitch.org Subject: Re: [ovs-dev]

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-09 Thread Joe Stringer
On 8 February 2017 at 18:50, Yang, Yi Y wrote: > Then, are you merging > https://mail.openvswitch.org/pipermail/ovs-dev/2017-February/328492.html or > do I need to do anything else for it? It looks like the backport breaks the existing kernel module tests on older kernels, without even using th

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-09 Thread Jan Scheurich
Hi Valentine, On 2017-02-09 08:58, Valentine Sinitsyn wrote: This L3 patchset looks similar to what we did internally with OVS 2.6 to add support for IPv6 tunnels. Could you please confirm that ovs-dpctl reports correct statistics with this patchset when one uses in-kernel Linux datapath? We

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-09 Thread Jan Scheurich
Scheurich [mailto:jan.scheur...@ericsson.com] Sent: Thursday, February 9, 2017 5:50 PM To: Yang, Yi Y ; Joe Stringer ; Jiri Benc (jb...@redhat.com) ; Eric Garver Cc: d...@openvswitch.org; Jarno Rajahalme (ja...@ovn.org) Subject: RE: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs Hi Y

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-09 Thread Yang, Yi Y
Benc (jb...@redhat.com) ; Eric Garver Cc: d...@openvswitch.org; Jarno Rajahalme (ja...@ovn.org) Subject: RE: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs Hi Yi, I very much doubt that it makes sense to first merge the obsolete user-space patches and then override them

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-09 Thread Jan Scheurich
; Eric > Garver > Cc: d...@openvswitch.org > Subject: RE: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs > > I'll check how we can rebase your changes against those three patches on top > of my patches, our goals are same, that is to merge your > change

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-08 Thread Valentine Sinitsyn
Hi all, This L3 patchset looks similar to what we did internally with OVS 2.6 to add support for IPv6 tunnels. Could you please confirm that ovs-dpctl reports correct statistics with this patchset when one uses in-kernel Linux datapath? We had some issues with this (the counters were always

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-08 Thread Yang, Yi Y
Subject: Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs On 7 February 2017 at 21:15, Yang, Yi Y wrote: > Joe, I investigated all the patches which changed > include/linux/if_vlan.h for 802.1ad in net-next again, I found ovs > compat mode doesn't need

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-08 Thread Joe Stringer
On 7 February 2017 at 21:15, Yang, Yi Y wrote: > Joe, I investigated all the patches which changed include/linux/if_vlan.h for > 802.1ad in net-next again, I found ovs compat mode doesn't need most of them > at all, datapath/linux/compat/include/linux/ only includes part of changes > which aren

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-08 Thread Joe Stringer
On 7 February 2017 at 20:55, Yang, Yi wrote: > On Tue, Feb 07, 2017 at 02:30:46PM -0800, Joe Stringer wrote: >> On 7 February 2017 at 09:44, Joe Stringer wrote: >> > On 6 February 2017 at 16:46, Yang, Yi Y wrote: >> >> Joe, I checked current ovs and net-next kernel, obviously some patches >> >>

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-07 Thread Yang, Yi Y
itch.org Subject: RE: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs Hi Yi, Both Joe and Jarno have indicated there are no principle problems merging our self-contained user-space patches for L3 tunneling. May I suggest that you (together with Jiri and Eric) focus on the da

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-07 Thread Yang, Yi Y
2017 6:31 AM To: Yang, Yi Y Cc: ovs dev ; Yi-Hung Wei Subject: Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs On 7 February 2017 at 09:44, Joe Stringer wrote: > On 6 February 2017 at 16:46, Yang, Yi Y wrote: >> Joe, I checked current ovs and net-next kernel, obviou

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-07 Thread Yang, Yi
On Tue, Feb 07, 2017 at 02:30:46PM -0800, Joe Stringer wrote: > On 7 February 2017 at 09:44, Joe Stringer wrote: > > On 6 February 2017 at 16:46, Yang, Yi Y wrote: > >> Joe, I checked current ovs and net-next kernel, obviously some patches > >> from net-next are selectively backported to ovs, bu

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-07 Thread Yang, Yi Y
..@ovn.org] Sent: Wednesday, February 8, 2017 1:55 AM To: Yang, Yi Y Cc: Jan Scheurich ; d...@openvswitch.org Subject: Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs On 7 February 2017 at 05:28, Yang, Yi Y wrote: > Jan, I know that, but per Joe's comments, it se

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-07 Thread Yang, Yi Y
Good point, I will add a following patch to do this. -Original Message- From: Joe Stringer [mailto:j...@ovn.org] Sent: Wednesday, February 8, 2017 1:44 AM To: Yang, Yi Y Cc: ovs dev ; Yi-Hung Wei Subject: Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs On 6 Feb

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-07 Thread Jan Scheurich
net-next and OVS tree kernel module. /Jan > -Original Message- > From: Joe Stringer [mailto:j...@ovn.org] > Sent: Tuesday, 07 February, 2017 18:55 > To: Yang, Yi Y > Cc: Jan Scheurich ; d...@openvswitch.org > Subject: Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc&#x

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-07 Thread Joe Stringer
On 7 February 2017 at 09:44, Joe Stringer wrote: > On 6 February 2017 at 16:46, Yang, Yi Y wrote: >> Joe, I checked current ovs and net-next kernel, obviously some patches from >> net-next are selectively backported to ovs, but others are not, I'm not sure >> what the policy is for a new patch.

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-07 Thread Joe Stringer
On 7 February 2017 at 05:28, Yang, Yi Y wrote: > Jan, I know that, but per Joe's comments, it seems he won't merge your patch > set unless the part in kernel side is merged before them. It can't work > without these three patches. I have primarily expressed concern about missing kernel patch ba

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-07 Thread Joe Stringer
On 6 February 2017 at 16:46, Yang, Yi Y wrote: > Joe, I checked current ovs and net-next kernel, obviously some patches from > net-next are selectively backported to ovs, but others are not, I'm not sure > what the policy is for a new patch. It will be better that the person who did > the patch

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-07 Thread Yang, Yi Y
ebruary 7, 2017 6:50 PM To: Yang, Yi Y ; d...@openvswitch.org Cc: Zoltán Balogh ; Jarno Rajahalme (ja...@ovn.org) Subject: RE: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs Hi Yi, Three of the user-space patches of your series have been superseded by our new patch series [

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-07 Thread Jan Scheurich
Hi Yi, Three of the user-space patches of your series have been superseded by our new patch series [PATCH 0/7] userspace: Support for L3 tunneling: https://mail.openvswitch.org/pipermail/ovs-dev/2017-February/328391.html Please remove them from the next version. The user-space patches for VXLAN

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-06 Thread Yang, Yi Y
inger [mailto:j...@ovn.org] Sent: Tuesday, February 7, 2017 8:34 AM To: Yang, Yi Y Cc: ovs dev ; Yi-Hung Wei Subject: Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs On 6 February 2017 at 05:04, Yi Yang wrote: > This patch set just ports Jiri Benc's L3 8 support p

Re: [ovs-dev] [PATCH v3 00/16] port Jiri Benc's L3 patchset to ovs

2017-02-06 Thread Joe Stringer
On 6 February 2017 at 05:04, Yi Yang wrote: > This patch set just ports Jiri Benc's L3 8 support patches for layer 3 > encapsulated packets from net-next to current ovs, it also includes Jiri > Benc's 3 userspace patches, Jarno Rajahalme and Pravin Shelar's vlan fix > patches for L3 patchset as