[ovs-dev] INVESTMENT

2020-06-16 Thread HANJA PARK
Good day, You were recommended by a mutual associate. I write you regarding an investment of bearer bonds I made on behalf of a client. The investment was made in 2009 and has been under my management. The said investor is deceased. The window is now available to assign these bonds t

Re: [ovs-dev] [PATCH] netdev-offload-tc: Revert tunnel src/dst port masks handling

2020-06-16 Thread Tonghao Zhang
On Tue, Jun 16, 2020 at 9:05 PM Roi Dayan wrote: > > The cited commit intended to add tc support for masking tunnel src/dst > ips and ports. It's not possible to do tunnel ports masking with > openflow rules and the default mask for tunnel ports set to 0 in > tnl_wc_init(), unlike tunnel ports def

Re: [ovs-dev] [PATCH ovs v3] ovsdb idl: Try committing the pending txn in ovsdb_idl_loop_run.

2020-06-16 Thread Numan Siddique
On Wed, Jun 17, 2020 at 2:49 AM Ben Pfaff wrote: > On Fri, Jun 05, 2020 at 02:00:29PM +0530, num...@ovn.org wrote: > > From: Numan Siddique > > > > The function ovsdb_idl_loop_run(), after calling ovsdb_idl_run(), > > returns a transaction object (of type 'struct ovsdb_idl_txn'). > > The returne

[ovs-dev] URGENT: You've got ONE DAY to watch this...

2020-06-16 Thread 5G Male
http://www.entrancepitch.ml/2_f5R23Z95SW8I611S5Gs62z100L20lHI5fstIr5w6bxGaEsvZ6QdpoSQ6iFL1Y06k0H2@D/victimizing-stiffness ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH] odp-util.c: Fix dp_hash execution with slowpath actions.

2020-06-16 Thread Han Zhou
On Tue, Jun 16, 2020 at 2:40 PM Ben Pfaff wrote: > > On Fri, May 15, 2020 at 12:17:47AM -0700, Han Zhou wrote: > > When dp_hash is executed with slowpath actions, it results in endless > > recirc loop in kernel datapath, and finally drops the packet, with > > kernel logs: > > > > openvswitch: ovs-

Re: [ovs-dev] [PATCH v2] ofproto-dpif-trace: Improve NAT tracing.

2020-06-16 Thread Ben Pfaff
On Fri, Jan 10, 2020 at 10:34:43AM +0100, Dumitru Ceara wrote: > When ofproto/trace detects a recirc action it resumes execution at the > specified next table. However, if the ct action performs SNAT/DNAT, > e.g., ct(commit,nat(src=1.1.1.1:4000),table=42), the src/dst IPs and > ports in the oftrace

Re: [ovs-dev] [PATCH] odp-util.c: Fix dp_hash execution with slowpath actions.

2020-06-16 Thread Ben Pfaff
On Thu, Jun 11, 2020 at 03:15:13PM +0200, Ilya Maximets wrote: > I understand that this patch fixes this particular case, however I still > think it's dangerous to pass the hash calculated in userspace to kernel > since it might cause mismatch for the later packets in case where the > flow doesn't

Re: [ovs-dev] [PATCH] odp-util.c: Fix dp_hash execution with slowpath actions.

2020-06-16 Thread Ben Pfaff
On Fri, May 15, 2020 at 12:17:47AM -0700, Han Zhou wrote: > When dp_hash is executed with slowpath actions, it results in endless > recirc loop in kernel datapath, and finally drops the packet, with > kernel logs: > > openvswitch: ovs-system: deferred action limit reached, drop recirc action > >

Re: [ovs-dev] [PATCH ovs v3] ovsdb idl: Try committing the pending txn in ovsdb_idl_loop_run.

2020-06-16 Thread Ben Pfaff
On Fri, Jun 05, 2020 at 02:00:29PM +0530, num...@ovn.org wrote: > From: Numan Siddique > > The function ovsdb_idl_loop_run(), after calling ovsdb_idl_run(), > returns a transaction object (of type 'struct ovsdb_idl_txn'). > The returned transaction object can be NULL if there is a pending > trans

[ovs-dev] [PATCH ovn] Fix the routing for external logical ports of bridged logical switches.

2020-06-16 Thread numans
From: Numan Siddique Routing for external logical ports is broken if these ports belonged to bridged logical switches (with localnet port) and 'ovn-chassis-mac-mappings' is configured. External logical ports are those which are external to OVN, but there is a logical port for it and it is claimed

Re: [ovs-dev] [PATCH v3 0/7] DPCLS Subtable ISA Optimization

2020-06-16 Thread Van Haaren, Harry
> -Original Message- > From: William Tu > Sent: Tuesday, June 16, 2020 4:50 PM > To: Van Haaren, Harry > Cc: ovs-dev ; Stokes, Ian ; > Ilya Maximets ; Federico Iezzi > Subject: Re: [PATCH v3 0/7] DPCLS Subtable ISA Optimization > > On Wed, Jun 10, 2020 at 3:47 AM Harry van Haaren > wro

Re: [ovs-dev] [PATCH v3 0/7] DPCLS Subtable ISA Optimization

2020-06-16 Thread William Tu
On Wed, Jun 10, 2020 at 3:47 AM Harry van Haaren wrote: > > v3 Changes Summary: > - Added new "subtable lookup get" command for ease of use > - Changed set command to include "prio" aligning with other commands > - Improved output of "subtable lookup prio set" command > - Added documentation > - M

Re: [ovs-dev] [PATCH] netdev-offload-tc: Revert tunnel src/dst port masks handling

2020-06-16 Thread Simon Horman
On Tue, Jun 16, 2020 at 05:06:49PM +0300, Roi Dayan wrote: > > > On 2020-06-16 4:03 PM, Roi Dayan wrote: > > The cited commit intended to add tc support for masking tunnel src/dst > > ips and ports. It's not possible to do tunnel ports masking with > > openflow rules and the default mask for tunn

Re: [ovs-dev] [PATCH] netdev-offload-tc: Revert tunnel src/dst port masks handling

2020-06-16 Thread Roi Dayan
On 2020-06-16 5:06 PM, Roi Dayan wrote: > > > On 2020-06-16 4:03 PM, Roi Dayan wrote: >> The cited commit intended to add tc support for masking tunnel src/dst >> ips and ports. It's not possible to do tunnel ports masking with >> openflow rules and the default mask for tunnel ports set to 0 i

Re: [ovs-dev] [PATCH v3 2/7] dpif-netdev: add subtable lookup set command

2020-06-16 Thread William Tu
Thanks for the patch. I have some minor comments. On Wed, Jun 10, 2020 at 3:47 AM Harry van Haaren wrote: the commit title should add "lookup prio set command"? > > This commit adds a command for the dpif-netdev to set a specific > lookup function to a particular priority level. The command enab

Re: [ovs-dev] [PATCH v3 3/7] dpif-netdev: add subtable-lookup-get command for usability

2020-06-16 Thread William Tu
On Wed, Jun 10, 2020 at 3:47 AM Harry van Haaren wrote: > > This commit introduces a new command, "dpif-netdev/subtable-lookup-get" add prio > which prints the avaiable sutable lookup functions available in this OVS typo available subtable > binary. Example output from the command: > > Available

Re: [ovs-dev] [PATCH v3 1/7] dpif: implement subtable lookup validation

2020-06-16 Thread William Tu
Thanks, some minor comments inline. On Wed, Jun 10, 2020 at 3:47 AM Harry van Haaren wrote: > > This commit refactors the existing dpif subtable function pointer > infrastructure, and implements an autovalidator component. > > The refactoring of the existing dpcls subtable lookup function > handl

Re: [ovs-dev] [PATCH v3 4/7] dpcls: enable cpu feature detection

2020-06-16 Thread William Tu
btw, remember to add "." at the end of the commit title. so "dpcls: enable cpu feature detection." On Wed, Jun 10, 2020 at 3:47 AM Harry van Haaren wrote: > > This commit implements a method to retrieve the CPU ISA capabilities. > These ISA capabilities can be used in OVS to select a function > i

Re: [ovs-dev] [PATCH v3 6/7] dpif-lookup: add avx512 gather implementation

2020-06-16 Thread William Tu
On Wed, Jun 10, 2020 at 3:47 AM Harry van Haaren wrote: > > This commit adds an AVX-512 dpcls lookup implementation. > It uses the AVX-512 SIMD ISA to perform multiple miniflow > operations in parallel. > > To run this implementation, the "avx512f" and "bmi2" ISAs are > required. These ISA checks

Re: [ovs-dev] [PATCH v3 5/7] lib/automake: split build multiple static library

2020-06-16 Thread William Tu
On Wed, Jun 10, 2020 at 3:47 AM Harry van Haaren wrote: > > This commit changes the way the core lib/* code is built. > Before this commit, the lib/libopenvswitch_la target contains > all the code, and is directly linked against by executable targets > like ovs-vswitchd, ovsdb, tests etc. > > This

Re: [ovs-dev] [PATCH v3 7/7] docs/dpdk/bridge: add datapath performance section

2020-06-16 Thread William Tu
On Wed, Jun 10, 2020 at 3:47 AM Harry van Haaren wrote: > > This commit adds a section to the dpdk/bridge.rst netdev documentation, > detailing the added DPCLS functionality. The newely added commands are typo: newly > documented, and sample output is provided. > > Signed-off-by: Harry van Haaren

Re: [ovs-dev] [PATCH] checkpatch: Increase recommended line length to 100

2020-06-16 Thread Aaron Conole
Simon Horman writes: > On Tue, Jun 16, 2020 at 04:14:55PM +0300, Roi Dayan wrote: >> >> >> On 2020-06-11 1:09 PM, Roi Dayan wrote: >> > This is to match a recent kernel checkpatch change that >> > also increased it to 100 line length. >> > >> > Signed-off-by: Roi Dayan >> > --- >> > utilitie

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

2020-06-16 Thread Simon Horman
On Tue, Jun 16, 2020 at 04:14:23PM +0300, Roi Dayan wrote: > > > On 2020-06-14 7:03 PM, Roi Dayan wrote: > > This arg can be used internally by groups using gerrit for code reviews. > > > > Signed-off-by: Roi Dayan > > --- > > utilities/checkpatch.py | 9 +++-- > > 1 file changed, 7 insert

Re: [ovs-dev] [PATCH] netdev-offload-tc: Revert tunnel src/dst port masks handling

2020-06-16 Thread Roi Dayan
On 2020-06-16 4:03 PM, Roi Dayan wrote: > The cited commit intended to add tc support for masking tunnel src/dst > ips and ports. It's not possible to do tunnel ports masking with > openflow rules and the default mask for tunnel ports set to 0 in > tnl_wc_init(), unlike tunnel ports default mask

Re: [ovs-dev] [PATCH] checkpatch: Increase recommended line length to 100

2020-06-16 Thread Simon Horman
On Tue, Jun 16, 2020 at 04:14:55PM +0300, Roi Dayan wrote: > > > On 2020-06-11 1:09 PM, Roi Dayan wrote: > > This is to match a recent kernel checkpatch change that > > also increased it to 100 line length. > > > > Signed-off-by: Roi Dayan > > --- > > utilities/checkpatch.py | 4 ++-- > > 1 fi

Re: [ovs-dev] [PATCH] checkpatch: Increase recommended line length to 100

2020-06-16 Thread Roi Dayan
On 2020-06-11 1:09 PM, Roi Dayan wrote: > This is to match a recent kernel checkpatch change that > also increased it to 100 line length. > > Signed-off-by: Roi Dayan > --- > utilities/checkpatch.py | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/utilities/checkp

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

2020-06-16 Thread Roi Dayan
On 2020-06-14 7:03 PM, Roi Dayan wrote: > This arg can be used internally by groups using gerrit for code reviews. > > Signed-off-by: Roi Dayan > --- > utilities/checkpatch.py | 9 +++-- > 1 file changed, 7 insertions(+), 2 deletions(-) > > diff --git a/utilities/checkpatch.py b/utilitie

[ovs-dev] [PATCH] netdev-offload-tc: Revert tunnel src/dst port masks handling

2020-06-16 Thread Roi Dayan
The cited commit intended to add tc support for masking tunnel src/dst ips and ports. It's not possible to do tunnel ports masking with openflow rules and the default mask for tunnel ports set to 0 in tnl_wc_init(), unlike tunnel ports default mask which is full mask. So instead of never passing tu

Re: [ovs-dev] [PATCH ovn] Fix ovn-controller generated packets from getting dropped for reject ACL action.

2020-06-16 Thread Mark Michelson
Looks good to me. Acked-by: Mark Michelson On 6/15/20 5:47 AM, num...@ovn.org wrote: From: Numan Siddique TCP reset/ICMP packet generated by ovn-controller for the ACL reject action gets dropped by ovs-vswithd with the below messages in ovs-vswitchd log even though ovn-controller sets the in

Re: [ovs-dev] [PATCH ovs] dpif-netdev: Delete the flow limit in dpif-netdev

2020-06-16 Thread Tonghao Zhang
On Tue, Jun 16, 2020 at 8:12 PM Tonghao Zhang wrote: > > On Mon, Mar 16, 2020 at 7:17 PM wrote: > > > > From: Tonghao Zhang > > > > The max number of flow in dpif-netdev is MAX_FLOWS, > > the previous patch[1] try to install more flows, > > and introduce the subcommand "dpif-netdev/pmd-set-max-f

[ovs-dev] Tränen und Trauer Liebste,

2020-06-16 Thread na Andrew via dev
Tränen und Trauer Liebste, Im Vertrauen schreibe ich Ihnen diese Nachricht mit Tränen und Trauer und ich weiß, dass diese Mail zu Ihnen kommen kann, als Überraschung muss ich mich vorstellen, ich bin Naomi Andrew, ich bin das einzige Kind von Mr & Mrs. Andrew. Ich habe gebetet, bevor ich mich m

Re: [ovs-dev] [PATCH 1/1] openvswitch: fix infoleak in conntrack

2020-06-16 Thread Simon Horman
On Mon, Jun 15, 2020 at 07:13:01PM -0700, Xidong Wang wrote: > From: xidongwang > > The stack object “zone_limit” has 3 members. In function > ovs_ct_limit_get_default_limit(), the member "count" is > not initialized and sent out via “nla_put_nohdr”. Hi Xidong, thanks for your patch. It appear

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

2020-06-16 Thread Tonghao Zhang
On Mon, Jun 15, 2020 at 7:13 PM Ilya Maximets wrote: > > On 6/14/20 1:40 PM, Tonghao Zhang wrote: > > On Sun, Jun 14, 2020 at 1:31 PM Roi Dayan wrote: > >> > >> > >> > >> On 2020-06-11 1:36 PM, xiangxia.m@gmail.com wrote: > >>> From: Tonghao Zhang > >>> > >>> When ports were removed from bri

Re: [ovs-dev] [PATCH 1/1] openvswitch: fix infoleak in conntrack

2020-06-16 Thread Tonghao Zhang
On Tue, Jun 16, 2020 at 10:13 AM Xidong Wang wrote: > > From: xidongwang > > The stack object “zone_limit” has 3 members. In function > ovs_ct_limit_get_default_limit(), the member "count" is > not initialized and sent out via “nla_put_nohdr”. > > Signed-off-by: xidongwang > --- > net/openvswit

Re: [ovs-dev] [PATCH 1/1] openvswitch: fix infoleak in conntrack

2020-06-16 Thread Pravin Shelar
On Mon, Jun 15, 2020 at 7:13 PM Xidong Wang wrote: > > From: xidongwang > > The stack object “zone_limit” has 3 members. In function > ovs_ct_limit_get_default_limit(), the member "count" is > not initialized and sent out via “nla_put_nohdr”. > > Signed-off-by: xidongwang Looks good. Acked-by:

[ovs-dev] Vor kurzem wurde bei mir Lungenkrebs diagnostiziert

2020-06-16 Thread Terry Allan
Hallo, Meine Entscheidung, Sie zu kontaktieren, ist, weil bei mir kürzlich Lungenkrebs diagnostiziert wurde und der Arzt sagte, ich habe weniger als 6 Wochen zu leben. Seit mir diese plötzliche Nachricht bekannt gegeben wurde, habe ich über mein Leben in der Vergangenheit nachgedacht. Es ist schme