Re: [ovs-dev] ovn-controller is taking 100% CPU all the time in one deployment

2019-08-30 Thread Han Zhou
On Thu, Aug 29, 2019 at 11:36 PM Numan Siddique wrote: > > > > On Fri, Aug 30, 2019 at 1:04 AM Han Zhou wrote: >> >> >> >> On Thu, Aug 29, 2019 at 12:16 PM Numan Siddique wrote: >>> >>> >>> >>> On Fri, Aug 30, 2019 at 12:37 AM Han Zhou wrote: On Thu, Aug 29, 2019 at 11:40

Re: [ovs-dev] [ovs-discuss] ovn-controller is taking 100% CPU all the time in one deployment

2019-08-30 Thread Han Zhou
On Fri, Aug 30, 2019 at 1:25 PM Numan Siddique wrote: > > Hi Han, > > I am thinking of this approach to solve this problem. I still need to test it. > If you have any comments or concerns do let me know. > > > ** > diff --git a/northd/ovn-northd.c

Re: [ovs-dev] [ovs-discuss] ovn-controller is taking 100% CPU all the time in one deployment

2019-08-30 Thread Numan Siddique
Hi Han, I am thinking of this approach to solve this problem. I still need to test it. If you have any comments or concerns do let me know. ** diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index 9a282..a83b56362 100644 ---

Re: [ovs-dev] [ovs-discuss] ovn-controller is taking 100% CPU all the time in one deployment

2019-08-30 Thread Han Zhou
On Fri, Aug 30, 2019 at 6:46 AM Mark Michelson wrote: > > On 8/30/19 5:39 AM, Daniel Alvarez Sanchez wrote: > > On Thu, Aug 29, 2019 at 10:01 PM Mark Michelson wrote: > >> > >> On 8/29/19 2:39 PM, Numan Siddique wrote: > >>> Hello Everyone, > >>> > >>> In one of the OVN deployments, we are

[ovs-dev] [patch v2] conntrack: Fix 'reverse_nat_packet()' variable datatype.

2019-08-30 Thread Darrell Ball
The datatype 'pad' in the function 'reverse_nat_packet()' was incorrectly declared as 'char' instead of 'uint8_t'. This can affect reverse natting of icmpX packets with padding > 127 bytes. At the same time, add some comments regarding 'extract_l3_ipvX' usage in this function. Found by

Re: [ovs-dev] Regarding TSO using AF_PACKET in OVS

2019-08-30 Thread William Tu
Hi Ramana, I'm trying to understand your setup. On Wed, Aug 28, 2019 at 4:11 AM Ramana Reddy wrote: > > Hi Ben, Justin, Jesse and All, > > Hope everyone doing great. > > During my work, I create a socket using AF_PACKET with virtio_net_hdr and > filled all the fields in the virtio_net_hdr > and

Re: [ovs-dev] [PATCH branch-2.8 2/2] datapath: Clear the L4 portion of the key for "later" fragments

2019-08-30 Thread Gregory Rose
On 8/29/2019 10:44 PM, Justin Pettit wrote: On Aug 29, 2019, at 6:46 PM, Gregory Rose wrote: On 8/29/2019 3:25 PM, Justin Pettit wrote: On Aug 29, 2019, at 10:55 AM, Greg Rose wrote: diff --git a/datapath/flow.c b/datapath/flow.c index 083288f..92fc6ac 100644 --- a/datapath/flow.c +++

Re: [ovs-dev] [PATCH v9] ovsdb-tool: Convert clustered db to standalone db.

2019-08-30 Thread aginwala
Thanks Han. Addressed the minor comment in v10. On Thu, Aug 29, 2019 at 10:31 PM Han Zhou wrote: > Thanks for the update. > > On Thu, Aug 29, 2019 at 6:20 PM wrote: > > > > From: Aliasgar Ginwala > > > > Add support in ovsdb-tool for migrating clustered dbs to standalone dbs. > > E.g. usage

[ovs-dev] [PATCH v10] ovsdb-tool: Convert clustered db to standalone db.

2019-08-30 Thread amginwal
From: Aliasgar Ginwala Add support in ovsdb-tool for migrating clustered dbs to standalone dbs. E.g. usage to migrate nb/sb db to standalone db from raft: ovsdb-tool cluster-to-standalone ovnnb_db.db ovnnb_db_cluster.db Signed-off-by: Aliasgar Ginwala Acked-by: Han Zhou ---

Re: [ovs-dev] [ovs-discuss] ovn-controller is taking 100% CPU all the time in one deployment

2019-08-30 Thread Mark Michelson
On 8/30/19 5:39 AM, Daniel Alvarez Sanchez wrote: On Thu, Aug 29, 2019 at 10:01 PM Mark Michelson wrote: On 8/29/19 2:39 PM, Numan Siddique wrote: Hello Everyone, In one of the OVN deployments, we are seeing 100% CPU usage by ovn-controllers all the time. After investigations we found the

Re: [ovs-dev] [PATCH v7] Detailed packet drop statistics per dpdk and vhostuser ports

2019-08-30 Thread Ilya Maximets
On 28.08.2019 15:58, Sriram Vatala wrote: > OVS may be unable to transmit packets for multiple reasons and > today there is a single counter to track packets dropped due to > any of those reasons. The most common reason is that a VM is > unable to read packets fast enough causing the vhostuser

[ovs-dev] [PATCH ovn] Fix configure help string for ovs source/build path

2019-08-30 Thread Lorenzo Bianconi
Signed-off-by: Lorenzo Bianconi --- acinclude.m4 | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/acinclude.m4 b/acinclude.m4 index d5552d708..31e4f0b54 100644 --- a/acinclude.m4 +++ b/acinclude.m4 @@ -1212,10 +1212,10 @@ dnl dnl Check for OVS sources

Re: [ovs-dev] [PATCH v2] Make pid_exists() more robust against empty pid argument

2019-08-30 Thread Ilya Maximets
On 29.08.2019 17:46, Ben Pfaff wrote: > On Tue, Aug 27, 2019 at 02:43:21PM +0300, Ilya Maximets wrote: >> On 14.08.2019 18:47, Michele Baldessari wrote: >>> In some of our destructive testing of ovn-dbs inside containers managed >>> by pacemaker we reached a situation where /var/run/openvswitch

Re: [ovs-dev] [PATCH v7] Detailed packet drop statistics per dpdk and vhostuser ports

2019-08-30 Thread Sriram Vatala via dev
Hi All, Please consider this as a gentle remainder. Thanks & Regards, Sriram. -Original Message- From: Sriram Vatala Sent: 28 August 2019 18:29 To: ovs-dev@openvswitch.org; i.maxim...@samsung.com Cc: b...@ovn.org; ian.sto...@intel.com; Sriram Vatala Subject: [PATCH v7] Detailed packet

[ovs-dev] ovs kernel sampling has memleak issue

2019-08-30 Thread Gao Zhenyu
Hi Found a potential memleak issue when I enable bridge ipfix and transmit traffic through geneve tunnel. The kernel is centos7.5 3.10.0-862.9.1.el7.x86_64 memleak show a lot of malloc call-stack : unreferenced object 0x93f473e64000 (size 512): comm "softirq", pid 0, jiffies

Re: [ovs-dev] [PATCH v2] flow: miniflow_extract metadata branchless optimization

2019-08-30 Thread Yanqin Wei (Arm Technology China)
Hi I run a basic nic2nic case for compiler branch stripping. The improvement is around 0.8%. It is observed that 0.37% branch miss in case of directly call miniflow_extract__(but it is not inline here) and 0.27% branch miss in case of enable multi-instance for miniflow_extract. But downside

Re: [ovs-dev] [ovs-discuss] ovn-controller is taking 100% CPU all the time in one deployment

2019-08-30 Thread Daniel Alvarez Sanchez
On Thu, Aug 29, 2019 at 10:01 PM Mark Michelson wrote: > > On 8/29/19 2:39 PM, Numan Siddique wrote: > > Hello Everyone, > > > > In one of the OVN deployments, we are seeing 100% CPU usage by > > ovn-controllers all the time. > > > > After investigations we found the below > > > > -

Re: [ovs-dev] ovn-controller is taking 100% CPU all the time in one deployment

2019-08-30 Thread Numan Siddique
On Fri, Aug 30, 2019 at 1:04 AM Han Zhou wrote: > > > On Thu, Aug 29, 2019 at 12:16 PM Numan Siddique > wrote: > >> >> >> On Fri, Aug 30, 2019 at 12:37 AM Han Zhou wrote: >> >>> >>> >>> On Thu, Aug 29, 2019 at 11:40 AM Numan Siddique >>> wrote: >>> > >>> > Hello Everyone, >>> > >>> > In one