Re: [ovs-dev] [PATCH v5 2/3] ovsdb-idl: Avoid inconsistent IDL state with OVSDB_MONITOR_V3.

2020-05-27 Thread Han Zhou
On Wed, May 27, 2020 at 2:03 AM Dumitru Ceara wrote: > > On 5/27/20 3:41 AM, Han Zhou wrote: > > Thanks Dumitru. Please see my comments below. > > > > On Thu, May 7, 2020 at 4:21 AM Dumitru Ceara > > wrote: > >> > >> Assuming an ovsdb client connected to a database using

[ovs-dev] DARLEHEN

2020-05-27 Thread emanuel . isaque
Benötigen Sie einen zuverlässigen Kredit? 3% Rate wenden mehr Details an ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

[ovs-dev] le bon moment

2020-05-27 Thread Benjamin Thierry
ouverte à tous ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

[ovs-dev] Protocolo y recomendaciones a seguir

2020-05-27 Thread Retorno al trabajo seguro - Protocolos a seguir
Buenos día Quise aprovechar la oportunidad de hacerte una invitación para tomar nuestro curso: Nombre: Retorno al trabajo seguro: Protocolo y recomendaciones. Horario: de 10:00 a 14:00 Hrs. ¿Cuándo?: Jueves 18 de Junio Formato: En línea con interacción en vivo. Lugar: En Vivo desde su computa

Re: [ovs-dev] [PATCH v4] classifier: Prevent tries vs n_tries race leading to NULL dereference.

2020-05-27 Thread Eiichi Tsukata
Hello Ilya > On May 28, 2020, at 7:28, Ilya Maximets wrote: > > On 5/27/20 4:13 AM, Eiichi Tsukata wrote: >> Currently classifier tries and n_tries can be updated not atomically, >> there is a race condition which can lead to NULL dereference. >> The race can happen when main thread updates a cl

Re: [ovs-dev] [PATCH v4] classifier: Prevent tries vs n_tries race leading to NULL dereference.

2020-05-27 Thread Ilya Maximets
On 5/27/20 4:13 AM, Eiichi Tsukata wrote: > Currently classifier tries and n_tries can be updated not atomically, > there is a race condition which can lead to NULL dereference. > The race can happen when main thread updates a classifier tries and > n_tries in classifier_set_prefix_fields() and at

Re: [ovs-dev] [PATCH] pvector: Document that elements with a given priority are allowed.

2020-05-27 Thread Ben Pfaff
On Wed, May 27, 2020 at 01:13:34PM -0700, Gregory Rose wrote: > > On 5/27/2020 12:11 PM, Ben Pfaff wrote: > > Signed-off-by: Ben Pfaff > > --- > > lib/pvector.h | 8 +--- > > 1 file changed, 5 insertions(+), 3 deletions(-) > > > > diff --git a/lib/pvector.h b/lib/pvector.h > > index 0d329

Re: [ovs-dev] Hello Confirm If It's True, From Mr. Joseph Brooks

2020-05-27 Thread Joseph
*Attention Please,* *This is to officially inform you that A power of attorney was forwarded to our office this morning by two gentle men, one of them is an American national and he is MR. PAUL DEANE by name while the other person is MR. JACK MORGAN by name a CANADIAN national. This gentleman cla

Re: [ovs-dev] [PATCH] pvector: Document that elements with a given priority are allowed.

2020-05-27 Thread Gregory Rose
On 5/27/2020 12:11 PM, Ben Pfaff wrote: Signed-off-by: Ben Pfaff --- lib/pvector.h | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/lib/pvector.h b/lib/pvector.h index 0d3290dc373b..3b0c312182d2 100644 --- a/lib/pvector.h +++ b/lib/pvector.h @@ -27,9 +27,11 @@

Re: [ovs-dev] [PATCH v2 ovn] northd: manage ARP request locally for FIP traffic

2020-05-27 Thread Han Zhou
On Wed, May 27, 2020 at 3:18 AM Dumitru Ceara wrote: > > On 5/26/20 6:46 PM, Lorenzo Bianconi wrote: > > Modify 100-priority logical flows in Gateway Redirect table of > > logical router ingress pipeline (table 15) in order to manage ARP > > request locally for FIP traffic. In particular set reg1

[ovs-dev] [PATCH] meta-flow: Document that constituents of conjunctive flows may overlap.

2020-05-27 Thread Ben Pfaff
Suggested-by: Antonin Bas Signed-off-by: Ben Pfaff --- lib/meta-flow.xml | 2 ++ manpages.mk | 3 --- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/lib/meta-flow.xml b/lib/meta-flow.xml index d4495552b3c2..1546758744b4 100644 --- a/lib/meta-flow.xml +++ b/lib/meta-flow.xml

[ovs-dev] [PATCH] pvector: Document that elements with a given priority are allowed.

2020-05-27 Thread Ben Pfaff
Signed-off-by: Ben Pfaff --- lib/pvector.h | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/lib/pvector.h b/lib/pvector.h index 0d3290dc373b..3b0c312182d2 100644 --- a/lib/pvector.h +++ b/lib/pvector.h @@ -27,9 +27,11 @@ * == * * Concurrent

Re: [ovs-dev] [PATCH V2 06/12] netdev-offload: Use dpif type instead of class.

2020-05-27 Thread 0-day Robot
Bleep bloop. Greetings Eli Britstein, 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

Re: [ovs-dev] [PATCH V2 05/12] netdev: Allow storing dpif type into netdev structure.

2020-05-27 Thread 0-day Robot
Bleep bloop. Greetings Eli Britstein, 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

[ovs-dev] [PATCH V2 11/12] netdev-offload-dpdk: Add testpmd log commands

2020-05-27 Thread Eli Britstein
To enhance debugability, add testpmd commands to the log. Signed-off-by: Eli Britstein Reviewed-by: Roni Bar Yanai --- lib/netdev-offload-dpdk.c | 217 +++--- 1 file changed, 204 insertions(+), 13 deletions(-) diff --git a/lib/netdev-offload-dpdk.c b/lib

[ovs-dev] [PATCH V2 12/12] netdev-offload-dpdk: Fix Ethernet matching for type only

2020-05-27 Thread Eli Britstein
For OVS rule of the form "eth type is 0x1234 / end", rule is offloaded in the form of "eth / end", which is incorrect. Fix it. Fixes: e8a2b5bf92bb ("netdev-dpdk: implement flow offload with rte flow") Signed-off-by: Eli Britstein Reviewed-by: Roni Bar Yanai --- lib/netdev-offload-dpdk.c | 5 +++

[ovs-dev] [PATCH V2 09/12] dpif-netdev: Don't use zero flow mark

2020-05-27 Thread Eli Britstein
Zero flow mark is used to indicate the HW to remove the mark. A packet marked with zero mark is received in SW without a mark at all, so it cannot be used as a valid mark. Change the pool range to fix it. Fixes: 241bad15d99a ("dpif-netdev: associate flow with a mark id") Signed-off-by: Eli Britste

[ovs-dev] [PATCH V2 06/12] netdev-offload: Use dpif type instead of class.

2020-05-27 Thread Eli Britstein
From: Ilya Maximets There is no real difference between the 'class' and 'type' in the context of common lookup operations inside netdev-offload module because it only checks the value of pointers without using the value itself. However, 'type' has some meaning and can be used by offload provides

[ovs-dev] [PATCH V2 02/12] netdev-offload-dpdk: Add IPv6 pattern matching

2020-05-27 Thread Eli Britstein
Add support for IPv6 pattern matching for offloading flows. Signed-off-by: Eli Britstein Reviewed-by: Roni Bar Yanai --- Documentation/howto/dpdk.rst | 2 +- NEWS | 1 + lib/netdev-offload-dpdk.c| 76 3 files changed, 78

[ovs-dev] [PATCH V2 05/12] netdev: Allow storing dpif type into netdev structure.

2020-05-27 Thread Eli Britstein
From: Ilya Maximets Storing of the dpif type of the owning datapath interface will allow us to easily distinguish, for example, userspace tunneling ports from the system ones. This is required in terms of HW offloading to avoid offloading of userspace flows to kernel interfaces that doesn't belo

[ovs-dev] [PATCH V2 07/12] netdev-offload-dpdk: Support offload of clone tnl_push/output actions

2020-05-27 Thread Eli Britstein
Signed-off-by: Eli Britstein Reviewed-by: Oz Shlomo --- Documentation/howto/dpdk.rst | 1 + NEWS | 2 +- lib/netdev-offload-dpdk.c| 57 3 files changed, 59 insertions(+), 1 deletion(-) diff --git a/Documentation/howto/dp

[ovs-dev] [PATCH V2 08/12] netdev-offload-dpdk: Support tnl/push using vxlan encap attribute

2020-05-27 Thread Eli Britstein
For DPDK, there is the RAW_ENCAP attribute which gets raw buffer of the encapsulation header. For specific protocol, such as vxlan, there is a more specific attribute, VXLAN_ENCAP, which gets the parsed fields of the outer header. In case tunnel type is vxlan, parse the header and use the specific

[ovs-dev] [PATCH V2 10/12] dpif-netdev: Add mega ufid in flow add log

2020-05-27 Thread Eli Britstein
As offload is done using the mega ufid of a flow, for better debugability, add it in the log message. Signed-off-by: Eli Britstein Reviewed-by: Roni Bar Yanai --- lib/dpif-netdev.c | 7 +-- tests/dpif-netdev.at| 2 ++ tests/ofproto-macros.at | 3 ++- 3 files changed, 9 insertions(

[ovs-dev] [PATCH V2 00/12] netdev datapath offload: Support IPv6 and VXLAN encap

2020-05-27 Thread Eli Britstein
This patch set includes additional offloads - IPv6 and VXLAN encap, and enhanced logging to increase debugability. Patches #1-#8: Add support for offloads of IPv6 patterns, partial TCP/UDP ports, set IPv6 and encap actions (clone/output). Patch #9:Bug fi

[ovs-dev] [PATCH V2 03/12] netdev-offload-dpdk: Support offload of set IPv6 actions

2020-05-27 Thread Eli Britstein
Signed-off-by: Eli Britstein Reviewed-by: Roni Bar Yanai --- Documentation/howto/dpdk.rst | 1 + NEWS | 2 ++ lib/netdev-offload-dpdk.c| 35 +++ 3 files changed, 38 insertions(+) diff --git a/Documentation/howto/dpdk.rst b/Documentat

[ovs-dev] [PATCH V2 01/12] netdev-offload-dpdk: Remove pre-validate of patterns function

2020-05-27 Thread Eli Britstein
The function of adding patterns by requested matches checks that it consumed all the required matches, and err if not. This nullify the purpose of the validation function. Future supported matches will only change the pattern parsing code. Signed-off-by: Eli Britstein Reviewed-by: Roni Bar Yanai

[ovs-dev] [PATCH V2 04/12] netdev-offload-dpdk: Support partial TCP/UDP port matching

2020-05-27 Thread Eli Britstein
The cited commit failed partial matching of TCP/UDP port matching, preventing such offload of supporting HWs. Remove this failure. Fixes: e8a2b5bf92bb ("netdev-dpdk: implement flow offload with rte flow") Signed-off-by: Eli Britstein Reviewed-by: Roni Bar Yanai --- lib/netdev-offload-dpdk.c | 5

Re: [ovs-dev] [PATCH 1/1] vswitchd: Allow setting MAC on DPDK interfaces

2020-05-27 Thread Ilya Maximets
On 4/20/20 9:26 AM, Roni Bar Yanai wrote: > Hi Ben, Ilya > > Going back to this thread. We've tried app-ctl approach and it fails on > consistency > problem. Orchestrator can configure on full system init, but now executing > local > restart will lose the configuration (again for bifurcated dri

Re: [ovs-dev] [PATCH ovn v8 1/8] ovn-controller: Store the local port bindings in the runtime data I-P state.

2020-05-27 Thread Dumitru Ceara
On 5/26/20 2:28 PM, num...@ovn.org wrote: > From: Numan Siddique > > This patch adds a new data structure - 'struct local_binding' which represents > a probable local port binding. A new object of this structure is creared for > each interface present in the integration bridge (br-int) with the >

Re: [ovs-dev] [RFC dpdk-latest 1/1] netdev-dpdk: integrate dpdk vhost pmd

2020-05-27 Thread Maxime Coquelin
Hi Siva, On 5/19/20 1:49 PM, Sivaprasad Tummala wrote: > The vHost PMD brings vHost User port types ('dpdkvhostuser' and > 'dpdkvhostuserclient') under control of DPDK's librte_ether API, like > other DPDK netdev types ('dpdk'). In doing so, direct > calls to DPDK's librte_vhost library are remove

Re: [ovs-dev] [PATCH ovn] ovn-northd: Address scale issues with DNAT flows.

2020-05-27 Thread Tomáš Sapák
On Wed, May 27, 2020 at 3:45 PM Numan Siddique wrote: > > > On Wed, May 27, 2020 at 7:07 PM Tomas Sapak wrote: > >> Hi all. >> We are experiencing the issue from this thread. We have ~15 flows, >> DVR enabled and ovn-controller and sbdb constantly running 100% CPU. >> Will this patch be back

Re: [ovs-dev] [PATCH dpdk-latest] sparse: Update to DPDK 20.05 trace point header.

2020-05-27 Thread Stokes, Ian
On 5/27/2020 1:26 PM, David Marchand wrote: A simplication happened in the trace point framework in 20.05-rc2. Align to the header renaming. Thanks for this David, looks simple enough, just giving it a run in travis, all going well will rebase dpdk-latest on master with this applied and pu

[ovs-dev] Spende

2020-05-27 Thread Mrs.Judith Rice
Attn: Es tut uns leid, dass wir Sie aufgrund eines Mismanagent of Beneficaries-Fonds von unseren ernannten Zonal Managern verspätet kontaktiert haben. Bitte beachten Sie, dass Sie qualifiziert sind, die Zahlung von 900.000,00 USD an der ATM-Karte mit neunhunderttausend Dollar zu erhalten. Als

Re: [ovs-dev] [PATCH ovn] ovn-northd: Address scale issues with DNAT flows.

2020-05-27 Thread Numan Siddique
On Wed, May 27, 2020 at 7:07 PM Tomas Sapak wrote: > Hi all. > We are experiencing the issue from this thread. We have ~15 flows, > DVR enabled and ovn-controller and sbdb constantly running 100% CPU. > Will this patch be backported to 2.12 branch, or do we need to upgrade > to OVS 2.13/ OVN

Re: [ovs-dev] [PATCH ovn] ovn-northd: Address scale issues with DNAT flows.

2020-05-27 Thread Tomas Sapak
Hi all. We are experiencing the issue from this thread. We have ~15 flows, DVR enabled and ovn-controller and sbdb constantly running 100% CPU. Will this patch be backported to 2.12 branch, or do we need to upgrade to OVS 2.13/ OVN 20.03? Kind regards Tomas __

[ovs-dev] Spende

2020-05-27 Thread Mrs.Judith Rice
Attn: Es tut uns leid, dass wir Sie aufgrund eines Mismanagent of Beneficaries-Fonds von unseren ernannten Zonal Managern verspätet kontaktiert haben. Bitte beachten Sie, dass Sie qualifiziert sind, die Zahlung von 900.000,00 USD an der ATM-Karte mit neunhunderttausend Dollar zu erhalten. Als

[ovs-dev] [PATCH dpdk-latest] sparse: Update to DPDK 20.05 trace point header.

2020-05-27 Thread David Marchand
A simplication happened in the trace point framework in 20.05-rc2. Align to the header renaming. Note: Can this be squashed with the previous fix at the next dpdk-latest rebase? Signed-off-by: David Marchand --- include/sparse/automake.mk| 2 +- .../sparse/{r

Re: [ovs-dev] [PATCH v2 5/5] dpif-lookup: add avx512 gather implementation

2020-05-27 Thread Van Haaren, Harry
> -Original Message- > From: dev On Behalf Of Van Haaren, Harry > Sent: Tuesday, May 26, 2020 3:52 PM > To: William Tu > Cc: ovs-dev@openvswitch.org; i.maxim...@ovn.org > Subject: Re: [ovs-dev] [PATCH v2 5/5] dpif-lookup: add avx512 gather > implementation > > Why ukey is related here?

Re: [ovs-dev] [PATCH ovn] northd: Add the new chassis's "other_config" column to RBAC

2020-05-27 Thread Numan Siddique
On Wed, May 27, 2020 at 4:59 PM Dumitru Ceara wrote: > On 5/27/20 1:04 PM, lmart...@redhat.com wrote: > > From: Lucas Alvares Gomes > > > > This patch is fixing a left-over from commit > > 74d90c2223d0a8c123823fb849b4c2de58c296e4, we need to add the new > > "other_config" column from the Chassis

[ovs-dev] Mobile Application to Manage your Sales Team

2020-05-27 Thread Aishwarya via dev
Hello, Greetings from Travelize!!! Reaching out as it seems you oversee digital strategy for your Company. I want to Introduce "Travelize" as a Business Management solution that allows you to keep track of your Field Staff. Travelize has a lot to Offer, 1. Remote Attendance using GPS

Re: [ovs-dev] [PATCH ovn] northd: Add the new chassis's "other_config" column to RBAC

2020-05-27 Thread Dumitru Ceara
On 5/27/20 1:04 PM, lmart...@redhat.com wrote: > From: Lucas Alvares Gomes > > This patch is fixing a left-over from commit > 74d90c2223d0a8c123823fb849b4c2de58c296e4, we need to add the new > "other_config" column from the Chassis table to the rbac_chassis_update > list for northd. > > Co-Autho

[ovs-dev] [PATCH ovn] northd: Add the new chassis's "other_config" column to RBAC

2020-05-27 Thread lmartins
From: Lucas Alvares Gomes This patch is fixing a left-over from commit 74d90c2223d0a8c123823fb849b4c2de58c296e4, we need to add the new "other_config" column from the Chassis table to the rbac_chassis_update list for northd. Co-Authored-by: Numan Siddique Signed-off-by: Numan Siddique Signed-o

[ovs-dev] Windows OVS datapath conntrack support CTA_HELP and CTA_TUPLE_MASTER

2020-05-27 Thread Jinjun Gao
Hi Alin/Guys, We have an ALG feature that needs Windows OVS conntrack to support CTA_HELP and CTA_TUPLE_MASTER attributes. I noticed these two attributes has not been implemented in current Windows OVS datapath. Do you know the context why them are not implemented at original design and impleme

Re: [ovs-dev] [PATCH v2 ovn] northd: manage ARP request locally for FIP traffic

2020-05-27 Thread Dumitru Ceara
On 5/26/20 6:46 PM, Lorenzo Bianconi wrote: > Modify 100-priority logical flows in Gateway Redirect table of > logical router ingress pipeline (table 15) in order to manage ARP > request locally for FIP traffic. In particular set reg1 and eth.src > to NAT external ip and NAT external mac respective

Re: [ovs-dev] [PATCH v5 2/3] ovsdb-idl: Avoid inconsistent IDL state with OVSDB_MONITOR_V3.

2020-05-27 Thread Dumitru Ceara
On 5/27/20 3:41 AM, Han Zhou wrote: > Thanks Dumitru. Please see my comments below. > > On Thu, May 7, 2020 at 4:21 AM Dumitru Ceara > wrote: >> >> Assuming an ovsdb client connected to a database using OVSDB_MONITOR_V3 >> (i.e., "monitor_cond_since" method) with the ini

Re: [ovs-dev] [PATCH ovn v7 8/9] ovn-controller: Use the tracked runtime data changes for flow calculation.

2020-05-27 Thread Numan Siddique
On Wed, May 27, 2020 at 3:30 AM Han Zhou wrote: > On Tue, May 26, 2020 at 6:57 AM Numan Siddique wrote: > > > > > > > > On Thu, May 21, 2020 at 12:42 PM Han Zhou wrote: > >> > >> On Wed, May 20, 2020 at 12:40 PM wrote: > >> > > >> > From: Venkata Anil > >> > > >> > This patch processes the lo

[ovs-dev] [PATCH v3] system-dpdk: add negotiation check for userspace-tso

2020-05-27 Thread Gowrishankar Muthukrishnan
This patch adds minimal check for userspace-tso in system-dpdk tests, starting with verification on virtio negotiation. Signed-off-by: Gowrishankar Muthukrishnan --- v3: - Removed other improvement(cleanup, optimization in general) for now and sending only tso tests. Other improvements will b

Re: [ovs-dev] [PATCH v2] datapath: Add hash info to upcall.

2020-05-27 Thread aginwala
Thanks Han for backporting it for lower kernels. I tested this patch with some running workloads with old 3.13* kernels (ovs 2.11*) and it works fine. In addition I am also considering the discussion [1], and for ecmp workloads I see all packets are hitting the different bucket instead of the sam