[ovs-dev] READ AND GET BACK TO ME

2020-07-16 Thread GUARANTY
Attention:. You refused to believe me that is the problem between me and you. I want you to think about this things that am about to tell you right now and get back to me after reading. 1. Please Remember How Much you have spent so far for this business. 2.. Remember how many years wasted 3.

Re: [ovs-dev] [ovs-dev v2] ofproto: add refcount to ofproto to fix crash at ofproto_destroy

2020-07-16 Thread 0-day Robot
Bleep bloop. Greetings hepeng.0320, 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: Inappropriate bracing around statement #182 FILE: ofproto/ofproto.c:1734: if

[ovs-dev] [ovs-dev v2] ofproto: add refcount to ofproto to fix crash at ofproto_destroy

2020-07-16 Thread hepeng.0320
From: Peng He The call stack of rule_destroy_cb: remove_rules_postponed (one grace period) -> remove_rule_rcu -> remove_rule_rcu__ -> ofproto_rule_unref -> ref count != 1 -> ... more grace periods. -> rule_destroy_cb (> 2 grace periods)

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

2020-07-16 Thread Ilya Maximets
On 7/14/20 3:03 PM, Flavio Leitner wrote: > 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 > Thanks! Applied.

Re: [ovs-dev] [PATCH] acinclude: Remove libmnl for MLX5 PMD

2020-07-16 Thread Ilya Maximets
On 6/25/20 2:13 PM, Thomas Monjalon wrote: > 25/06/2020 13:28, Timothy Redaelli: >> On Thu, 25 Jun 2020 09:32:23 +0200 >> Thomas Monjalon wrote: >> >>> On Tue, Jun 23, 2020 at 6:49 PM Timothy Redaelli >>> wrote: libmnl is not used anymore for MLX5 PMD since DPDK 19.08. >>> >>> Why

Re: [ovs-dev] [PATCH v12] AB bonding: Add "primary" interface concept

2020-07-16 Thread Ilya Maximets
On 7/10/20 4:20 PM, Flavio Leitner wrote: > On Thu, Jul 09, 2020 at 04:57:47PM -0700, Jeff Squyres via dev wrote: >> In AB bonding, if the current active slave becomes disabled, a >> replacement slave is arbitrarily picked from the remaining set of >> enabled slaves. This commit adds the concept

Re: [ovs-dev] [PATCH v2] dpdk: Add commands to configure log levels.

2020-07-16 Thread Ilya Maximets
On 7/13/20 10:06 AM, David Marchand wrote: > Enabling debug logs in dpdk can be a challenge to be sure of what is > actually enabled, add commands to list and change those log levels. > However, these commands do not help when tracking issues in dpdk init > itself: dump log levels right after

Re: [ovs-dev] [PATCH] rhel: openvswitch-fedora.spec.in: Fix installed but not packaged

2020-07-16 Thread Ilya Maximets
On 7/16/20 2:32 PM, Stokes, Ian wrote: >> With the cited commit, we get an error from rpmbuild about installed but not >> packaged /usr/lib64/libopenvswitchavx512.a. >> Fix it by treating it as the other la files. >> >> Fixes: 352b6c7116cd ("dpif-lookup: add avx512 gather implementation.") >>

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

2020-07-16 Thread Ilya Maximets
On 7/15/20 6:00 PM, 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

[ovs-dev] Medición y estrategias de mejora

2020-07-16 Thread Engagement laboral
Buenas Tardes Quise aprovechar la oportunidad de hacerte una invitación para tomar nuestro curso: Nombre:Engagement laboral: Medición y estrategias de mejora. Horario: de 10:00 am a 17:00 pm. ¿Cuándo?: Miércoles 12 de Agosto. Formato: En línea con interacción en vivo. Lugar: En Vivo desde su

[ovs-dev] [PATCH ovn] Allow force_snat options to work for dual-stack routers.

2020-07-16 Thread Mark Michelson
The lb_force_snat and dnat_force_snat options could accept only a single IP address. For routers that only route traffic of a single IP address family, this is fine. However, if a router routes both IPv4 and IPv6 traffic, then this limitation is a problem. This patch addresses this problem by

[ovs-dev] OVN Meeting Logs 16 July, 2020

2020-07-16 Thread mmichels
Here is the IRC log for the OVN meeting for 16 July, 2020 http://eavesdrop.openstack.org/meetings//ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-07-16-17.18.log.html If you are interested in attending this meeting, it happens every Thursday in the

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

2020-07-16 Thread Ilya Maximets
On 7/14/20 3:35 PM, Dumitru Ceara wrote: > 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

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

2020-07-16 Thread Ilya Maximets
On 7/14/20 3:35 PM, Dumitru Ceara wrote: > 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 >>

Re: [ovs-dev] [PATCH] rhel: openvswitch-fedora.spec.in: Fix installed but not packaged

2020-07-16 Thread Gregory Rose
On 7/15/2020 5:40 AM, Roi Dayan wrote: With the cited commit, we get an error from rpmbuild about installed but not packaged /usr/lib64/libopenvswitchavx512.a. Fix it by treating it as the other la files. Fixes: 352b6c7116cd ("dpif-lookup: add avx512 gather implementation.") Signed-off-by:

Re: [ovs-dev] [PATCH v3 2/4] netdev-offload: Add "offload-driver" other_config to specify offload driver

2020-07-16 Thread William Tu
On Mon, Jun 29, 2020 at 8:30 AM Toshiaki Makita wrote: > > The following commit will introduce another offload driver using XDP. > When using afxdp netdev, both of TC and XDP will be supported, so let's > add an other_config to specify which offload driver is preferable. > When not specified and

Re: [ovs-dev] [PATCH] ofproto: add refcount to ofproto to fix crash at ofproto_destroy

2020-07-16 Thread 0-day Robot
Bleep bloop. Greetings hepeng.0320, 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: Inappropriate bracing around statement #182 FILE: ofproto/ofproto.c:1734: } else

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

2020-07-16 Thread Eli Britstein
Tested-by: Eli Britstein I was able to reproduce (saw by gdb the points in the deadlock). Fixed by this commit. On 7/16/2020 3:20 PM, Ilya Maximets wrote: On 7/16/20 12:25 PM, Sriharsha Basavapatna wrote: On Thu, Jul 16, 2020 at 2:52 PM Ilya Maximets wrote: On 7/16/20 10:01 AM, Eli

Re: [ovs-dev] [PATCH ovs-dev v1] netdev-offload: Flush offloaded rules when ports removed

2020-07-16 Thread William Tu
> > > > Hi > > > > Thanks for your review. I test it with openflow, It works fine because > > > > the *revalidator > > > > will clean up the flows(which may be offloaded). > > > > > In case above scenario leaves ufids in OVS that cannot be removed, > > > > > I'd suggest > > > > > reverting of the

[ovs-dev] [PATCH] ofproto: add refcount to ofproto to fix crash at ofproto_destroy

2020-07-16 Thread hepeng.0320
From: Peng He The call stack of rule_destroy_cb: remove_rules_postponed (one grace period) -> remove_rule_rcu -> remove_rule_rcu__ -> ofproto_rule_unref -> ref count != 1 -> ... more grace periods. -> rule_destroy_cb (> 2 grace periods)

Re: [ovs-dev] [PATCH v3 1/4] netdev-afxdp: Enable loading XDP program.

2020-07-16 Thread Toshiaki Makita
On 2020/07/16 10:30, William Tu wrote: On Mon, Jun 29, 2020 at 8:30 AM Toshiaki Makita wrote: From: William Tu Now netdev-afxdp always forwards all packets to userspace because it is using libbpf's default XDP program, see 'xsk_load_xdp_prog'. There are some cases when users want to keep

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

2020-07-16 Thread Toshiaki Makita
Hi William, Thank you for taking a look! On 2020/07/16 0:49, William Tu wrote: Hi Toshiaki, Thanks for the interesting patch series! I haven't finished reviewing, but I went through a couple of steps to make sure it works in my environment. Comments below: On Tue, Jun 30, 2020 at 12:30:29AM

Re: [ovs-dev] [PATCH v6] Bareudp Tunnel Support

2020-07-16 Thread Martin Varghese
On Fri, Jul 10, 2020 at 07:20:02PM +0200, Ilya Maximets wrote: > On 6/29/20 3:31 PM, Martin Varghese wrote: > > From: Martin Varghese > > > > There are various L3 encapsulation standards using UDP being discussed to > > leverage the UDP based load balancing capability of different networks. > >

[ovs-dev] [PATCH ovn RFC v3 29/29] Set cut-off values to best guesstimates

2020-07-16 Thread anton . ivanov
From: Anton Ivanov This sets the cut-off values for switching between parallel and sequential processing to a reasonable guesstimate. Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/northd/ovn-northd.c

[ovs-dev] [PATCH ovn RFC v3 11/29] Move ARP request to a separate rule function

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 139 1 file changed, 75 insertions(+), 64 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index 81a6407a9..6b7a57c41 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 28/29] Parallel reconciliation of southdb flows

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 196 1 file changed, 178 insertions(+), 18 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index a53b31906..ef188fb88 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 15/29] Split port security into two separate functions

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 106 +--- 1 file changed, 61 insertions(+), 45 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index d00fef94d..878cfb493 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 20/29] Pull DNS response defaults into a separate function

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 36 1 file changed, 20 insertions(+), 16 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index da0ee4509..e5d64993a 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 10/29] Move GW redirect rules to a new function

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 80 ++--- 1 file changed, 46 insertions(+), 34 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index 0b4926a75..81a6407a9 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 17/29] Pull out lswitch ARP/ND respond, known IPs into functions

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 196 +--- 1 file changed, 112 insertions(+), 84 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index 13b3793fc..2226f5882 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 21/29] Pull external port ARP responses into a separate function

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 31 +++ 1 file changed, 19 insertions(+), 12 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index e5d64993a..e2060e8ac 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 24/29] Mark the has unknown portion as "leave as is"

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index c49d72462..783a4137f 100644 --- a/northd/ovn-northd.c +++ b/northd/ovn-northd.c @@ -7284,6 +7284,8 @@

[ovs-dev] [PATCH ovn RFC v3 23/29] Move destination lookup, unicast out of into a function

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 427 +++- 1 file changed, 227 insertions(+), 200 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index e215bcd63..c49d72462 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 25/29] Split and reorganize build_output_port_security

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Split and reorganize into iterator functions build_output_port_security Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 82 ++--- 1 file changed, 40 insertions(+), 42 deletions(-) diff --git a/northd/ovn-northd.c

[ovs-dev] [PATCH ovn RFC v3 12/29] Move egress to a separate rule and finalize

2020-07-16 Thread anton . ivanov
From: Anton Ivanov 1. Move egress to a separate rule function 2. Clean-up build_lrouter_flows from variables which are not used Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 107 1 file changed, 58 insertions(+), 49 deletions(-) diff --git

[ovs-dev] [PATCH ovn RFC v3 22/29] Pull out Destination lookup, broadcast and multicast handling

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Pulls out Destination lookup, broadcast and multicast handling into a separate function Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 160 1 file changed, 86 insertions(+), 74 deletions(-) diff --git

[ovs-dev] [PATCH ovn RFC v3 27/29] Parallelise lswitch lflow generation

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 285 1 file changed, 211 insertions(+), 74 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index b64c0c1a0..a53b31906 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 16/29] Pull Table 13 Arp/ND responder to a separate function

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 36 +--- 1 file changed, 21 insertions(+), 15 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index 878cfb493..13b3793fc 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 19/29] Pull DNS lookup and response to a separate function

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 67 + 1 file changed, 38 insertions(+), 29 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index 803971de0..da0ee4509 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 18/29] Pull DHCP Options into a function

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 98 - 1 file changed, 52 insertions(+), 46 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index 2226f5882..803971de0 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 26/29] Pallelise lrouter flow generation

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Generate lrouter flows in parallel, set threshold to switch between parallel and single threaded to always-parallel for temporary debugging purposes Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 252 ++-- 1 file changed, 171

[ovs-dev] [PATCH ovn RFC v3 14/29] Pull forwarding groups and admission control to separate functions

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 54 +++-- 1 file changed, 32 insertions(+), 22 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index 750a2a2b7..d00fef94d 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 13/29] Pull tables 3-10 ingress and 0-7 egress into a separate function

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 34 +- 1 file changed, 21 insertions(+), 13 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index 69972d55e..750a2a2b7 100644 --- a/northd/ovn-northd.c +++

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

2020-07-16 Thread Anton Ivanov
On 15/07/2020 11:04, Dumitru Ceara wrote: On 7/15/20 11:54 AM, Anton Ivanov wrote: On 15/07/2020 09:57, Dumitru Ceara wrote: On 7/14/20 6:06 PM, Anton Ivanov wrote: 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

[ovs-dev] [PATCH ovn RFC v3 09/29] Move packet size rules to a separate function

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 752 +++- 1 file changed, 397 insertions(+), 355 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index 7932ab9e3..0b4926a75 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 06/29] Move static route datapath mapping to a function

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 96 - 1 file changed, 51 insertions(+), 45 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index 807bf23ec..c6e58576a 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 07/29] Move out multicast lookup into a function

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 128 1 file changed, 69 insertions(+), 59 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index c6e58576a..61f9b4ab8 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 05/29] Move Logical router ingress table IP_ROUTING to a function

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 97 ++--- 1 file changed, 56 insertions(+), 41 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index 95c17af98..807bf23ec 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] OVN Northd parallelisation series redone in small steps

2020-07-16 Thread anton . ivanov
Hi All, This is the OVN northd parallel processing, redone in small atomic steps to make merging/rebasing easier. Patches 01-25 reorganize the processing in per-rule functions so that it can be run in parallel. Patches 26-28 move processing of lflows to a parallel model with everything forced

[ovs-dev] [PATCH ovn RFC v3 04/29] Move NAT and Load Balancing to a separate function

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 144 ++-- 1 file changed, 87 insertions(+), 57 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index 8a0e28040..95c17af98 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 08/29] Move Logical router ingress policy to a function

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 52 ++--- 1 file changed, 30 insertions(+), 22 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index 61f9b4ab8..7932ab9e3 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 03/29] Move out Table 3 operations to functions

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 143 ++-- 1 file changed, 84 insertions(+), 59 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index 1ec530854..8a0e28040 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 01/29] Move out Table 0 operations to functions

2020-07-16 Thread anton . ivanov
From: Anton Ivanov Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 71 - 1 file changed, 38 insertions(+), 33 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index 192198272..4374b88a6 100644 --- a/northd/ovn-northd.c +++

[ovs-dev] [PATCH ovn RFC v3 02/29] Move out Table 1 operations to functions

2020-07-16 Thread anton . ivanov
From: Anton Ivanov The functions are numbered in 10 step intervals to minimize renumbering if new pipeline stages are added in the future. Signed-off-by: Anton Ivanov --- northd/ovn-northd.c | 75 +++-- 1 file changed, 46 insertions(+), 29 deletions(-)

Re: [ovs-dev] [PATCH] rhel: openvswitch-fedora.spec.in: Fix installed but not packaged

2020-07-16 Thread Stokes, Ian
> With the cited commit, we get an error from rpmbuild about installed but not > packaged /usr/lib64/libopenvswitchavx512.a. > Fix it by treating it as the other la files. > > Fixes: 352b6c7116cd ("dpif-lookup: add avx512 gather implementation.") > Signed-off-by: Roi Dayan > --- >

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

2020-07-16 Thread Ilya Maximets
On 7/16/20 12:25 PM, Sriharsha Basavapatna wrote: > On Thu, Jul 16, 2020 at 2:52 PM Ilya Maximets wrote: >> >> On 7/16/20 10:01 AM, Eli Britstein wrote: >>> >>> On 7/16/2020 1:37 AM, Ilya Maximets wrote: On 7/15/20 8:30 PM, Stokes, Ian wrote: >> On 15/07/2020 17:00, Ilya Maximets wrote:

Re: [ovs-dev] [PATCH ovs-dev v1 2/2] dpctl-netdev: Add the option "pmd" for dump-flows

2020-07-16 Thread 0-day Robot
Bleep bloop. Greetings Tonghao Zhang, 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. git-am: error: Failed to merge in the changes. hint: Use 'git am --show-current-patch' to see the failed

[ovs-dev] [PATCH ovs-dev v1 1/2] dpctl: Fix memory leak in dpctl_dump_flows()

2020-07-16 Thread xiangxia . m . yue
From: Tonghao Zhang Goto label accurately to avoid memleak. Fixes: a692410af0f7 ("dpctl: Expand the flow dump type filter") Cc: Gavi Teitz Signed-off-by: Tonghao Zhang --- lib/dpctl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/dpctl.c b/lib/dpctl.c index

[ovs-dev] [PATCH ovs-dev v1 2/2] dpctl-netdev: Add the option "pmd" for dump-flows

2020-07-16 Thread xiangxia . m . yue
From: Tonghao Zhang "ovs-appctl dpctl/dump-flows" added the option "pmd" which allow user to dump pmd specified. That option is useful to dump rules of pmd when we have a large number of rules in dp. Signed-off-by: Tonghao Zhang --- NEWS| 2 ++ lib/dpctl.c | 28

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

2020-07-16 Thread Sriharsha Basavapatna via dev
On Thu, Jul 16, 2020 at 2:52 PM Ilya Maximets wrote: > > On 7/16/20 10:01 AM, Eli Britstein wrote: > > > > On 7/16/2020 1:37 AM, Ilya Maximets wrote: > >> On 7/15/20 8:30 PM, Stokes, Ian wrote: > On 15/07/2020 17:00, Ilya Maximets wrote: > > Main thread will try to pause/stop all

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

2020-07-16 Thread Ilya Maximets
On 7/16/20 10:01 AM, Eli Britstein wrote: > > On 7/16/2020 1:37 AM, Ilya Maximets wrote: >> On 7/15/20 8:30 PM, Stokes, Ian wrote: On 15/07/2020 17:00, Ilya Maximets wrote: > Main thread will try to pause/stop all revalidators during datapath > reconfiguration via datapath purge

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

2020-07-16 Thread Eli Britstein
On 7/16/2020 1:37 AM, Ilya Maximets wrote: On 7/15/20 8:30 PM, Stokes, Ian wrote: On 15/07/2020 17:00, 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

[ovs-dev] I BRING CALVARY GREETNG FROM MRS.SUZAN BONI.

2020-07-16 Thread suzanboni
-- Dear friend. I got your details after an extensive on-line search Via (Network Power Charitable Trust) for a reliable person, I'm Mrs.Suzan Boni, a 57years old dying woman who was diagnosed for cancer about 4 years ago, I have decided to donate my late husband fund the sum of ($1.000,000.00)