[ovs-dev] [PATCH v7 1/4] Avoid tunneling for VLAN packets redirected to a gateway chassis

2018-08-01 Thread vkommadi
From: venkata anil When a vm on a vlan tenant network sends traffic to an external network, it is tunneled from host chassis to gateway chassis. In the earlier discussion [1], Russel (also in his doc [2]) suggested if we can figure out a way for OVN to do this redirect to the gateway host over a

[ovs-dev] [PATCH v7 0/4] Use VLANs for VLAN packets redirected to a gateway chassis

2018-08-01 Thread vkommadi
From: venkata anil This patch avoids tunneling and instead uses source tenant vlan network across hypervisors for traffic from vlan network on local hypervisor towards gateway hypervisor hosting redirect chassiss port. On the local hypervisor, when the packet enters logical router ingress

Re: [ovs-dev] [PATCH V2 0/4] Enable set/match of tos/ttl for IP tunnels on TC data-path

2018-08-01 Thread Simon Horman
Hi Or, On 1 August 2018 at 13:21, Or Gerlitz wrote: > On Wed, Aug 1, 2018 at 2:07 PM, Simon Horman > wrote: > > On 1 August 2018 at 11:31, Simon Horman > wrote: > >> > >> Thanks Or, Thanks Ben, > >> > >> On 1 August 2018 at 08:43, Or Gerlitz wrote: > >>> > >>> On Tue, Jul 31, 2018 at 1:40

Re: [ovs-dev] [PATCH V2 0/4] Enable set/match of tos/ttl for IP tunnels on TC data-path

2018-08-01 Thread Or Gerlitz
On Wed, Aug 1, 2018 at 2:07 PM, Simon Horman wrote: > On 1 August 2018 at 11:31, Simon Horman wrote: >> >> Thanks Or, Thanks Ben, >> >> On 1 August 2018 at 08:43, Or Gerlitz wrote: >>> >>> On Tue, Jul 31, 2018 at 1:40 PM, Or Gerlitz >>> wrote: >>> > This series comes to address the case to set

Re: [ovs-dev] [PATCH V2 0/4] Enable set/match of tos/ttl for IP tunnels on TC data-path

2018-08-01 Thread Simon Horman
On 1 August 2018 at 11:31, Simon Horman wrote: > Thanks Or, Thanks Ben, > > On 1 August 2018 at 08:43, Or Gerlitz wrote: > >> On Tue, Jul 31, 2018 at 1:40 PM, Or Gerlitz >> wrote: >> > This series comes to address the case to set (encap) and match (decap) >> > also the tos and ttl fields of TC

Re: [ovs-dev] [PATCH v2 3/3] ovn-northd: Propagate dynamic addresses to port group address sets.

2018-08-01 Thread Simon Horman
On Wed, Aug 01, 2018 at 11:44:58AM +0200, Jakub Sitnicki wrote: > Hi Simon, > > On Wed, 1 Aug 2018 11:23:30 +0200 > Simon Horman wrote: > > > Hi Jakub, > > > > On Mon, Jul 30, 2018 at 04:37:49PM +0200, Jakub Sitnicki wrote: > > > If a logical switch port belongs to a port group and has dynamic

Re: [ovs-dev] [PATCH] ovn: Fix typos in "ovn -- Address Set generation..." test.

2018-08-01 Thread Simon Horman
On Tue, Jul 31, 2018 at 12:45:41PM -0700, Ben Pfaff wrote: > These caused the test to fail. > > CC: Jakub Sitnicki > Fixes: 984c7d5ea8fe ("ovn-northd: Propagate dynamic addresses to port group > address sets.") > Signed-off-by: Ben Pfaff > --- > tests/ovn.at | 4 ++-- > 1 file changed, 2

Re: [ovs-dev] [PATCH v2 3/3] ovn-northd: Propagate dynamic addresses to port group address sets.

2018-08-01 Thread Jakub Sitnicki
Hi Simon, On Wed, 1 Aug 2018 11:23:30 +0200 Simon Horman wrote: > Hi Jakub, > > On Mon, Jul 30, 2018 at 04:37:49PM +0200, Jakub Sitnicki wrote: > > If a logical switch port belongs to a port group and has dynamic > > addresses assigned, propagate the addresses to the auto-generated > > address

Re: [ovs-dev] [patch v8 8/9] ipf: Add set maximum fragments supported command.

2018-08-01 Thread Justin Pettit
> On Jul 16, 2018, at 4:39 PM, Darrell Ball wrote: > > diff --git a/lib/dpctl.man b/lib/dpctl.man > index a75fc77..f5a36b0 100644 > --- a/lib/dpctl.man > +++ b/lib/dpctl.man > @@ -296,3 +296,11 @@ must be specified. The default v4 value is 1200 and the > clamped minimum is > 400. The

Re: [ovs-dev] [patch v8 7/9] ipf: Add set minimum fragment size command.

2018-08-01 Thread Justin Pettit
> On Jul 16, 2018, at 4:39 PM, Darrell Ball wrote: > diff --git a/lib/ct-dpif.h b/lib/ct-dpif.h > index bd6234d..6eb55b4 100644 > --- a/lib/ct-dpif.h > +++ b/lib/ct-dpif.h > @@ -17,6 +17,7 @@ > #ifndef CT_DPIF_H > #define CT_DPIF_H > > +#include "ipf.h" I don't think it's necessary to add

Re: [ovs-dev] [patch v8 6/9] ipf: Add command to disable fragmentation handling.

2018-08-01 Thread Justin Pettit
> On Jul 16, 2018, at 4:39 PM, Darrell Ball wrote: > > diff --git a/lib/dpctl.man b/lib/dpctl.man > index 5d987e6..a1eb026 100644 > --- a/lib/dpctl.man > +++ b/lib/dpctl.man > @@ -272,3 +272,18 @@ Only supported for userspace datapath. > \*(DX\fBct\-get\-nconns\fR [\fIdp\fR] > Prints the

Re: [ovs-dev] [patch v8 5/9] Userspace datapath: ipf_count

2018-08-01 Thread Justin Pettit
> On Jul 16, 2018, at 7:39 PM, Darrell Ball wrote: > > +static void > +ipf_expiry_list_add(struct ipf_list *ipf_list, long long now) > +OVS_REQUIRES(ipf_lock) > +{ > +enum { > +IPF_FRAG_LIST_TIMEOUT_DEFAULT = 15000, > +}; > + > +ipf_list->expiration = now +

Re: [ovs-dev] [PATCH V2 0/4] Enable set/match of tos/ttl for IP tunnels on TC data-path

2018-08-01 Thread Simon Horman
Thanks Or, Thanks Ben, On 1 August 2018 at 08:43, Or Gerlitz wrote: > On Tue, Jul 31, 2018 at 1:40 PM, Or Gerlitz wrote: > > This series comes to address the case to set (encap) and match (decap) > > also the tos and ttl fields of TC based IP tunnels. This happens e.g > > due to inherit setup

Re: [ovs-dev] [PATCH v2 3/3] ovn-northd: Propagate dynamic addresses to port group address sets.

2018-08-01 Thread Simon Horman
Hi Jakub, On Mon, Jul 30, 2018 at 04:37:49PM +0200, Jakub Sitnicki wrote: > If a logical switch port belongs to a port group and has dynamic > addresses assigned, propagate the addresses to the auto-generated > address sets for the port group. > > Signed-off-by: Jakub Sitnicki > Acked-by: Han

Re: [ovs-dev] [PATCH] ovn: Fix typos in "ovn -- Address Set generation..." test.

2018-08-01 Thread Jakub Sitnicki
On Tue, 31 Jul 2018 12:45:41 -0700 Ben Pfaff wrote: > These caused the test to fail. > > CC: Jakub Sitnicki > Fixes: 984c7d5ea8fe ("ovn-northd: Propagate dynamic addresses to port group > address sets.") > Signed-off-by: Ben Pfaff > --- > tests/ovn.at | 4 ++-- > 1 file changed, 2

Re: [ovs-dev] [PATCH v4] Introduce ovs-appctl command to monitor HVs sb connection status

2018-08-01 Thread Jakub Sitnicki
On Tue, 31 Jul 2018 17:35:00 +0200 Lorenzo Bianconi wrote: > Add 'connection-status' command to ovs-appctl utility in order to check > if a given chassis is currently connected to SB db > > Acked-by: Mark Michelson > Co-authored-by: aginwala > Signed-off-by: aginwala > Signed-off-by: Lorenzo

[ovs-dev] [PATCH v2] rhel: support kmod build against multiple 7.2 kernels, rhel6

2018-08-01 Thread Martin Xu
This patch extends commit 89dd5819cf18 (rhel: support kmod-openvswitch build against multiple kernels, rhel6) to support building kmod RPMs with multiple minor revisions within 3.10.0-327 kernels. It was discovered for RHEL 7.2 that 41.3 minor revision introduced backward incompatible changes.

Re: [ovs-dev] [ovs-dev, v1] rhel: support kmod build against multiple 7.2 kernels, rhel6

2018-08-01 Thread 0-day Robot
Bleep bloop. Greetings Martin Xu, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: ERROR: No signatures found. Lines checked: 124, Warnings: 0, Errors: 1 Please check this out.

Re: [ovs-dev] [PATCH V2 0/4] Enable set/match of tos/ttl for IP tunnels on TC data-path

2018-08-01 Thread Or Gerlitz
On Tue, Jul 31, 2018 at 1:40 PM, Or Gerlitz wrote: > This series comes to address the case to set (encap) and match (decap) > also the tos and ttl fields of TC based IP tunnels. This happens e.g > due to inherit setup of tunnel port for tos or due to specific OF rule. > > The series is rebased

[ovs-dev] [PATCH v1] rhel: support kmod build against multiple 7.2 kernels, rhel6

2018-08-01 Thread Martin Xu
This patch extends commit 89dd5819cf18 (rhel: support kmod-openvswitch build against multiple kernels, rhel6) to support building kmod RPMs with multiple minor revisions within 3.10.0-327 kernels. It was discovered for RHEL 7.2 that 41.3 minor revision introduced backward incompatible changes.

<    1   2