Re: [ovs-dev] [PATCH] memory: kill ovs-vswitchd under super

Hi, I also found that if once theres are lots of flows, the memory (RSS) usage of OVS process would be quite high, 2~3GB. Even then the flows disappear later, the memory still keeps. I am not sure how many people notices this, but if indeed OVS has such defect, I guess this should be critical

Re: [ovs-dev] [PATCH V2] OF1.5/EXT-334 OXS/Extensible Flow Entry Statistics Support

Hi Ben, We didn't observed any test case breaks and we've updated the same with logs in our previous conversations. Could you please provide your inputs regarding the Jan's comments about command syntax modifications. Thanks & Regards Satya Valli Tata Consultancy Services Mailto:

[ovs-dev] [ovs-dev, v4] netdev-dpdk: Configurable Link State Change (LSC) detection mode

It is possible to change LSC detection mode to polling or interrupt mode for DPDK interfaces. The default is polling mode. To set interrupt mode, option dpdk-lsc-interrupt has to be set to true. In polling mode more processor time is needed, since the OVS repeatedly reads the link state with a

Re: [ovs-dev] [PATCH branch-2.7] docs: Use DPDK 16.11.4 stable release.

>From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev- >boun...@openvswitch.org] On Behalf Of Ian Stokes >Sent: Tuesday, January 30, 2018 10:07 AM >To: d...@openvswitch.org >Subject: [ovs-dev] [PATCH branch-2.7] docs: Use DPDK 16.11.4 stable release. > >Modify docs and travis linux build script

Re: [ovs-dev] [PATCH] docs: Update supported DPDK versions.

> -Original Message- > From: Kavanagh, Mark B > Sent: Friday, February 2, 2018 11:43 AM > To: Stokes, Ian ; d...@openvswitch.org > Subject: RE: [ovs-dev] [PATCH] docs: Update supported DPDK versions. > > >From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev- >

Re: [ovs-dev] [PATCH v3 3/3] xlate: call tnl_neigh_snoop() from terminate_native_tunnel()

Hi Justin, I rebased the patches to recent master. Please find them attached. Best regards, Zoltan > -Original Message- > From: Justin Pettit [mailto:jpet...@ovn.org] > Sent: Friday, February 02, 2018 12:00 AM > To: Ben Pfaff > Cc: Zoltán Balogh

[ovs-dev] [RFC PATCH 1/1]: Enhance conjunctive match support in OVN

From: Numan Siddique Presently, if a logical flow has possible conjunctive matches, OVN expression parser has support for that. But certain fields like ip4.src, ip4.dst are not considered as dimensions in the conjunctive matches. In order to support all the possible fields

Re: [ovs-dev] [PATCH V2] OF1.5/EXT-334 OXS/Extensible Flow Entry Statistics Support

I'm not going to change the established semantics of dump-flows, but I want a patch that applies and doesn't break tests before I provide any more detailed feedback. On Fri, Feb 02, 2018 at 04:00:44PM +, Jan Scheurich wrote: > Hi Ben, > > I would appreciate if you could comment on my general

Re: [ovs-dev] [PATCH 12/20] datapath: Add meter infrastructure

On Tue, Jan 30, 2018 at 3:40 PM, Greg Rose wrote: > From: Andy Zhou > > Upstream commit: > commit 96fbc13d7e770b542d2d1fcf700d0baadc6e8063 > Author: Andy Zhou > Date: Fri Nov 10 12:09:42 2017 -0800 > > openvswitch: Add meter

Re: [ovs-dev] [PATCH 18/20] datapath: use ktime_get_ts64() instead of ktime_get_ts()

On Tue, Jan 30, 2018 at 4:40 PM, Gregory Rose wrote: > On 1/30/2018 3:40 PM, Greg Rose wrote: >> >> From: Arnd Bergmann >> >> Upstream commit: >> commit 311af51dcb5629f04976a8e451673f77e3301041 >> Author: Arnd Bergmann >> Date:

Re: [ovs-dev] [PATCH 09/20] datapath: reliable interface indentification in port dumps

On Tue, Jan 30, 2018 at 3:40 PM, Greg Rose wrote: > From: Jiri Benc > > Upstream commit: > commit 9354d452034273a50a4fd703bea31e5d6b1fc20b > Author: Jiri Benc > Date: Thu Nov 2 17:04:37 2017 -0200 > > openvswitch:

Re: [ovs-dev] [PATCH 11/20] datapath: Add meter netlink definitions

On 2/2/2018 10:18 AM, Pravin Shelar wrote: On Tue, Jan 30, 2018 at 3:40 PM, Greg Rose wrote: From: Andy Zhou Upstream commit: commit 5794040647de4011598a6d005fdad95d24fd385b Author: Andy Zhou Date: Fri Nov 10 12:09:40 2017

[ovs-dev] [RFC PATCH 0/1] Enhance conjunctive match support in OVN

From: Numan Siddique Presently, if a logical flow has possible conjunctive matches, OVN expression parser has support for that. But certain fields like ip4.src, ip4.dst are not considered as dimensions in the conjunctive matches. In order to support all the possible fields

Re: [ovs-dev] [PATCH] memory: kill ovs-vswitchd under super

On Fri, Feb 02, 2018 at 12:37:58PM +, 王志克 wrote: > I also found that if once theres are lots of flows, the memory (RSS) usage of > OVS process would be quite high, 2~3GB. Even then the flows disappear later, > the memory still keeps. > > I am not sure how many people notices this, but if

Re: [ovs-dev] [ovs-dev, ovs-dev, v4] netdev-dpdk: Configurable Link State Change (LSC) detection mode

On 02.02.2018 17:05, Róbert Mulik wrote: > It is possible to change LSC detection mode to polling or interrupt mode > for DPDK interfaces. The default is polling mode. To set interrupt mode, > option dpdk-lsc-interrupt has to be set to true. > > In polling mode more processor time is needed,

Re: [ovs-dev] [PATCH 01/20] datapath: Fix netdev_master_upper_dev_link for 4.14

On Tue, Jan 30, 2018 at 3:40 PM, Greg Rose wrote: > An extended netlink ack has been added for 4.14 - add compat layer > changes so that it compiles for all kernels up to and including > 4.14. > > Signed-off-by: Greg Rose > --- > acinclude.m4

Re: [ovs-dev] [PATCH 01/20] datapath: Fix netdev_master_upper_dev_link for 4.14

On 2/2/2018 10:17 AM, Pravin Shelar wrote: On Tue, Jan 30, 2018 at 3:40 PM, Greg Rose wrote: An extended netlink ack has been added for 4.14 - add compat layer changes so that it compiles for all kernels up to and including 4.14. Signed-off-by: Greg Rose

Re: [ovs-dev] [PATCH 18/20] datapath: use ktime_get_ts64() instead of ktime_get_ts()

On 2/2/2018 10:18 AM, Pravin Shelar wrote: On Tue, Jan 30, 2018 at 4:40 PM, Gregory Rose wrote: On 1/30/2018 3:40 PM, Greg Rose wrote: From: Arnd Bergmann Upstream commit: commit 311af51dcb5629f04976a8e451673f77e3301041 Author: Arnd Bergmann

Re: [ovs-dev] [PATCH V2] OF1.5/EXT-334 OXS/Extensible Flow Entry Statistics Support

Please start by rebasing and reposting. On Fri, Feb 02, 2018 at 06:04:49PM +0530, Satyavalli Rama wrote: > Hi Ben, > > We didn't observed any test case breaks and we've updated the same with logs > in our previous conversations. > Could you please provide your inputs regarding the Jan's

Re: [ovs-dev] [PATCH V2] OF1.5/EXT-334 OXS/Extensible Flow Entry Statistics Support

Hi Ben, I would appreciate if you could comment on my general concerns with this patch. I think it is unwise to sacrifice the semantics of the established ovs-ofctl dump-flow command just in order to align the CLI commands with the particular re-arrangement of multipart requests messages in

Re: [ovs-dev] [PATCH] memory: kill ovs-vswitchd under super

Hi Zhike, On Fri, Feb 2, 2018 at 7:48 AM, Ben Pfaff wrote: > On Fri, Feb 02, 2018 at 12:37:58PM +, 王志克 wrote: >> I also found that if once theres are lots of flows, the memory (RSS) usage >> of OVS process would be quite high, 2~3GB. Even then the flows disappear >> later,

Re: [ovs-dev] [PATCH 00/20] Update and backport of upstream Linux

On Tue, Jan 30, 2018 at 3:40 PM, Greg Rose wrote: > Allow OVS to compile and build on Linux 4.14.x kernels. Added > necessary compatability layer changes to the respective patches > as required for our OOT build environment. > > Note that NSH and ERSPAN patches are not in

Re: [ovs-dev] [PATCH 11/20] datapath: Add meter netlink definitions

On Tue, Jan 30, 2018 at 3:40 PM, Greg Rose wrote: > From: Andy Zhou > > Upstream commit: > commit 5794040647de4011598a6d005fdad95d24fd385b > Author: Andy Zhou > Date: Fri Nov 10 12:09:40 2017 -0800 > > openvswitch: Add meter

[ovs-dev] [PATCH] ovn-controller: Document southbound database use and graceful termination.

A lot of people seem to think that "kill" gracefully terminates ovn-controller, but it doesn't, so this documentation at least provides something to point to. Signed-off-by: Ben Pfaff --- ovn/controller/ovn-controller.8.xml | 47 + 1 file

Re: [ovs-dev] [PATCH 00/20] Update and backport of upstream Linux

On 2/2/2018 10:20 AM, Pravin Shelar wrote: On Tue, Jan 30, 2018 at 3:40 PM, Greg Rose wrote: Allow OVS to compile and build on Linux 4.14.x kernels. Added necessary compatability layer changes to the respective patches as required for our OOT build environment. Note

Re: [ovs-dev] [PATCH v2 0/5] datapath: enable NSH support in kernel compat mode

On 1/31/2018 5:53 AM, Yi Yang wrote: v1->v2 - Fix compilation error in linux-3.10.107 This patch series is to backport NSH support patches in Linux net-next tree to OVS in order that it can support NSH in kernel compat mode. Yi Yang (5): datapath: ether: add NSH ethertype datapath:

[ovs-dev] [PATCH 1/4] ovs-vsctl: Remove superfluous OVS_VSCTL_CLEANUP from tests.

Since on_exit was introduced a long, long time ago, it has no longer been necessary to have individual calls to OVS_VSCTL_CLEANUP sprinkled everywhere in the test code. This change makes the tests easier to read. Signed-off-by: Ben Pfaff --- tests/ovs-vsctl.at | 321

Re: [ovs-dev] [PATCH] ovn-controller: Document southbound database use and graceful termination.

On Fri, Feb 02, 2018 at 12:07:26PM -0800, Han Zhou wrote: > On Fri, Feb 2, 2018 at 9:44 AM, Ben Pfaff wrote: > > > > A lot of people seem to think that "kill" gracefully terminates > > ovn-controller, but it doesn't, so this documentation at least provides > > something to point to.

Re: [ovs-dev] [PATCH] datapath-windows: Add trace level logs in conntrack for invalid ct state.

Looks good just a small nit: s/syn/SYN/g s/ack/ACK/g Also I would prefer if you drop the text `Invalid!`. Either just remove it or expand it, i.e.: "Invalid! ICMPhdr cannot be NULL" => "Invalid ICMP packet detected the header cannot be NULL" Thanks, Alin. -Mesaj original- De la:

[ovs-dev] [PATCH] datapath-windows: Add trace level logs in conntrack for invalid ct state.

Signed-off-by: Anand Kumar --- datapath-windows/ovsext/Conntrack-icmp.c | 1 + datapath-windows/ovsext/Conntrack-tcp.c | 4 datapath-windows/ovsext/Conntrack.c | 4 3 files changed, 9 insertions(+) diff --git a/datapath-windows/ovsext/Conntrack-icmp.c

Re: [ovs-dev] [PATCH 18/20] datapath: use ktime_get_ts64() instead of ktime_get_ts()

On Fri, Feb 2, 2018 at 7:31 PM, Gregory Rose wrote: > On 2/2/2018 10:18 AM, Pravin Shelar wrote: >> >> On Tue, Jan 30, 2018 at 4:40 PM, Gregory Rose >> >> This is done in compat code, can you move it to respective header file? > > > Yes - my own

Re: [ovs-dev] [PATCH v4 3/3] datapath-windows: Optimize conntrack lock implementation.

Thanks for the series Anand. I folded the following to suppress the warning: diff --git a/datapath-windows/ovsext/Conntrack.c b/datapath-windows/ovsext/Conntrack.c index c90c00074..43c9dd319 100644 --- a/datapath-windows/ovsext/Conntrack.c +++

[ovs-dev] [PATCH 4/4] ovs-vsctl: Add commands "add-bond-iface" and "del-bond-iface".

It was not too hard to build these commands using the database commands, but a few people have asked for them over the years, so here they are. Signed-off-by: Ben Pfaff --- NEWS | 1 + tests/ovs-vsctl.at | 60 +

[ovs-dev] [PATCH 3/4] NEWS: Consolidate ovs-vswitchd sections and fix indentation.

Signed-off-by: Ben Pfaff --- NEWS | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/NEWS b/NEWS index 8c360ba6cb9d..3b6ff74ad368 100644 --- a/NEWS +++ b/NEWS @@ -1,15 +1,14 @@ Post-v2.9.0 -- ovs-vswitchd: - * New options

[ovs-dev] [PATCH 2/4] ovs-vsctl: Use default socket name in tests.

By using the default socket name "db.sock", instead of "socket", we can avoid passing --db=unix:socket to all the ovs-vsctl invocations, which is kind of nice. Signed-off-by: Ben Pfaff --- tests/ovs-vsctl.at | 16 1 file changed, 8 insertions(+), 8 deletions(-)

Re: [ovs-dev] [PATCH v4 3/3] datapath-windows: Optimize conntrack lock implementation.

Acked-by: Alin Gabriel Serdean -Mesaj original- De la: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev-boun...@openvswitch.org] În numele Anand Kumar Trimis: Monday, January 29, 2018 8:28 PM Către: d...@openvswitch.org Subiect: [ovs-dev] [PATCH v4 3/3]

[ovs-dev] Infracciones y sanciones

OUTSOURCING Y SUS CONSECUENCIAS FISCALES Febrero 07 - webinar Interactivo Este webinar está diseñado para las empresas que prestan servicios de outsourcing de personal o las interesadas en contratar personal bajo la figura del outsourcing. TEMARIO: El Outsourcing en términos de la Ley

Re: [ovs-dev] [PATCH] ovn-controller: Document southbound database use and graceful termination.

On Fri, Feb 2, 2018 at 9:44 AM, Ben Pfaff wrote: > > A lot of people seem to think that "kill" gracefully terminates > ovn-controller, but it doesn't, so this documentation at least provides > something to point to. > > Signed-off-by: Ben Pfaff > --- >

Re: [ovs-dev] [PATCH 18/20] datapath: use ktime_get_ts64() instead of ktime_get_ts()

On 2/2/2018 12:13 PM, Arnd Bergmann wrote: On Fri, Feb 2, 2018 at 7:31 PM, Gregory Rose wrote: On 2/2/2018 10:18 AM, Pravin Shelar wrote: On Tue, Jan 30, 2018 at 4:40 PM, Gregory Rose This is done in compat code, can you move it to respective

Re: [ovs-dev] [PATCH] datapath-windows: Add trace level logs in conntrack for invalid ct state.

-Mesaj original- De la: Anand Kumar [mailto:kumaran...@vmware.com] Trimis: Saturday, February 3, 2018 12:28 AM Către: Alin Serdean ; d...@openvswitch.org Subiect: Re: [ovs-dev] [PATCH] datapath-windows: Add trace level logs in conntrack for invalid ct

Re: [ovs-dev] [PATCH v2] datapath-windows: Add trace level logs in conntrack for invalid ct state.

Much better  Acked-by: Alin Gabriel Serdean -Mesaj original- De la: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev-boun...@openvswitch.org] În numele Anand Kumar Trimis: Saturday, February 3, 2018 12:43 AM Către: d...@openvswitch.org Subiect: [ovs-dev] [PATCH v2]

Re: [ovs-dev] [PATCH] ovn-controller: processing matches precedes processing action in logical flow.

On Fri, Feb 02, 2018 at 10:50:29AM +0800, Guoshuai Li wrote: > This is to fix such a problem: > When the match field has "is_chassis_resident", and the chassis has no > resident, > and the action has meter or group, the group/meter ID is assigned. > > I hope to parse match field with the first,

[ovs-dev] Calendario del mes de Febrero

Buenos días Le hacemos llegar la programación de Febrero para que elija los temas que necesita usted y su equipo de trabajo. FEBRERO (3 hrs.) Cómo Implementar el Home Office en su Empresa – 01 de Febrero Actualización para Compradores Profesionales – Cómo ser el Mejor – 08 de Febrero

Re: [ovs-dev] [PATCH v2] datapath-windows: Add trace level logs in conntrack for invalid ct state.

Applied on master. Ty! -Mesaj original- De la: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev-boun...@openvswitch.org] În numele aserd...@ovn.org Trimis: Saturday, February 3, 2018 12:48 AM Către: 'Anand Kumar' ; d...@openvswitch.org Subiect: Re: [ovs-dev] [PATCH

[ovs-dev] [PATCH] util: Use lookup table to optimize hexit_value().

Daniel Alvarez Sanchez reported a significant overall speedup in ovn-northd due to a similar patch. Reported-by: Daniel Alvarez Sanchez Reported-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2018-February/046120.html Signed-off-by: Ben Pfaff ---

Re: [ovs-dev] [PATCH] datapath-windows: Add trace level logs in conntrack for invalid ct state.

Hi Alin, Thanks for the quick review. I will expand log messages to “Invalid XXX packet detected”. I’m not sure if I follow your comment about “s/syn/SYN/g”. Do you want me to change it to Uppercase? Thanks, Anand Kumar On 2/2/18, 2:06 PM, "Alin Serdean"

Re: [ovs-dev] [PATCH V3] rhel: Fix support for root user using DPDK

Well, is it ever useful to be able to drop unneeded capabilities while retaining the same uid/gid? It certainly sounds like a reasonable thing to want to do. I'm reluctant to apply this without at least considering that possibility. On Fri, Feb 02, 2018 at 12:30:50AM -0200, Marcos Felipe

[ovs-dev] [PATCH v2] Makefile.am: Use correct path separator for Windows

Signed-off-by: Shashank Ram --- Makefile.am | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Makefile.am b/Makefile.am index 5988c02..d397f65 100644 --- a/Makefile.am +++ b/Makefile.am @@ -148,7 +148,7 @@ ro_shell = printf '\043 Generated automatically --

Re: [ovs-dev] [PATCH] util: Use lookup table to optimize hexit_value().

Looks good to me, thanks. Reviewed-by: Yifeng Sun On Fri, Feb 2, 2018 at 3:16 PM, Ben Pfaff wrote: > Daniel Alvarez Sanchez reported a significant overall speedup in ovn-northd > due to a similar patch. > > Reported-by: Daniel Alvarez Sanchez

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: native tunnel using valid tun_src specified by flow or port options

Thank you for submitting a patch to improve Open vSwitch. I do not yet understand the purpose of this patch. Is it a bug fix or a new feature? Thanks, Ben. On Fri, Feb 02, 2018 at 02:08:52PM +0800, we...@ucloud.cn wrote: > From: wenxu > > native tunnel build tunnel with

[ovs-dev] [PATCH v2] datapath-windows: Add trace level logs in conntrack for invalid ct state.

Signed-off-by: Anand Kumar --- datapath-windows/ovsext/Conntrack-icmp.c | 1 + datapath-windows/ovsext/Conntrack-tcp.c | 4 datapath-windows/ovsext/Conntrack.c | 6 ++ 3 files changed, 11 insertions(+) diff --git a/datapath-windows/ovsext/Conntrack-icmp.c

Re: [ovs-dev] [PATCH] Makefile.am: Use correct path separator for Windows

From: Ben Pfaff Sent: Friday, February 2, 2018 2:07 PM To: Shashank Ram Cc: d...@openvswitch.org Subject: Re: [ovs-dev] [PATCH] Makefile.am: Use correct path separator for Windows On Thu, Feb 01, 2018 at 05:03:52PM -0800, Shashank Ram

[ovs-dev] Greetings

My name is Mrs. Sarah Adam, a woman of 75 years, I am in the sick bed now ready to die soon, but before I die I have an important massage that I want to tell you, very urgent. ___ dev mailing list d...@openvswitch.org

[ovs-dev] Fidelice a sus clientes y aumente sus ganancias

Haga que sus clientes vivan la experiencia inspirada en el Modelo Disney® Basado en el Modelo Disney®: Servicio de Excelencia para el éxito financiero 15 de Febrero- Dr. Arturo Ruiz Velasco Romero - 9am- 6pm Muchos de los objetivos del Quality Service de Disney®, se centran en ofrecer al

Re: [ovs-dev] [PATCH V2] ovn-controller: Reduce the number of flows by use conjunction action

On Wed, Oct 18, 2017 at 10:39:12AM +0800, wei wrote: > This patch convert ovn-sb lflow match expr "(1 or 2) and (3 or 4)" to > match 1 aciton connjunction(1, 1/2) > match 2 aciton connjunction(1, 1/2) > match 3 aciton connjunction(1, 2/2) > match 4 aciton connjunction(1, 2/2) > match conj_id=1,

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: native tunnel using valid tun_src specified by flow or port options

Hi ben, This patch can be a bugfix. The tunnel_src of packet maybe not the IP address set by gre port options:local_ip dpdk-br has two address 10.1.1.7/24 and 10.1.1.254/32 Interface gre options:local_ip="10.1.1.254" But the tunnel_src of tunnel packet is always 10.1.1.7 Also It can be a

Re: [ovs-dev] [PATCH] docs: Update supported DPDK versions.

>From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev- >boun...@openvswitch.org] On Behalf Of Ian Stokes >Sent: Tuesday, January 30, 2018 10:08 AM >To: d...@openvswitch.org >Subject: [ovs-dev] [PATCH] docs: Update supported DPDK versions. > >Update the OVS to DPDK release table to use the latest

Re: [ovs-dev] [PATCH] ovn-northd: Do not add lflows in lr_in_arp_resolve stage for disabled logical ports

On Fri, Feb 2, 2018 at 6:02 AM, Ben Pfaff wrote: > On Thu, Oct 26, 2017 at 06:56:46AM +0530, Numan Siddique wrote: > > On Thu, Oct 26, 2017 at 2:59 AM, Ben Pfaff wrote: > > > > > On Wed, Sep 27, 2017 at 09:36:18PM +0530, nusid...@redhat.com wrote: > > > > From: Numan