[ovs-dev] [PATCH v1] ofproto-dpif-xlate: fix zone set from non frozen metadata field

2021-02-15 Thread Peng He
CT zone could be set from a field that is not included in frozen metedata. Consider the belowing cases which is normally used in OpenStack security group rules: priority=100,in_port=1,tcp,ct_state=-trk,action=ct(zone=5,table=0) priority=100,in_port=1,tcp,ct_state=+trk,action=ct(commit,zone=NXM_NX_

Re: [ovs-dev] [PATCH v1] ofproto-dpif-xlate: fix zone set from non frozen metadata field

2021-02-15 Thread 0-day Robot
Bleep bloop. Greetings Peng He, 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: Author Peng He needs to sign off. WARNING: Unexpected sign-offs from developers who are not a

Re: [ovs-dev] [PATCH ovn] expr: Combine multiple ipv4 with wildcard mask.

2021-02-15 Thread 陈供明
Hi Mark, Sorry for getting back to you late. Thanks for your review. I totally agree with your very detailed suggestion. I think it is very useful, especially x2nrealloc. I will resubmit later. Thanks, Gongming On 2021/2/11, 9:26 AM, "Mark Michelson" wrote: Hi Gongming. I saw Dumitru's rev

Re: [ovs-dev] [PATCH 2/2] dpif-netdev: auto load balance improve enable/disable logs.

2021-02-15 Thread Ilya Maximets
On 2/13/21 9:33 AM, Stokes, Ian wrote: >> On 12/02/2021 17:42, Stokes, Ian wrote: In order for auto load balance to be enabled, there are minimum requirements of more than one PMD and more than one Rxq on at least one PMD. If these conditions are not met a rebalance would b

Re: [ovs-dev] [PATCH 2/2] dpif-netdev: auto load balance improve enable/disable logs.

2021-02-15 Thread Kevin Traynor
On 15/02/2021 13:06, Ilya Maximets wrote: > On 2/13/21 9:33 AM, Stokes, Ian wrote: >>> On 12/02/2021 17:42, Stokes, Ian wrote: > In order for auto load balance to be enabled, there are > minimum requirements of more than one PMD and more than > one Rxq on at least one PMD. > > I

Re: [ovs-dev] [PATCH v3] datapath: Add a new action dec_ttl

2021-02-15 Thread Matteo Croce
On Tue, Nov 24, 2020 at 11:43 AM Eelco Chaudron wrote: > > Add support for the dec_ttl action. Instead of programming the datapath with > a flow that matches the packet TTL and an IP set, use a single dec_ttl action. > > The old behavior is kept if the new action is not supported by the datapath.

Re: [ovs-dev] [PATCH 2/2] dpif-netdev: auto load balance improve enable/disable logs.

2021-02-15 Thread Stokes, Ian
> On 15/02/2021 13:06, Ilya Maximets wrote: > > On 2/13/21 9:33 AM, Stokes, Ian wrote: > >>> On 12/02/2021 17:42, Stokes, Ian wrote: > > In order for auto load balance to be enabled, there are > > minimum requirements of more than one PMD and more than > > one Rxq on at least one PMD. >

Re: [ovs-dev] [PATCH 2/2] dpif-netdev: auto load balance improve enable/disable logs.

2021-02-15 Thread Ilya Maximets
On 2/15/21 5:02 PM, Stokes, Ian wrote: >> On 15/02/2021 13:06, Ilya Maximets wrote: >>> On 2/13/21 9:33 AM, Stokes, Ian wrote: > On 12/02/2021 17:42, Stokes, Ian wrote: >>> In order for auto load balance to be enabled, there are >>> minimum requirements of more than one PMD and more tha

Re: [ovs-dev] [PATCH 2/2] dpif-netdev: auto load balance improve enable/disable logs.

2021-02-15 Thread Stokes, Ian
> On 2/15/21 5:02 PM, Stokes, Ian wrote: > >> On 15/02/2021 13:06, Ilya Maximets wrote: > >>> On 2/13/21 9:33 AM, Stokes, Ian wrote: > > On 12/02/2021 17:42, Stokes, Ian wrote: > >>> In order for auto load balance to be enabled, there are > >>> minimum requirements of more than one PMD

[ovs-dev] [PATCH branch-2.15 1/2] Set release date for 2.15.0.

2021-02-15 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 2 +- debian/changelog | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index a94af0279..bc901efdb 100644 --- a/NEWS +++ b/NEWS @@ -1,4 +1,4 @@ -v2.15.0 - xx xxx +v2.15.0 - 15 Feb 2021

[ovs-dev] [PATCH branch-2.15 0/2] Release patches for v2.15.0.

2021-02-15 Thread Ilya Maximets
Ilya Maximets (2): Set release date for 2.15.0. Prepare for 2.15.1. NEWS | 5 - configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 12 insertions(+), 3 deletions(-) -- 2.26.2 ___ dev mailing list d...@openvs

[ovs-dev] [PATCH branch-2.15 2/2] Prepare for 2.15.1.

2021-02-15 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 +++ configure.ac | 2 +- debian/changelog | 6 ++ 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index bc901efdb..fb769c819 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v2.15.1 - xx xxx +-

Re: [ovs-dev] [PATCH 2/2] dpif-netdev: auto load balance improve enable/disable logs.

2021-02-15 Thread Ilya Maximets
On 2/15/21 5:20 PM, Stokes, Ian wrote: >> On 2/15/21 5:02 PM, Stokes, Ian wrote: On 15/02/2021 13:06, Ilya Maximets wrote: > On 2/13/21 9:33 AM, Stokes, Ian wrote: >>> On 12/02/2021 17:42, Stokes, Ian wrote: > In order for auto load balance to be enabled, there are > mi

Re: [ovs-dev] [PATCH branch-2.15 0/2] Release patches for v2.15.0.

2021-02-15 Thread Stokes, Ian
> Ilya Maximets (2): > Set release date for 2.15.0. > Prepare for 2.15.1. > > NEWS | 5 - > configure.ac | 2 +- > debian/changelog | 8 +++- > 3 files changed, 12 insertions(+), 3 deletions(-) Thanks for preparing this Ilya, series LGTM. Acked-by: Ian Stokes > >

Re: [ovs-dev] [PATCH branch-2.15 0/2] Release patches for v2.15.0.

2021-02-15 Thread Ilya Maximets
On 2/15/21 6:03 PM, Stokes, Ian wrote: >> Ilya Maximets (2): >> Set release date for 2.15.0. >> Prepare for 2.15.1. >> >> NEWS | 5 - >> configure.ac | 2 +- >> debian/changelog | 8 +++- >> 3 files changed, 12 insertions(+), 3 deletions(-) > > Thanks for preparing thi

Re: [ovs-dev] [PATCH v4 0/5] XDP offload using flow API provider

2021-02-15 Thread William Tu
On Tue, Feb 9, 2021 at 1:39 AM Toshiaki Makita wrote: > > On 2021/02/05 2:36, William Tu wrote: > > Hi Toshiaki, > > > > Thanks for the patch. I've been testing it for a couple days. > > I liked it a lot! The compile and build process all work without any issues. > > Hi, thank you for reviewing! >

[ovs-dev] [PATCH ovn] ofctrl: Fix the assert seen when flood removing flows.

2021-02-15 Thread numans
From: Numan Siddique In one of the scaled deployments, ovn-controller is asserting with the below stack trace (gdb) bt #0 0x7fa6aed4970f in raise () from /lib64/libc.so.6 #1 0x7fa6aed33b25 in abort () from /lib64/libc.so.6 #2 0x55d46594a714 in ovs_abort_v

Re: [ovs-dev] [RFC ovn] northd: set max priority for automatic routes

2021-02-15 Thread Han Zhou
On Wed, Feb 10, 2021 at 2:17 PM Lorenzo Bianconi < lorenzo.bianc...@redhat.com> wrote: > > Increase priority for automatic routes (routes created assigning IP > addresses to OVN logical router interfaces) in order to always prefer them > over static routes since the router has a direct link to the