Re: [ovs-dev] [PATCH V5 0/2] Do not rewrite fields with the same values as matched

2019-02-28 Thread Eli Britstein
ping On 2/25/2019 9:42 PM, Eli Britstein wrote: > > On 2/25/2019 7:31 PM, Ben Pfaff wrote: >> On Sun, Feb 24, 2019 at 09:45:21AM +, Eli Britstein wrote: >>> On 2/22/2019 6:42 PM, Ben Pfaff wrote: On Sun, Feb 17, 2019 at 09:18:46AM +, Eli Britstein wrote: > This patch set avoids

Re: [ovs-dev] [PATCH] netdev-tc-offloads: Improve log message for icmpv6 offload not supported

2019-02-28 Thread Gregory Rose
On 2/28/2019 11:29 AM, Moshe Levi wrote: Signed-off-by: Moshe Levi --- lib/netdev-tc-offloads.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c index 36bce15..e65f5b9 100644 --- a/lib/netdev-tc-offloads.c +++

[ovs-dev] DANKE, WIE ICH IHRE ANTWORT ERWARTET,

2019-02-28 Thread Hamza Hamden via dev
LIEBER FREUND, MEIN NAME IST HAMZA HAMDEN, EIN INVESTOR-BERATER. Ich habe einen Kunden, der bereit ist, die Summe von fünf Millionen US-Dollar (45.000.000,00 Euro) in eine rentable Investition in Ihr Land unter Ihrer Obhut zu investieren. MEIN KUNDEN IST EIN TOP-POLITIKER IN SEINEM LAND,

Re: [ovs-dev] [PATCH v4 1/2] Avoid packet drop on LACP bond after link up

2019-02-28 Thread Ben Pfaff
On Thu, Feb 28, 2019 at 01:56:02PM +, Nitin Katiyar wrote: > Problem: > > During port DOWN->UP of link (slave) in a LACP bond, after receiving the > LACPDU with SYNC set for both actor and partner, the bond-slave remains > "disabled" until OVS main thread runs LACP state machine and

Re: [ovs-dev] [PATCH] rhel: Fix tests on mock and koji

2019-02-28 Thread Ben Pfaff
On Thu, Feb 28, 2019 at 01:24:08PM -0300, Flavio Leitner wrote: > On Thu, Feb 28, 2019 at 04:55:11PM +0100, Timothy Redaelli wrote: > > Currently many tests fails on mock/koji since /etc/resolv.conf is not > > present. The unexpected warning causes them to abort. > > > > After this patch an empty

Re: [ovs-dev] [PATCH] rhel: Use PIDFile on forking systemd service files

2019-02-28 Thread Ben Pfaff
On Thu, Feb 28, 2019 at 05:06:40PM -0300, Flavio Leitner wrote: > On Thu, Feb 28, 2019 at 06:27:46PM +0100, Timothy Redaelli wrote: > > Currently, PIDFile is not used in systemd service files with > > Type=forking. This means sometimes systemd fails to restart a daemon > > that is killed (with

Re: [ovs-dev] [PATCH] NEWS: Clean up the 2.11.0 release notes a bit.

2019-02-28 Thread Justin Pettit
> On Feb 28, 2019, at 10:59 AM, Ben Pfaff wrote: > > On Thu, Feb 28, 2019 at 10:39:16AM -0800, Justin Pettit wrote: >> Signed-off-by: Justin Pettit > > Acked-by: Ben Pfaff Thanks. I pushed this to master and branch-2.11. --Justin ___ dev

Re: [ovs-dev] [PATCH] rhel: Use PIDFile on forking systemd service files

2019-02-28 Thread Flavio Leitner
On Thu, Feb 28, 2019 at 06:27:46PM +0100, Timothy Redaelli wrote: > Currently, PIDFile is not used in systemd service files with > Type=forking. This means sometimes systemd fails to restart a daemon > that is killed (with SIGKILL) or that is crashed. > > This commit adds PIDFile to all systemd

[ovs-dev] [PATCH] netdev-tc-offloads: Improve log message for icmpv6 offload not supported

2019-02-28 Thread Moshe Levi
Signed-off-by: Moshe Levi --- lib/netdev-tc-offloads.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c index 36bce15..e65f5b9 100644 --- a/lib/netdev-tc-offloads.c +++ b/lib/netdev-tc-offloads.c @@ -1015,12 +1015,12

Re: [ovs-dev] [PATCH] rhel: Use PIDFile on forking systemd service files

2019-02-28 Thread Candido Campos Rivas
hi, yes, if there are some expert in systemd, it'd be the best :) I only can help passing you the systemd doc cof that recommends use this option with the forking type : https://www.freedesktop.org/software/systemd/man/systemd.service.html . Type= Configures the process start-up type

Re: [ovs-dev] [PATCH] rhel: limit stack size to 2M.

2019-02-28 Thread Ben Pfaff
On Thu, Feb 28, 2019 at 07:28:58PM +0100, Timothy Redaelli wrote: > On Thu, 28 Feb 2019 13:13:57 -0300 > Flavio Leitner wrote: > > > The default stack size in Fedora/RHEL is 8M, which means when ovs-vswitchd > > daemon starts and uses --mlockall (default), it will dirty all memory > > regions

Re: [ovs-dev] [PATCH v3] OVN: select a random mac_prefix if not provided

2019-02-28 Thread Ben Pfaff
On Wed, Feb 27, 2019 at 07:01:10PM +0100, Lorenzo Bianconi wrote: > Select a random IPAM mac_prefix if it has not been provided by the user. > With this patch the admin can avoid to configure mac_prefix in order to > avoid L2 address collisions if multiple OVN deployments share the same >

Re: [ovs-dev] [PATCH] ovn: Make DHCP log messages unique

2019-02-28 Thread Ben Pfaff
On Thu, Feb 28, 2019 at 01:06:50PM -0500, Brian Haley wrote: > From: Brian Haley > > Two messags were using the same string, add info to one > to make it unique. Also cleaned-up some of the others > to make them consistent throughout. > > Signed-off-by: Brian Haley Thanks, applied to master.

Re: [ovs-dev] [PATCH v4 0/5] Fast OVSDB resync after restart or fail-over.

2019-02-28 Thread Ben Pfaff
On Thu, Feb 28, 2019 at 09:15:15AM -0800, Han Zhou wrote: > In scalability test with ovn-scale-test, ovsdb-server SB load is not a > problem at least with 1k HVs. However, if we restart the ovsdb-server, > depending on the number of HVs and scale of logical objects, e.g. the > number of logical

Re: [ovs-dev] ovn: Make DHCP log messages unique

2019-02-28 Thread 0-day Robot
Bleep bloop. Greetings Brian Haley, 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: Author Brian Haley needs to sign off. WARNING: Unexpected sign-offs from developers who

Re: [ovs-dev] [PATCH] NEWS: Clean up the 2.11.0 release notes a bit.

2019-02-28 Thread Ben Pfaff
On Thu, Feb 28, 2019 at 10:39:16AM -0800, Justin Pettit wrote: > Signed-off-by: Justin Pettit Acked-by: Ben Pfaff ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH v3] OVN: select a random mac_prefix if not provided

2019-02-28 Thread Numan Siddique
On Wed, Feb 27, 2019 at 11:43 PM Lorenzo Bianconi < lorenzo.bianc...@redhat.com> wrote: > Select a random IPAM mac_prefix if it has not been provided by the user. > With this patch the admin can avoid to configure mac_prefix in order to > avoid L2 address collisions if multiple OVN deployments

[ovs-dev] RE

2019-02-28 Thread wsolorzano
Avez-vous besoin d'un prêt? Contactez-nous aujourd'hui pour votre prêt @ 3% par an pour plus de détails. ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

[ovs-dev] [PATCH] NEWS: Clean up the 2.11.0 release notes a bit.

2019-02-28 Thread Justin Pettit
Signed-off-by: Justin Pettit --- NEWS | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/NEWS b/NEWS index c86c13a23c38..0d736541da07 100644 --- a/NEWS +++ b/NEWS @@ -20,16 +20,10 @@ Post-v2.11.0 v2.11.0 - 19 Feb 2019 - - - Linux datapath:

Re: [ovs-dev] [PATCH] rhel: limit stack size to 2M.

2019-02-28 Thread Timothy Redaelli
On Thu, 28 Feb 2019 13:13:57 -0300 Flavio Leitner wrote: > The default stack size in Fedora/RHEL is 8M, which means when ovs-vswitchd > daemon starts and uses --mlockall (default), it will dirty all memory > regions for all threads which is proportionally to the number of CPUs. > > On a big

Re: [ovs-dev] [PATCH] rhel: Use PIDFile on forking systemd service files

2019-02-28 Thread Ben Pfaff
On Thu, Feb 28, 2019 at 06:27:46PM +0100, Timothy Redaelli wrote: > Currently, PIDFile is not used in systemd service files with > Type=forking. This means sometimes systemd fails to restart a daemon > that is killed (with SIGKILL) or that is crashed. > > This commit adds PIDFile to all systemd

Re: [ovs-dev] ovsdb-monitor: Support monitor_cond_since.

2019-02-28 Thread 0-day Robot
Bleep bloop. Greetings Han Zhou, 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 is 98 characters long (recommended limit is 79) #595 FILE: ovsdb/ovsdb-client.c:433:

[ovs-dev] [PATCH] ovn: Make DHCP log messages unique

2019-02-28 Thread Brian Haley
From: Brian Haley Two messags were using the same string, add info to one to make it unique. Also cleaned-up some of the others to make them consistent throughout. Signed-off-by: Brian Haley --- ovn/controller/pinctrl.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-)

[ovs-dev] [PATCH] rhel: Use PIDFile on forking systemd service files

2019-02-28 Thread Timothy Redaelli
Currently, PIDFile is not used in systemd service files with Type=forking. This means sometimes systemd fails to restart a daemon that is killed (with SIGKILL) or that is crashed. This commit adds PIDFile to all systemd service file with Type=forking in order to always have the correct PID to

[ovs-dev] [PATCH v4 3/5] ovsdb-monitor: Support monitor_cond_since.

2019-02-28 Thread Han Zhou
From: Han Zhou Support the new monitor method monitor_cond_since so that a client can request monitoring start from a specific point instead of always from beginning. This will reduce the cost at scenarios when server is restarted/failed-over but client still has all existing data. In these

[ovs-dev] [PATCH v4 5/5] ovsdb-idl.c: Fast resync from server when connection reset.

2019-02-28 Thread Han Zhou
From: Han Zhou Use monitor_cond_since to request changes after last version of local data when connection to server is reset, without clearing the local data. It falls back to clearing and repopulating all the data when the requested id cannot be fulfilled by the server. Test result at

[ovs-dev] [PATCH v4 4/5] ovsdb-idl.c: Support monitor_cond_since method in C IDL.

2019-02-28 Thread Han Zhou
From: Han Zhou Use monitor_cond_since in C IDL. If it is not supported by server, fall back to old method (monitor_cond, or monitor). Signed-off-by: Han Zhou --- lib/ovsdb-idl.c | 203 ++-- 1 file changed, 155 insertions(+), 48 deletions(-)

[ovs-dev] [PATCH v4 1/5] ovsdb-monitor: Refactor ovsdb monitor implementation.

2019-02-28 Thread Han Zhou
From: Han Zhou Current ovsdb monitor maintains its own transaction version through an incremental integer and use it to identify changes starting from different version, and also use it to figure out if each set of changes should be flushed. In particular, it uses number 0 to represent that the

[ovs-dev] [PATCH v4 2/5] ovsdb-server: Transaction history tracking.

2019-02-28 Thread Han Zhou
From: Han Zhou Maintaining last N (n = 100) transactions in memory, which will be used for future patches for generating monitor data from any point in this N transactions. Signed-off-by: Han Zhou --- ovsdb/ovsdb-server.c | 9 ovsdb/ovsdb.c| 6 +++ ovsdb/ovsdb.h| 10

[ovs-dev] [PATCH v4 0/5] Fast OVSDB resync after restart or fail-over.

2019-02-28 Thread Han Zhou
In scalability test with ovn-scale-test, ovsdb-server SB load is not a problem at least with 1k HVs. However, if we restart the ovsdb-server, depending on the number of HVs and scale of logical objects, e.g. the number of logical ports, ovsdb-server of SB become an obvious bottleneck. In our test

Re: [ovs-dev] [PATCH] rhel: Fix tests on mock and koji

2019-02-28 Thread Flavio Leitner
On Thu, Feb 28, 2019 at 04:55:11PM +0100, Timothy Redaelli wrote: > Currently many tests fails on mock/koji since /etc/resolv.conf is not > present. The unexpected warning causes them to abort. > > After this patch an empty resolv.conf is created and used before issuing > "make check". > >

[ovs-dev] [PATCH] rhel: limit stack size to 2M.

2019-02-28 Thread Flavio Leitner
The default stack size in Fedora/RHEL is 8M, which means when ovs-vswitchd daemon starts and uses --mlockall (default), it will dirty all memory regions for all threads which is proportionally to the number of CPUs. On a big host this increases memory usage to many hundreds of megabytes while OVS

[ovs-dev] [PATCH] rhel: Fix tests on mock and koji

2019-02-28 Thread Timothy Redaelli
Currently many tests fails on mock/koji since /etc/resolv.conf is not present. The unexpected warning causes them to abort. After this patch an empty resolv.conf is created and used before issuing "make check". Signed-off-by: Timothy Redaelli --- rhel/openvswitch-fedora.spec.in | 2 ++

Re: [ovs-dev] netdev-dpdk: Expose flow creation/destruction calls

2019-02-28 Thread 0-day Robot
Bleep bloop. Greetings Ophir Munk, 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] Ultimos Lugares

2019-02-28 Thread Planes de carrera y sucesión
Cursos TOP - Webinar Interactivo – Miércoles 06 de Marzo Planes de carrera y sucesión Desarrollaremos planes de carrera y sucesión que aseguren el funcionamiento óptimo de una empresa. Ejes Temáticos: • Competencias organizacionales complejas. • Desarrollo de plan de vida y carrera. •

[ovs-dev] No more worries for your credit inquiries//Keine Sorgen mehr um Ihre Kreditanfragen

2019-02-28 Thread Hundsdorff Frank Michael
Hello In partnership with a financial group (individuals and banks), I can grant you loans ranging in size from 10,000 to 10,000,000 euros, with a very low interest rate of up to 2.8% and 6% in depending on the amount borrowed. We operate in the following areas: Credit, personal loan, instant

Re: [ovs-dev] [PATCH v3 2/3] netdev-dpdk: Move offloading code to a new file

2019-02-28 Thread Ophir Munk
> -Original Message- > From: Ilya Maximets > Sent: Thursday, February 28, 2019 11:59 AM > To: Ophir Munk ; ovs-dev@openvswitch.org > Cc: Asaf Penso ; Ian Stokes ; > Shahaf Shuler ; Olga Shern > ; Kevin Traynor ; Roni Bar > Yanai > Subject: Re: [PATCH v3 2/3] netdev-dpdk: Move

[ovs-dev] [PATCH v5 2/3] netdev-dpdk: Move offloading code to a new file

2019-02-28 Thread Ophir Munk
From: Roni Bar Yanai Hardware offloading code is moved to a new file called netdev-rte-offloads.c. The original offloading code is copied from the netdev-dpdk.c file to the new file, where future offloading code should be added as well. The copied code was refactored based on coding style. The

[ovs-dev] [PATCH v5 3/3] netdev-rte-offloads: Rename netdev_dpdk_* functions

2019-02-28 Thread Ophir Munk
Rename all the netdev_dpdk_* functions names (originated from the file netdev-dpdk.c) into the netdev_rte_offloads_* functions names. Signed-off-by: Ophir Munk --- lib/netdev-rte-offloads.c | 45 +++-- lib/netdev-rte-offloads.h | 17 + 2

[ovs-dev] [PATCH v5 1/3] netdev-dpdk: Expose flow creation/destruction calls

2019-02-28 Thread Ophir Munk
From: Roni Bar Yanai Before offloading code was added to the netdev-dpdk.c file (MARK and RSS actions) the only DPDK RTE calls in use were rte_flow_create() and rte_flow_destroy(). In preparation for splitting the offloading code from the netdev-dpdk.c file to a separate file, it is required to

[ovs-dev] [PATCH v5 0/3] Move offloading code into a new file

2019-02-28 Thread Ophir Munk
Hardware offloading code is moved to a new file called netdev-rte-offloads.c. The original offloading code is copied as is from the netdev-dpdk.c file to the new file where future offloading code should be added as well. The new code is refactored based on code styling. This series is essential

[ovs-dev] [PATCH v4 2/2] Avoid packet drop on LACP bond after link up

2019-02-28 Thread Nitin Katiyar
Problem: On certain Fortville NICs it has been observed that PHY UP detection can get delayed (sometimes up to 4-5 secs). When the driver fails to fetch PHY status as UP even though its actually UP, LACP packets can get exchanged and LACP slave brought UP. In such a case, the remote end

[ovs-dev] [PATCH v4 1/2] Avoid packet drop on LACP bond after link up

2019-02-28 Thread Nitin Katiyar
Problem: During port DOWN->UP of link (slave) in a LACP bond, after receiving the LACPDU with SYNC set for both actor and partner, the bond-slave remains "disabled" until OVS main thread runs LACP state machine and eventually "enables" the bond-slave. With this, we have observed delays in

Re: [ovs-dev] netdev-dpdk: Expose flow creation/destruction calls

2019-02-28 Thread 0-day Robot
Bleep bloop. Greetings Ophir Munk, 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 v4 1/3] netdev-dpdk: Expose flow creation/destruction calls

2019-02-28 Thread Ophir Munk
From: Roni Bar Yanai Before offloading code was added to the netdev-dpdk.c file (MARK and RSS actions) the only DPDK RTE calls in use were rte_flow_create() and rte_flow_destroy(). In preparation for splitting the offloading code from the netdev-dpdk.c file to a separate file, it is required to

[ovs-dev] [PATCH v4 3/3] netdev-rte-offloads: Rename netdev_dpdk_* functions

2019-02-28 Thread Ophir Munk
Rename all the netdev_dpdk_* functions names (originated from the file netdev-dpdk.c) into the netdev_rte_offloads_* functions names. Signed-off-by: Ophir Munk --- lib/netdev-rte-offloads.c | 45 +++-- lib/netdev-rte-offloads.h | 17 + 2

[ovs-dev] [PATCH v4 0/3] Move offloading code into a new file

2019-02-28 Thread Ophir Munk
Hardware offloading code is moved to a new file called netdev-rte-offloads.c. The original offloading code is copied as is from the netdev-dpdk.c file to the new file where future offloading code should be added as well. The new code is refactored based on code styling. This series is essential

[ovs-dev] [PATCH v4 2/3] netdev-dpdk: Move offloading code to a new file

2019-02-28 Thread Ophir Munk
From: Roni Bar Yanai Hardware offloading code is moved to a new file called netdev-rte-offloads.c. The original offloading code is copied from the netdev-dpdk.c file to the new file, where future offloading code should be added as well. The copied code was refactored based on coding style. The

Re: [ovs-dev] [PATCH] datapath-windows: Guard vport usage in user.c

2019-02-28 Thread aserdean
> Do we need to keep the dispatchLock for reading vport structures? [Alin Serdean] We need to make sure that port is not deleted since we use it for processing. >If that's > the case, then we will need to fix it in other areas of the code too. [Alin Serdean] I sent out

Re: [ovs-dev] [PATCH v3 2/3] netdev-dpdk: Move offloading code to a new file

2019-02-28 Thread Ilya Maximets
On 28.02.2019 12:44, Ophir Munk wrote: > > >> -Original Message- >> From: Ilya Maximets >> Sent: Thursday, February 28, 2019 10:25 AM >> To: Ophir Munk ; ovs-dev@openvswitch.org >> Cc: Asaf Penso ; Ian Stokes ; >> Shahaf Shuler ; Olga Shern >> ; Kevin Traynor ; Roni Bar >> Yanai >>

Re: [ovs-dev] [PATCH v3 2/3] netdev-dpdk: Move offloading code to a new file

2019-02-28 Thread Ophir Munk
> -Original Message- > From: Ilya Maximets > Sent: Thursday, February 28, 2019 10:25 AM > To: Ophir Munk ; ovs-dev@openvswitch.org > Cc: Asaf Penso ; Ian Stokes ; > Shahaf Shuler ; Olga Shern > ; Kevin Traynor ; Roni Bar > Yanai > Subject: Re: [PATCH v3 2/3] netdev-dpdk: Move

Re: [ovs-dev] [PATCH v3 3/3] netdev-rte-offloads: Rename netdev_dpdk_* functions

2019-02-28 Thread Ophir Munk
> -Original Message- > From: Ilya Maximets > Sent: Thursday, February 28, 2019 10:30 AM > To: Ophir Munk ; ovs-dev@openvswitch.org > Cc: Asaf Penso ; Ian Stokes ; > Shahaf Shuler ; Olga Shern > ; Kevin Traynor ; Roni Bar > Yanai > Subject: Re: [PATCH v3 3/3] netdev-rte-offloads:

Re: [ovs-dev] [PATCH v3 1/3] netdev-dpdk: Expose flow creation/destruction calls

2019-02-28 Thread Ophir Munk
> -Original Message- > From: Ilya Maximets > Sent: Thursday, February 28, 2019 9:40 AM > To: Ophir Munk ; ovs-dev@openvswitch.org > Cc: Asaf Penso ; Ian Stokes ; > Shahaf Shuler ; Olga Shern > ; Kevin Traynor ; Roni Bar > Yanai > Subject: Re: [PATCH v3 1/3] netdev-dpdk: Expose flow

Re: [ovs-dev] [PATCH v2 2/5] ovn: Add generic HA chassis group

2019-02-28 Thread Numan Siddique
On Thu, Feb 28, 2019 at 1:50 AM Han Zhou wrote: > On Wed, Feb 27, 2019 at 2:45 AM wrote: > > > > From: Numan Siddique > > > > This patch adds the tables - 'HA_Chassis_Group' and 'HA_Chassis' in > > both OVN Northbound and Southbound DBs to support generic HA Chassis > > groups in OVN. CMS can

Re: [ovs-dev] [PATCH v3 3/3] netdev-rte-offloads: Rename netdev_dpdk_* functions

2019-02-28 Thread Ilya Maximets
On 28.02.2019 1:37, Ophir Munk wrote: > Rename all the netdev_dpdk_* functions names (originated from the file > netdev-dpdk.c) into the netdev_rte_offloads_* functions names. > > Signed-off-by: Ophir Munk > --- > lib/netdev-rte-offloads.c | 39 --- >

Re: [ovs-dev] [PATCH v3 2/3] netdev-dpdk: Move offloading code to a new file

2019-02-28 Thread Ilya Maximets
On 28.02.2019 1:37, Ophir Munk wrote: > From: Roni Bar Yanai > > Hardware offloading code is moved to a new file called > netdev-rte-offloads.c. The original offloading code is copied > from the netdev-dpdk.c file to the new file, where future > offloading code should be added as well. > The