Re: [ovs-dev] [ovs-dev v3 3/4] ofproto-dpif-upcall: new ukey needs to take the old ukey's dump seq

2022-10-07 Thread Peng He
Hi,Eelco, Which kernel version you have tested on? I've also tested the #50 for several hundreds times, but only get 0 packets once. I have put your patches upon on commits 49efc63ad: ofproto-dpif-xlate: Fix error messages for nonexistent ports/recirc_ids. not sure if it's related to the

Re: [ovs-dev] [PATCH v4 3/3] dpif-netdev: fix inconsistent processing between ukey and megaflow

2022-10-07 Thread Peng He
Hi,Eelco after a second thought, I think this patch is not needed neither, the code here is trying to find a rule which cover the packet, it does not mean the match and action of rule equals to the ones of the ukey. So the code here is just a prevention, no need to make it consistent with ukey.

Re: [ovs-dev] [PATCH ovn 5/8] controller: Add support for templated actions and matches.

2022-10-07 Thread Han Zhou
On Fri, Oct 7, 2022 at 12:23 PM Ilya Maximets wrote: > > On 10/7/22 20:31, Han Zhou wrote: > > > > > > On Fri, Oct 7, 2022 at 10:24 AM Ilya Maximets mailto:i.maxim...@ovn.org>> wrote: > >> > >> On 10/7/22 10:26, Dumitru Ceara wrote: > >> > On 10/7/22 04:03, Han Zhou wrote: > >> >> On Fri, Sep 30,

Re: [ovs-dev] [PATCH ovn 5/8] controller: Add support for templated actions and matches.

2022-10-07 Thread Ilya Maximets
On 10/7/22 20:31, Han Zhou wrote: > > > On Fri, Oct 7, 2022 at 10:24 AM Ilya Maximets > wrote: >> >> On 10/7/22 10:26, Dumitru Ceara wrote: >> > On 10/7/22 04:03, Han Zhou wrote: >> >> On Fri, Sep 30, 2022 at 7:02 AM Dumitru Ceara > >> >

Re: [ovs-dev] [PATCH ovn 5/8] controller: Add support for templated actions and matches.

2022-10-07 Thread Han Zhou
On Fri, Oct 7, 2022 at 10:24 AM Ilya Maximets wrote: > > On 10/7/22 10:26, Dumitru Ceara wrote: > > On 10/7/22 04:03, Han Zhou wrote: > >> On Fri, Sep 30, 2022 at 7:02 AM Dumitru Ceara wrote: > >>> > >>> Expand SB.Template_Var records in two stages: > >>> 1. first expand them to local values in

Re: [ovs-dev] [PATCH 1/2] netdev-offload: Expose error when init_flow_api() returns EBUSY.

2022-10-07 Thread Mike Pattrick
On Mon, Aug 8, 2022 at 12:54 PM Han Zhou wrote: > > init_flow_api() can fail due to errors, such as netlink "Device or > resource busy". Today the errors are ignored except just an INFO log. > This ends up with an interface created without offload enabled, and > application won't notice it. This

Re: [ovs-dev] [OVSCON] Talk descriptions and abstracts are up

2022-10-07 Thread Aaron Conole
Apologies for the truncation. Remeber that for the hotel, we have partnered with Mariott to provide a block of discounted rooms. These are available on a first-come, first-served basis and are limited in quantity. The link to register is at:

[ovs-dev] [OVSCON] Talk descriptions and abstracts are up

2022-10-07 Thread Aaron Conole
Greetings, The initial list of talks can now be found on https://ovscon.site along with a registration link to register. Remember that the hotel We have a number of workshops to announce in the coming days as well, so stay tuned. -OVS+OvN Planning committee

Re: [ovs-dev] [PATCH] tests: fix reference output for meter offload stats

2022-10-07 Thread Marcelo Leitner
On Fri, Oct 07, 2022 at 11:59:42AM -0400, Jamal Hadi Salim wrote: > On Fri, Oct 7, 2022 at 11:01 AM Marcelo Leitner wrote: > > > > On Fri, Oct 07, 2022 at 04:39:25PM +0200, Davide Caratti wrote: > > > On Fri, Oct 7, 2022 at 3:21 PM Marcelo Leitner > > > wrote: > > > > > > > > (+TC folks and

Re: [ovs-dev] [PATCH ovn 5/8] controller: Add support for templated actions and matches.

2022-10-07 Thread Ilya Maximets
On 10/7/22 10:26, Dumitru Ceara wrote: > On 10/7/22 04:03, Han Zhou wrote: >> On Fri, Sep 30, 2022 at 7:02 AM Dumitru Ceara wrote: >>> >>> Expand SB.Template_Var records in two stages: >>> 1. first expand them to local values in match/action strings >>> 2. then reparse the expanded strings >>>

Re: [ovs-dev] [PATCH branch-3.0 0/2] Release patches for v3.0.1.

2022-10-07 Thread Ilya Maximets
On 10/7/22 16:57, Flavio Leitner wrote: > On Fri, Oct 07, 2022 at 01:17:33PM +0200, Ilya Maximets wrote: >> >> Ilya Maximets (2): >> Set release date for 3.0.1. >> Prepare for 3.0.2. >> >> NEWS | 6 +- >> configure.ac | 2 +- >> debian/changelog | 8 +++- >> 3 files

[ovs-dev] [PATCH v4 ovn 6/7] Add connectivity test for 2 controllers on the same host

2022-10-07 Thread Ihar Hrachyshka
Signed-off-by: Ihar Hrachyshka --- tests/ovn.at | 123 +++ 1 file changed, 123 insertions(+) diff --git a/tests/ovn.at b/tests/ovn.at index 3c76ba49c..1d69de96e 100644 --- a/tests/ovn.at +++ b/tests/ovn.at @@ -33170,3 +33170,126 @@

[ovs-dev] [PATCH v4 ovn 5/7] Don't touch tunnel ports from a different br-int

2022-10-07 Thread Ihar Hrachyshka
When multiple controllers are running using the same vswitchd, controllers should delete only those tunnel ports that belong to the integration bridge that is managed by the controller instance. This makes sure multiple controllers don't step on each other when running using the same vswitchd

[ovs-dev] [PATCH v4 ovn 2/7] Support ovn-...- specific global ovsdb options

2022-10-07 Thread Ihar Hrachyshka
Before the patch, all controller instances were reading configuration from the same external-ids:ovn-* options. This patch adds support for distinct config otions for different chassis names stored in the same ovsdb global config object. To configure an option for a distinct chassis name, an

[ovs-dev] [PATCH v4 ovn 4/7] Support passing chassis name via CLI

2022-10-07 Thread Ihar Hrachyshka
This patch adds support for the desired system-id (chassis name) to be passed via CLI: $ ovn-controller -n If passed, CLI overrides any settings stored in ovsdb or in system-id-override file. This may be useful when running multiple controller instances using the same vswitchd instance.

[ovs-dev] [PATCH v4 ovn 7/7] Document experimental support for co-hosted controllers

2022-10-07 Thread Ihar Hrachyshka
Signed-off-by: Ihar Hrachyshka --- NEWS| 2 ++ controller/ovn-controller.8.xml | 12 2 files changed, 14 insertions(+) diff --git a/NEWS b/NEWS index 224a7b83e..63afcea9d 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,7 @@ Post v22.09.0 - + -

[ovs-dev] [PATCH v4 ovn 1/7] Include "chassis index" into tunnel port name

2022-10-07 Thread Ihar Hrachyshka
This is in preparation to support multiple separate controller instances with distinct chassis names operating on the same vswitchd instance. To avoid conflicts, this patch introduces a unique "index" (from 0-9a-z range) into the port name. Each chassis allocates a separate index for itself on

[ovs-dev] [PATCH v4 ovn 3/7] Allow to override system-id via file

2022-10-07 Thread Ihar Hrachyshka
Before the patch, system-id could be configured via a global config option in ovsdb. This patch adds another option - configure system-id via a file. This is achieved by writing the desired system-id into the following file location: ${OVN_SYSCONFDIR}/system-id-override. The file is read on

[ovs-dev] [PATCH v4 ovn 0/7] Support 2+ controllers on the same vswitchd

2022-10-07 Thread Ihar Hrachyshka
This series adds support to run multiple ovn-controller instances using the same vswitchd instance. This may be used to reuse a single host level vswitchd installation to run multiple CMS (e.g. k8s and openstack), each having its own OVN stack running on a separate integration bridge. This setup

Re: [ovs-dev] [PATCH v6 9/9] mfex-avx512: Add support for tunnel packets in avx512 MFEX.

2022-10-07 Thread Pai G, Sunil
> -Original Message- > From: dev On Behalf Of Kumar Amber > Sent: Thursday, October 6, 2022 3:54 PM > To: ovs-dev@openvswitch.org > Cc: i.maxim...@ovn.org; f...@sysclose.org; Amber, Kumar > > Subject: [ovs-dev] [PATCH v6 9/9] mfex-avx512: Add support for tunnel > packets in avx512

Re: [ovs-dev] [PATCH v6 8/9] mfex-study: Modify study func to select outer and inner MFEX funcs.

2022-10-07 Thread Pai G, Sunil
> -Original Message- > From: dev On Behalf Of Kumar Amber > Sent: Thursday, October 6, 2022 3:54 PM > To: ovs-dev@openvswitch.org > Cc: i.maxim...@ovn.org; f...@sysclose.org; Amber, Kumar > > Subject: [ovs-dev] [PATCH v6 8/9] mfex-study: Modify study func to select > outer and inner

Re: [ovs-dev] [PATCH v6 7/9] dpif-mfex: Change MFEX fn pointer prototype to include md_is_valid.

2022-10-07 Thread Pai G, Sunil
> -Original Message- > From: dev On Behalf Of Kumar Amber > Sent: Thursday, October 6, 2022 3:54 PM > To: ovs-dev@openvswitch.org > Cc: i.maxim...@ovn.org; f...@sysclose.org; Amber, Kumar > > Subject: [ovs-dev] [PATCH v6 7/9] dpif-mfex: Change MFEX fn pointer > prototype to include

Re: [ovs-dev] [PATCH] tests: fix reference output for meter offload stats

2022-10-07 Thread Jamal Hadi Salim
On Fri, Oct 7, 2022 at 11:01 AM Marcelo Leitner wrote: > > On Fri, Oct 07, 2022 at 04:39:25PM +0200, Davide Caratti wrote: > > On Fri, Oct 7, 2022 at 3:21 PM Marcelo Leitner wrote: > > > > > > (+TC folks and netdev@) > > > > > > On Fri, Oct 07, 2022 at 02:42:56PM +0200, Ilya Maximets wrote: > >

Re: [ovs-dev] [RFC dpdk-latest v2 3/3] netdev-dpdk: Add per virtqueue statistics.

2022-10-07 Thread Maxime Coquelin
Hi David, On 10/7/22 13:16, David Marchand wrote: Request per virtqueue statistics from the vhost library and expose them as per port OVS custom stats. Thanks for the patch! Note: - the vhost stats API is experimental at the moment, this patch is sent as a demonstrator, I'm going to

Re: [ovs-dev] [PATCH] tests: fix reference output for meter offload stats

2022-10-07 Thread Ilya Maximets
On 10/7/22 16:49, Eelco Chaudron wrote: > > > On 7 Oct 2022, at 14:42, Ilya Maximets wrote: > >> On 10/7/22 13:37, Eelco Chaudron wrote: >>> >>> >>> On 15 Sep 2022, at 14:03, Ilya Maximets wrote: >>> On 9/15/22 13:56, Ilya Maximets wrote: > On 9/15/22 13:38, Tianyu Yuan wrote: >>

Re: [ovs-dev] [PATCH v6 5/9] dpif-netdev: Add function pointer for dpif re-circulate.

2022-10-07 Thread Pai G, Sunil
> -Original Message- > From: dev On Behalf Of Kumar Amber > Sent: Thursday, October 6, 2022 3:54 PM > To: ovs-dev@openvswitch.org > Cc: i.maxim...@ovn.org; f...@sysclose.org; Amber, Kumar > > Subject: [ovs-dev] [PATCH v6 5/9] dpif-netdev: Add function pointer for > dpif re-circulate.

Re: [ovs-dev] [PATCH] tests: fix reference output for meter offload stats

2022-10-07 Thread Eelco Chaudron
On 7 Oct 2022, at 16:39, Davide Caratti wrote: > On Fri, Oct 7, 2022 at 3:21 PM Marcelo Leitner wrote: >> >> (+TC folks and netdev@) >> >> On Fri, Oct 07, 2022 at 02:42:56PM +0200, Ilya Maximets wrote: >>> On 10/7/22 13:37, Eelco Chaudron wrote: > > [...] > >> I don't see how we could achieve

Re: [ovs-dev] [PATCH] tests: fix reference output for meter offload stats

2022-10-07 Thread Marcelo Leitner
On Fri, Oct 07, 2022 at 04:39:25PM +0200, Davide Caratti wrote: > On Fri, Oct 7, 2022 at 3:21 PM Marcelo Leitner wrote: > > > > (+TC folks and netdev@) > > > > On Fri, Oct 07, 2022 at 02:42:56PM +0200, Ilya Maximets wrote: > > > On 10/7/22 13:37, Eelco Chaudron wrote: > > [...] > > > I don't see

Re: [ovs-dev] [PATCH branch-3.0 0/2] Release patches for v3.0.1.

2022-10-07 Thread Flavio Leitner
On Fri, Oct 07, 2022 at 01:17:33PM +0200, Ilya Maximets wrote: > > Ilya Maximets (2): > Set release date for 3.0.1. > Prepare for 3.0.2. > > NEWS | 6 +- > configure.ac | 2 +- > debian/changelog | 8 +++- > 3 files changed, 13 insertions(+), 3 deletions(-) > > --

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

2022-10-07 Thread Flavio Leitner
On Fri, Oct 07, 2022 at 01:17:21PM +0200, Ilya Maximets wrote: > > Ilya Maximets (2): > Set release date for 2.17.3. > Prepare for 2.17.4. > > NEWS | 6 +- > configure.ac | 2 +- > debian/changelog | 8 +++- > 3 files changed, 13 insertions(+), 3 deletions(-) > > --

Re: [ovs-dev] [PATCH] tests: fix reference output for meter offload stats

2022-10-07 Thread Eelco Chaudron
On 7 Oct 2022, at 14:42, Ilya Maximets wrote: > On 10/7/22 13:37, Eelco Chaudron wrote: >> >> >> On 15 Sep 2022, at 14:03, Ilya Maximets wrote: >> >>> On 9/15/22 13:56, Ilya Maximets wrote: On 9/15/22 13:38, Tianyu Yuan wrote: > > On 9/15/22 19:28, Ilya Maximets wrote: >> On

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

2022-10-07 Thread Flavio Leitner
On Fri, Oct 07, 2022 at 01:17:06PM +0200, Ilya Maximets wrote: > > Ilya Maximets (2): > Set release date for 2.16.5. > Prepare for 2.16.6. > > NEWS | 6 +- > configure.ac | 2 +- > debian/changelog | 8 +++- > 3 files changed, 13 insertions(+), 3 deletions(-) > > --

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

2022-10-07 Thread Flavio Leitner
On Fri, Oct 07, 2022 at 01:16:54PM +0200, Ilya Maximets wrote: > > Ilya Maximets (2): > Set release date for 2.15.6. > Prepare for 2.15.7. > > NEWS | 6 +- > configure.ac | 2 +- > debian/changelog | 8 +++- > 3 files changed, 13 insertions(+), 3 deletions(-) > > --

Re: [ovs-dev] [PATCH] tests: fix reference output for meter offload stats

2022-10-07 Thread Davide Caratti
On Fri, Oct 7, 2022 at 3:21 PM Marcelo Leitner wrote: > > (+TC folks and netdev@) > > On Fri, Oct 07, 2022 at 02:42:56PM +0200, Ilya Maximets wrote: > > On 10/7/22 13:37, Eelco Chaudron wrote: [...] > I don't see how we could achieve this without breaking much of the > user experience. > > > >

Re: [ovs-dev] [PATCH v2] dpif-netlink: add revalidator for offload of meters

2022-10-07 Thread Eelco Chaudron
On 30 Sep 2022, at 13:12, Simon Horman wrote: > From: Yifan Li > > Allow revalidator for hardware offload of meters via OVS-TC. > Without revalidator, tc meter action can not be deleted while > flow exists. The revalidator fix this bug by continuously > checking existing meters and delete the

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

2022-10-07 Thread Flavio Leitner
On Fri, Oct 07, 2022 at 01:16:41PM +0200, Ilya Maximets wrote: > > Ilya Maximets (2): > Set release date for 2.14.7. > Prepare for 2.14.8. > > NEWS | 6 +- > configure.ac | 2 +- > debian/changelog | 8 +++- > 3 files changed, 13 insertions(+), 3 deletions(-) > > --

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

2022-10-07 Thread Flavio Leitner
On Fri, Oct 07, 2022 at 01:16:29PM +0200, Ilya Maximets wrote: > > Ilya Maximets (2): > Set release date for 2.13.9. > Prepare for 2.13.10. > > NEWS | 6 +- > configure.ac | 2 +- > debian/changelog | 8 +++- > 3 files changed, 13 insertions(+), 3 deletions(-) > >

Re: [ovs-dev] [PATCH] tests: fix reference output for meter offload stats

2022-10-07 Thread Marcelo Leitner
(+TC folks and netdev@) On Fri, Oct 07, 2022 at 02:42:56PM +0200, Ilya Maximets wrote: > On 10/7/22 13:37, Eelco Chaudron wrote: > > > > > > On 15 Sep 2022, at 14:03, Ilya Maximets wrote: > > > >> On 9/15/22 13:56, Ilya Maximets wrote: > >>> On 9/15/22 13:38, Tianyu Yuan wrote: > > On

Re: [ovs-dev] [PATCH] tests: fix reference output for meter offload stats

2022-10-07 Thread Ilya Maximets
On 10/7/22 13:37, Eelco Chaudron wrote: > > > On 15 Sep 2022, at 14:03, Ilya Maximets wrote: > >> On 9/15/22 13:56, Ilya Maximets wrote: >>> On 9/15/22 13:38, Tianyu Yuan wrote: On 9/15/22 19:28, Ilya Maximets wrote: > On 9/14/22 16:19, Simon Horman wrote: >> From: Tianyu

Re: [ovs-dev] [PATCH] tests: fix reference output for meter offload stats

2022-10-07 Thread Eelco Chaudron
On 15 Sep 2022, at 14:03, Ilya Maximets wrote: > On 9/15/22 13:56, Ilya Maximets wrote: >> On 9/15/22 13:38, Tianyu Yuan wrote: >>> >>> On 9/15/22 19:28, Ilya Maximets wrote: On 9/14/22 16:19, Simon Horman wrote: > From: Tianyu Yuan > > Since dd9881ed55e6 ("tc: Fix stats dump

Re: [ovs-dev] [OVN v10] OVN - Add Support for Remote Port Mirroring

2022-10-07 Thread 0-day Robot
Bleep bloop. Greetings Abhiram R N, 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: WARNING: Line lacks whitespace around operator #2148 FILE: utilities/ovn-nbctl.c:275:

[ovs-dev] [PATCH branch-2.17 2/2] Prepare for 2.17.4.

2022-10-07 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 7858b957f..b04b7b4c9 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v2.17.4 - xx xxx

[ovs-dev] [PATCH branch-2.17 0/2] Release patches for v2.17.3.

2022-10-07 Thread Ilya Maximets
Ilya Maximets (2): Set release date for 2.17.3. Prepare for 2.17.4. NEWS | 6 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 13 insertions(+), 3 deletions(-) -- 2.37.3 ___ dev mailing list

[ovs-dev] [PATCH branch-2.17 1/2] Set release date for 2.17.3.

2022-10-07 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 ++- debian/changelog | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index 36fcbb874..7858b957f 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,6 @@ -v2.17.3 - xx xxx +v2.17.3 - 07 Oct 2022

[ovs-dev] [PATCH branch-3.0 2/2] Prepare for 3.0.2.

2022-10-07 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 a7f367b7c..eee77189b 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v3.0.2 - xx xxx

[ovs-dev] [PATCH branch-3.0 1/2] Set release date for 3.0.1.

2022-10-07 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 ++- debian/changelog | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index ad5ba021d..a7f367b7c 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,6 @@ -v3.0.1 - xx xxx +v3.0.1 - 07 Oct 2022

[ovs-dev] [PATCH branch-3.0 0/2] Release patches for v3.0.1.

2022-10-07 Thread Ilya Maximets
Ilya Maximets (2): Set release date for 3.0.1. Prepare for 3.0.2. NEWS | 6 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 13 insertions(+), 3 deletions(-) -- 2.37.3 ___ dev mailing list

[ovs-dev] [PATCH branch-2.16 2/2] Prepare for 2.16.6.

2022-10-07 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 9693a17db..331af302b 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v2.16.6 - xx xxx

[ovs-dev] [PATCH branch-2.16 1/2] Set release date for 2.16.5.

2022-10-07 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 ++- debian/changelog | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index 578b845dd..9693a17db 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,6 @@ -v2.16.5 - xx xxx +v2.16.5 - 07 Oct 2022

[ovs-dev] [PATCH branch-2.16 0/2] Release patches for v2.16.5.

2022-10-07 Thread Ilya Maximets
Ilya Maximets (2): Set release date for 2.16.5. Prepare for 2.16.6. NEWS | 6 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 13 insertions(+), 3 deletions(-) -- 2.37.3 ___ dev mailing list

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

2022-10-07 Thread Ilya Maximets
Ilya Maximets (2): Set release date for 2.15.6. Prepare for 2.15.7. NEWS | 6 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 13 insertions(+), 3 deletions(-) -- 2.37.3 ___ dev mailing list

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

2022-10-07 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 ++- debian/changelog | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index 4c264c090..775f36bb9 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,6 @@ -v2.15.6 - xx xxx +v2.15.6 - 07 Oct 2022

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

2022-10-07 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 775f36bb9..6510dfc96 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v2.15.7 - xx xxx

[ovs-dev] [PATCH branch-2.14 2/2] Prepare for 2.14.8.

2022-10-07 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 57b703503..817249593 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v2.14.8 - xx xxx

[ovs-dev] [PATCH branch-2.14 0/2] Release patches for v2.14.7.

2022-10-07 Thread Ilya Maximets
Ilya Maximets (2): Set release date for 2.14.7. Prepare for 2.14.8. NEWS | 6 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 13 insertions(+), 3 deletions(-) -- 2.37.3 ___ dev mailing list

[ovs-dev] [PATCH branch-2.14 1/2] Set release date for 2.14.7.

2022-10-07 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 ++- debian/changelog | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index 67690e519..57b703503 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,6 @@ -v2.14.7 - xx xxx +v2.14.7 - 07 Oct 2022

[ovs-dev] [PATCH branch-2.13 2/2] Prepare for 2.13.10.

2022-10-07 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 3b7ece1d3..fe5743bb8 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v2.13.10 - xx xxx

[ovs-dev] [PATCH branch-2.13 0/2] Release patches for v2.13.9.

2022-10-07 Thread Ilya Maximets
Ilya Maximets (2): Set release date for 2.13.9. Prepare for 2.13.10. NEWS | 6 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 13 insertions(+), 3 deletions(-) -- 2.37.3 ___ dev mailing list

[ovs-dev] [PATCH branch-2.13 1/2] Set release date for 2.13.9.

2022-10-07 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 ++- debian/changelog | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index 8e2553901..3b7ece1d3 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,6 @@ -v2.13.9 - xx xxx +v2.13.9 - 07 Oct 2022

[ovs-dev] [RFC dpdk-latest v2 3/3] netdev-dpdk: Add per virtqueue statistics.

2022-10-07 Thread David Marchand
Request per virtqueue statistics from the vhost library and expose them as per port OVS custom stats. Note: - the vhost stats API is experimental at the moment, this patch is sent as a demonstrator, - we may drop maintaining rx stats in OVS itself and instead aggregate the per vq stats, this

[ovs-dev] [PATCH dpdk-latest v2 2/3] netdev-dpdk: Drop reference to Rx header split.

2022-10-07 Thread David Marchand
The Rx header split feature was implemented by no driver in DPDK. This feature is being reworked, and one associated field has been removed from the Rx port configuration (see [1]). OVS was not using it, simply drop reference to this field. 1: https://git.dpdk.org/dpdk/commit/?id=8d54b1ec4a8b

[ovs-dev] [PATCH dpdk-latest v2 0/3] API updates for DPDK 22.11

2022-10-07 Thread David Marchand
Hello, This series has two parts. First, it aligns OVS for some small updates on DPDK API side: - make use of new API following hiding of DPDK internals, - remove reference to some unused Rx feature in port configuration, Then, a patch adds per vhost virtqueue statistics. This last patch is not

[ovs-dev] [PATCH dpdk-latest v2 1/3] netdev-dpdk: Report device bus specific information.

2022-10-07 Thread David Marchand
22.11 dropped direct access to bus specific structures. Instead, a new API has been added. Report bus name and device bus specific information. The difference looks like: -driver_name=mlx5_pci, if_descr="DPDK 21.11.0 mlx5_pci" +driver_name=mlx5_pci, if_descr="DPDK 22.11.0-rc0 mlx5_pci"

[ovs-dev] [PATCH ovn] controller: improve buffered packets management

2022-10-07 Thread Lorenzo Bianconi
Improve buffered packet management in ovn-controller avoid useless loop in run_buffered_binding routine and using datapath key and output port key as buffered_packets_map hashmap hash. Add new selftest for buffered packets. Signed-off-by: Lorenzo Bianconi --- controller/pinctrl.c | 119

Re: [ovs-dev] [PATCH v3 3/3] netdev-offload-dpdk: Enhance the support of tunnel pop action

2022-10-07 Thread 0-day Robot
References: <20221007103923.228716-4-simon.hor...@corigine.com> Bleep bloop. Greetings Simon Horman, 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: WARNING: Unexpected

[ovs-dev] [OVN v10] OVN - Add Support for Remote Port Mirroring

2022-10-07 Thread Abhiram R N
Added changes in ovn-nbctl, ovn-sbctl, northd and in ovn-controller. While Mirror creation just creates the mirror, the lsp-attach-mirror triggers the sequence to create Mirror in OVS DB on compute node. OVS already supports Port Mirroring. Note: This is targeted to mirror to destinations

Re: [ovs-dev] [PATCH 0/2] Remove deprecated OpenSSL functions on openssl 3.0

2022-10-07 Thread Ilya Maximets
On 9/22/22 15:40, Timothy Redaelli wrote: > Currently, it's not possible to build OVS using OpenSSL 3.0 with > --enable-Werror since OpenSSL 3.0 deprecated some functions. > Moreover, it's not possible to generate dhparams.c anymore on > OpenSSL 3.0 since -C option was removed from openssl dhparam

Re: [ovs-dev] [PATCH v3 2/3] netdev-offload-dpdk: Support offload of set IPv6 DSCP action

2022-10-07 Thread 0-day Robot
References: <20221007103923.228716-3-simon.hor...@corigine.com> Bleep bloop. Greetings Simon Horman, 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: WARNING: Unexpected

Re: [ovs-dev] [PATCH v3 1/3] netdev-offload-dpdk: Support offload of set IPv4 DSCP action

2022-10-07 Thread 0-day Robot
References: <20221007103923.228716-2-simon.hor...@corigine.com> Bleep bloop. Greetings Simon Horman, 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: WARNING: Unexpected

Re: [ovs-dev] [ovs-dev v3 3/4] ofproto-dpif-upcall: new ukey needs to take the old ukey's dump seq

2022-10-07 Thread Eelco Chaudron
Hi hepeng, I tried your changes, but I also do get a lot of zero packet failures on test 50. Not sure what happened, maybe it was related to syncing to the latest master. However I focused on test 49, and I can replicate it every x test, but within 40 tries: ##

[ovs-dev] [PATCH v3 2/3] netdev-offload-dpdk: Support offload of set IPv6 DSCP action

2022-10-07 Thread Simon Horman
From: Chaoyong He Add the support of offload of set IPv6 DSCP action. Signed-off-by: Chaoyong He Reviewed-by: Louis Peens Signed-off-by: Simon Horman --- lib/netdev-offload-dpdk.c | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/lib/netdev-offload-dpdk.c

[ovs-dev] [PATCH v3 1/3] netdev-offload-dpdk: Support offload of set IPv4 DSCP action

2022-10-07 Thread Simon Horman
From: Chaoyong He Both the 'mod_nw_tos' and 'mod_nw_enc' command of ovs will cause the 'RTE_FLOW_ACTION_TYPE_SET_IPV4_DSCP' be sent to PMD, and they share one mask 'ipv4_tos' of size 1 byte. According the original logic, both of them won't pass the mask check because the check is based on size

[ovs-dev] [PATCH v3 3/3] netdev-offload-dpdk: Enhance the support of tunnel pop action

2022-10-07 Thread Simon Horman
From: Chaoyong He We should try our best to fill all the fields of 'struct rte_flow_tunnel', rather than just give PMD what it need for now. The number and logic of PMD are increase everyday, and they can use any field in the API I think. For nfp PMD, to implement the offload of tunnel pop

[ovs-dev] [PATCH v3 0/3] add functions about rte_flow to ovs-dpdk

2022-10-07 Thread Simon Horman
This patch series add some logics about rte_flow to ovs-dpdk, includes: * Support offload of set IPv4/IPv6 DSCP action * Enhanced the support of tunnel pop action Changes since v2 * Revise commit messages * Add the fix tag to patch 3/3 * Revise dscp masking to check for correct partial mask *

Re: [ovs-dev] [PATCH v3] daemon-unix: Fix file descriptor leak when monitor restarts child

2022-10-07 Thread Ilya Maximets
On 9/30/22 03:09, Fengqi Li wrote: > When segmentation fault occured in ovn-northd, monitor will try to > restart the ovn-northd daemon process every 10s. > Assume the following scenarios: There is a segmentation fault and > the ovn-northd daemon process doen not restart properly everytime. > New

Re: [ovs-dev] [PATCH] vconn: Allow ECONNREFUSED in refuse connection test

2022-10-07 Thread Ilya Maximets
On 9/28/22 11:00, Eelco Chaudron wrote: > > > On 27 Sep 2022, at 18:04, Mike Pattrick wrote: > >> The "tcp vconn - refuse connection" test may fail due to a Connection >> Refused error. The network stack returns ECONNREFUSED on a reset >> connection in SYN_SENT state and EPIPE or ECONNRESET in

Re: [ovs-dev] [PATCH v2] sparse: Add a guard for netinet/ip6.h header on FreeBSD.

2022-10-07 Thread Ilya Maximets
On 10/6/22 17:53, Mike Pattrick wrote: > On Mon, Sep 26, 2022 at 5:19 PM Ilya Maximets wrote: >> >> Same as arpa/inet.h, the netinet/ip6.h on FreeBSD requires >> netinet/in.h to be included first. So, adding a similar guard. >> >> Also fixing one instance where this is not respected at the

Re: [ovs-dev] [PATCH ovn 5/8] controller: Add support for templated actions and matches.

2022-10-07 Thread Dumitru Ceara
On 10/7/22 04:03, Han Zhou wrote: > On Fri, Sep 30, 2022 at 7:02 AM Dumitru Ceara wrote: >> >> Expand SB.Template_Var records in two stages: >> 1. first expand them to local values in match/action strings >> 2. then reparse the expanded strings >> >> For the case when a lflow references a

Re: [ovs-dev] [OVN v9] OVN - Add Support for Remote Port Mirroring

2022-10-07 Thread 0-day Robot
Bleep bloop. Greetings Abhiram R N, 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: WARNING: Line lacks whitespace around operator #2148 FILE: utilities/ovn-nbctl.c:275:

[ovs-dev] [OVN v9] OVN - Add Support for Remote Port Mirroring

2022-10-07 Thread Abhiram R N
Added changes in ovn-nbctl, ovn-sbctl, northd and in ovn-controller. While Mirror creation just creates the mirror, the lsp-attach-mirror triggers the sequence to create Mirror in OVS DB on compute node. OVS already supports Port Mirroring. Note: This is targeted to mirror to destinations