Re: [ovs-dev] [PATCH ovn] ovn-northd: Add the option to configure probe interval

2020-07-14 Thread Girish Moodalbail
Hello Numan, We found a minor issue with this patch in our OVN K8s setup. Say, we have set the northd_probe_interval to 50ms, and we have 3 instances of `ovn-northd` running. In this setup, one of the ovn-northd instances will be active, whilst the remaining two instances will be in 'standby' mode

[ovs-dev] Redacción de documentos y textos ejecutivos.

2020-07-14 Thread PDF - Paquete de 3 Cursos
El entorno laboral actual requiere de profesionistas capaces de expresarse por escrito de forma clara, coherente y con impecable ortografía. Este curso en línea está enfocado en abordar las principales reglas ortográficas del idioma español y en otorgar recomendaciones a los participantes para un

Re: [ovs-dev] [PATCH] ovs-bugtool: Fix Python3 bytes str issue.

2020-07-14 Thread William Tu
On Tue, Jul 14, 2020 at 10:14 AM Yi-Hung Wei wrote: > > On Mon, Jul 13, 2020 at 1:46 PM William Tu wrote: > > > > The patch fixes two errors due to type mismatched, when converting > > between str and bytes: > > File "/usr/local/sbin/ovs-bugtool", line 649, in main > > cmd_output(CAP_NETWOR

Re: [ovs-dev] [PATCH] rpm-fedora: Add missing dist library

2020-07-14 Thread William Tu
On Tue, Jul 14, 2020 at 03:40:39PM -0700, Gregory Rose wrote: > > > On 7/14/2020 1:49 PM, Greg Rose wrote: > >libopenvswitchavx512.a is needed for the fedora rpm spec. > > > >Signed-off-by: Greg Rose > >--- > > rhel/openvswitch-fedora.spec.in | 1 + > > 1 file changed, 1 insertion(+) > > > >dif

Re: [ovs-dev] (no subject)

2020-07-14 Thread Steve & Lenka Thomson
We are pleased to inform you that an amount of Ј1,,000.00 (GBP) has been donated and given to you and your family by Steve & Lenka Thomson Family ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH] rpm-fedora: Add missing dist library

2020-07-14 Thread Gregory Rose
On 7/14/2020 1:49 PM, Greg Rose wrote: libopenvswitchavx512.a is needed for the fedora rpm spec. Signed-off-by: Greg Rose --- rhel/openvswitch-fedora.spec.in | 1 + 1 file changed, 1 insertion(+) diff --git a/rhel/openvswitch-fedora.spec.in b/rhel/openvswitch-fedora.spec.in index 7bc8c34

[ovs-dev] [PATCH] rpm-fedora: Add missing dist library

2020-07-14 Thread Greg Rose
libopenvswitchavx512.a is needed for the fedora rpm spec. Signed-off-by: Greg Rose --- rhel/openvswitch-fedora.spec.in | 1 + 1 file changed, 1 insertion(+) diff --git a/rhel/openvswitch-fedora.spec.in b/rhel/openvswitch-fedora.spec.in index 7bc8c34..154b49e 100644 --- a/rhel/openvswitch-fedora

Re: [ovs-dev] make rpm-fedora broken on RHEL 7.7 - avx512 related?

2020-07-14 Thread Gregory Rose
On 7/14/2020 1:01 PM, Gregory Rose wrote: Greetings, I updated my repo to the latest master branch this morning and now when I try to run 'make rpm-fedora' I get the following error: Checking for unpackaged file(s): /usr/lib/rpm/check-files /home/gvrose/prj/ovs-experimental/_build/rpm/rpmbui

[ovs-dev] make rpm-fedora broken on RHEL 7.7 - avx512 related?

2020-07-14 Thread Gregory Rose
Greetings, I updated my repo to the latest master branch this morning and now when I try to run 'make rpm-fedora' I get the following error: Checking for unpackaged file(s): /usr/lib/rpm/check-files /home/gvrose/prj/ovs-experimental/_build/rpm/rpmbuild/BUILDROOT/openvswitch-2.13.90-1.el7.x86_64

Re: [ovs-dev] [PATCH] dpif-netdev: Avoid deadlock with offloading during PMD thread deletion.

2020-07-14 Thread Ilya Maximets
On 7/14/20 7:52 PM, Ilya Maximets wrote: > On 7/14/20 5:33 PM, Kevin Traynor wrote: >> On 09/07/2020 16:19, Ilya Maximets wrote: >>> Main thread will try to pause/stop all revalidators during datapath >>> reconfiguration via datapath purge callback (dp_purge_cb) while >>> holding 'dp->port_mutex'.

Re: [ovs-dev] [PATCH] dpif-netdev: Avoid deadlock with offloading during PMD thread deletion.

2020-07-14 Thread Ilya Maximets
On 7/14/20 5:33 PM, Kevin Traynor wrote: > On 09/07/2020 16:19, Ilya Maximets wrote: >> Main thread will try to pause/stop all revalidators during datapath >> reconfiguration via datapath purge callback (dp_purge_cb) while >> holding 'dp->port_mutex'. And deadlock happens in case any of >> revalid

Re: [ovs-dev] [PATCH] ovs-bugtool: Fix Python3 bytes str issue.

2020-07-14 Thread Yi-Hung Wei
On Mon, Jul 13, 2020 at 1:46 PM William Tu wrote: > > The patch fixes two errors due to type mismatched, when converting > between str and bytes: > File "/usr/local/sbin/ovs-bugtool", line 649, in main > cmd_output(CAP_NETWORK_STATUS, [OVS_DPCTL, 'dump-flows', '-m', d]) > File "/usr/local/

Re: [ovs-dev] RFC DEMO of using parallel processing in OVN

2020-07-14 Thread Anton Ivanov
On 14/07/2020 16:27, Dumitru Ceara wrote: On 7/10/20 10:39 AM, anton.iva...@cambridgegreys.com wrote: Hi all, This is a series of patches to demo the use in OVN of the parallel processing of hashes patchset which I submitted to OVS. The OVS patch series required for this patch can be found her

Re: [ovs-dev] [PATCH] dpif-netdev: Avoid deadlock with offloading during PMD thread deletion.

2020-07-14 Thread Kevin Traynor
On 09/07/2020 16:19, Ilya Maximets wrote: > Main thread will try to pause/stop all revalidators during datapath > reconfiguration via datapath purge callback (dp_purge_cb) while > holding 'dp->port_mutex'. And deadlock happens in case any of > revalidator threads is already waiting on 'dp->port_mu

Re: [ovs-dev] RFC DEMO of using parallel processing in OVN

2020-07-14 Thread Dumitru Ceara
On 7/10/20 10:39 AM, anton.iva...@cambridgegreys.com wrote: > Hi all, > > This is a series of patches to demo the use in OVN of the parallel > processing of hashes patchset which I submitted to OVS. > > The OVS patch series required for this patch can be found here: > > https://patchwork.ozlabs

Re: [ovs-dev] Dear Beneficiary !!!

2020-07-14 Thread AZIM PREMJI FOUNDATION
Congratulation! The Azim Premji Foundation donated $2,000,000.00 to you due to Covid-19, Get back for more details: -- This email has been checked for viruses by Avast antivirus software. https://www.avast.com/antivirus ___ dev mailing list d...@openv

Re: [ovs-dev] [PATCH 0/5] Stream Record/Replay.

2020-07-14 Thread Ilya Maximets
On 7/14/20 3:34 PM, Dumitru Ceara wrote: > On 6/30/20 3:24 AM, Ilya Maximets wrote: >> This patch set adds new stream provider and other functionality in >> order to record all the incoming data on all the steams (ssl, tcp, >> unixctl) of openvswitch library based applications and replay these >> s

Re: [ovs-dev] [PATCH 2/3] netdev-offload-dpdk: Pass L4 proto-id to match in the L3 rte_flow_item

2020-07-14 Thread Eli Britstein
Acked-by: Eli Britstein On 7/10/2020 3:07 PM, Sriharsha Basavapatna wrote: The offload layer clears the L4 protocol mask in the L3 item, when the L4 item is passed for matching, as an optimization. This can be confusing while parsing the headers in the PMD. Also, the datapath flow specifies thi

Re: [ovs-dev] [PATCH 0/3] Remove duplicated includes

2020-07-14 Thread William Tu
On Tue, Jul 14, 2020 at 2:45 AM wangyunjian wrote: > > > -Original Message- > > From: William Tu [mailto:u9012...@gmail.com] > > Sent: Tuesday, July 14, 2020 5:52 AM > > To: wangyunjian > > Cc: d...@openvswitch.org; Lilijun (Jerry) ; > > i.maxim...@ovn.org > > Subject: Re: [ovs-dev] [PATC

[ovs-dev] Questions about flow put failed(File exist)

2020-07-14 Thread Yutao (Simon, Cloud Infrastructure Service Product Dept.)
Hi all, The ovs ran into a problem in our environment: Flow continuous to upcall , because of flow_put_on_pmd failed. Steps to trigger this problem: Step1 Add two openflow rules: #1. ovs-ofctl add-flow br-forward "table=0,in_port=1,dl_dst:3E:5B:51:81:5A:2F,ip,nw_dst=123.1.1.1/32,priorit

Re: [ovs-dev] [PATCH] vswitchd: Pass MCL_ONFAULT to mlockall

2020-07-14 Thread William Tu
On Tue, Jul 14, 2020 at 6:32 AM Flavio Leitner wrote: > > On Mon, Jul 13, 2020 at 02:49:50PM -0700, William Tu wrote: > > On Fri, Jul 10, 2020 at 12:12:57PM -0300, Flavio Leitner wrote: > > > On Fri, Jul 10, 2020 at 12:53:13PM +0100, Ross Lagerwall wrote: > > > > mlockall locks thread stack pages

Re: [ovs-dev] [PATCH 2/5] stream: Add record/replay functionality.

2020-07-14 Thread Dumitru Ceara
On 6/30/20 3:24 AM, Ilya Maximets wrote: > For debugging purposes it is useful to be able to record all the > incoming transactions and commands and replay them locally under > debugger or with additional logging enabled. This patch introduces > ability to record all the incoming stream data and r

Re: [ovs-dev] [PATCH 3/5] ovsdb-server: Integrate stream replay engine.

2020-07-14 Thread Dumitru Ceara
On 6/30/20 3:24 AM, Ilya Maximets wrote: > This change adds support of stream record/replay functionality to > ovsdb-server. > > Since current replay engine doesn't work well with time-based > events generated locally, it will work only with standalone databases > for now (raft heavily depends on

Re: [ovs-dev] [PATCH 0/5] Stream Record/Replay.

2020-07-14 Thread Dumitru Ceara
On 6/30/20 3:24 AM, Ilya Maximets wrote: > This patch set adds new stream provider and other functionality in > order to record all the incoming data on all the steams (ssl, tcp, > unixctl) of openvswitch library based applications and replay these > streams later for debugging purposes or performa

Re: [ovs-dev] [PATCH] vswitchd: Pass MCL_ONFAULT to mlockall

2020-07-14 Thread Flavio Leitner
On Mon, Jul 13, 2020 at 02:49:50PM -0700, William Tu wrote: > On Fri, Jul 10, 2020 at 12:12:57PM -0300, Flavio Leitner wrote: > > On Fri, Jul 10, 2020 at 12:53:13PM +0100, Ross Lagerwall wrote: > > > mlockall locks thread stack pages into memory, even pages which have not > > > yet been demand-page

Re: [ovs-dev] [PATCH v3] checkpatch: Add argument to skip gerrit change id check

2020-07-14 Thread Flavio Leitner
On Tue, Jul 14, 2020 at 10:24:41AM +0300, Roi Dayan wrote: > This arg can be used internally by groups using gerrit for code reviews. > > Signed-off-by: Roi Dayan > Signed-off-by: Ben Pfaff > --- Acked-by: Flavio Leitner ___ dev mailing list d...@op

Re: [ovs-dev] [PATCH 0/3] Remove duplicated includes

2020-07-14 Thread wangyunjian
> -Original Message- > From: William Tu [mailto:u9012...@gmail.com] > Sent: Tuesday, July 14, 2020 5:52 AM > To: wangyunjian > Cc: d...@openvswitch.org; Lilijun (Jerry) ; > i.maxim...@ovn.org > Subject: Re: [ovs-dev] [PATCH 0/3] Remove duplicated includes > > On Fri, Jul 10, 2020 at 08:57:

Re: [ovs-dev] [PATCH v3] checkpatch: Add argument to skip gerrit change id check

2020-07-14 Thread Roi Dayan
On 2020-07-14 10:24 AM, Roi Dayan wrote: > This arg can be used internally by groups using gerrit for code reviews. > > Signed-off-by: Roi Dayan > Signed-off-by: Ben Pfaff > --- > > Notes: > v3 > - use line length 79 > > utilities/checkpatch.py | 10 -- > 1 file changed, 8

Re: [ovs-dev] [PATCH v3] ovsdb-tool: Add a db consistency check to the ovsdb-tool check-cluster command

2020-07-14 Thread Dumitru Ceara
On 7/10/20 10:18 AM, Federico Paolinelli wrote: > There are some occurrences where the database ends up in an inconsistent > state. This happened in ovn-k8s and is described in [0]. > Here we are adding a supported way to check that a given db is consistent, > which is less error prone than checkin

Re: [ovs-dev] [PATCH v3] checkpatch: Add argument to skip gerrit change id check

2020-07-14 Thread 0-day Robot
Bleep bloop. Greetings Roi Dayan, 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 sign-offs from developers who are not authors or co-authors or committers: Ben

Re: [ovs-dev] [PATCH] checkpatch: Add argument to skip gerrit change id check

2020-07-14 Thread Roi Dayan
On 2020-07-09 9:30 PM, Ilya Maximets wrote: > On 7/9/20 5:09 PM, Flavio Leitner wrote: >> On Sun, Jun 14, 2020 at 07:03:03PM +0300, Roi Dayan wrote: >>> This arg can be used internally by groups using gerrit for code reviews. >> >> The patch looks good, but there is the condition line >> that go

[ovs-dev] [PATCH v3] checkpatch: Add argument to skip gerrit change id check

2020-07-14 Thread Roi Dayan
This arg can be used internally by groups using gerrit for code reviews. Signed-off-by: Roi Dayan Signed-off-by: Ben Pfaff --- Notes: v3 - use line length 79 utilities/checkpatch.py | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/utilities/checkpatch.py