Re: [ovs-dev] [PATCH] Fix compiling error in netbsd

On Tue, Dec 6, 2016 at 12:01 PM, Ben Pfaff <b...@ovn.org> wrote: > On Tue, Dec 06, 2016 at 12:04:00AM -0500, Hui Kang wrote: >> In some netbsd version, RTF_LLINFO is undefined >> >> Signed-off-by: Hui Kang <hkang.sun...@gmail.com> > > Applied to master, th

Re: [ovs-dev] [PATCH v2] Fix missing system-id in INSTALL.Docker

I like your version better. I am ok abandoning this patch. Thanks. - Hui On Tue, Oct 4, 2016 at 7:01 PM, Ben Pfaff <b...@ovn.org> wrote: > On Tue, Oct 04, 2016 at 04:33:02PM -0400, Hui Kang wrote: >> Signed-off-by: Hui Kang <hkang.sun...@gmail.com> >> --- >> INS

[ovs-dev] [PATCH v2] Fix missing system-id in INSTALL.Docker

Signed-off-by: Hui Kang <hkang.sun...@gmail.com> --- INSTALL.Docker.md | 8 1 file changed, 8 insertions(+) diff --git a/INSTALL.Docker.md b/INSTALL.Docker.md index b62922d..4606354 100644 --- a/INSTALL.Docker.md +++ b/INSTALL.Docker.md @@ -83,6 +83,14 @@ ovs-vsctl set Open_v

Re: [ovs-dev] [PATCH] Fix missing system-id in INSTALL.Docker

Ben Pfaff <b...@ovn.org> wrote on 10/04/2016 12:25:50 AM: > From: Ben Pfaff <b...@ovn.org> > To: Hui Kang/Watson/IBM@IBMUS > Cc: dev@openvswitch.org, Hui Kang <hkang.sun...@gmail.com> > Date: 10/04/2016 12:26 AM > Subject: Re: [ovs-dev] [PATCH] Fix missing syste

Re: [ovs-dev] [PATCH] Fix missing system-id in INSTALL.Docker

Ben Pfaff <b...@ovn.org> wrote on 10/03/2016 09:23:29 PM: > From: Ben Pfaff <b...@ovn.org> > To: Hui Kang/Watson/IBM@IBMUS > Cc: Hui Kang <hkang.sun...@gmail.com>, dev@openvswitch.org > Date: 10/03/2016 09:23 PM > Subject: Re: [ovs-dev] [PATCH] Fix missing syste

Re: [ovs-dev] [PATCH] Fix missing system-id in INSTALL.Docker

"dev" <dev-boun...@openvswitch.org> wrote on 10/03/2016 03:57:21 PM: > From: Ben Pfaff <b...@ovn.org> > To: Hui Kang <hkang.sun...@gmail.com> > Cc: dev@openvswitch.org > Date: 10/03/2016 03:57 PM > Subject: Re: [ovs-dev] [PATCH] Fix missing system-id i

[ovs-dev] [PATCH] Fix missing system-id in INSTALL.Docker

Signed-off-by: Hui Kang <ka...@us.ibm.com> --- INSTALL.Docker.md | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/INSTALL.Docker.md b/INSTALL.Docker.md index b62922d..5cb49d0 100644 --- a/INSTALL.Docker.md +++ b/INSTALL.Docker.md @@ -69,6 +69,8 @@ gets c

Re: [ovs-dev] [PATCH] ovn: Delete stale MAC_Bindings that result in Referential Integrity Violation

"dev" wrote on 08/24/2016 03:10:29 AM: > From: Chandra Sekhar Vejendla/San Jose/IBM@IBMUS > To: dev@openvswitch.org > Date: 08/24/2016 03:11 AM > Subject: [ovs-dev] [PATCH] ovn: Delete stale MAC_Bindings that > result in Referential Integrity Violation > Sent by:

Re: [ovs-dev] [PATCH, v5] Scanning only changed entries in the ovnsb

"dev" <dev-boun...@openvswitch.org> wrote on 08/06/2016 08:38:27 AM: > From: Ryan Moats/Omaha/IBM@IBMUS > To: Hui Kang <hkang.sun...@gmail.com> > Cc: dev@openvswitch.org > Date: 08/06/2016 08:38 AM > Subject: Re: [ovs-dev] [PATCH, v5] Scanning only changed

[ovs-dev] [PATCH, v5] Scanning only changed entries in the ovnsb

ed-off-by: Hui Kang <ka...@us.ibm.com> --- ovn/northd/ovn-northd.c | 100 1 file changed, 68 insertions(+), 32 deletions(-) diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c index 716f123..32c14a2 100644 --- a/ovn/northd/ovn-northd

Re: [ovs-dev] [PATCH, v4] Scanning only changed entries in the ovnsb

"dev" <dev-boun...@openvswitch.org> wrote on 07/27/2016 05:53:17 PM: > From: Ben Pfaff <b...@ovn.org> > To: Hui Kang <hkang.sun...@gmail.com> > Cc: dev@openvswitch.org > Date: 07/27/2016 05:53 PM > Subject: Re: [ovs-dev] [PATCH, v4] Scanning only c

Re: [ovs-dev] [ovs-dev,ovn-controller] Physical: persist tunnels

ts <rmo...@us.ibm.com> > > > > Acked-by: Flavio Fernandes <fla...@flaviof.com> > > Tested-by: Flavio Fernandes <fla...@flaviof.com> > > > > Tested this fix with a cluster of 50 hosts. > > Acked-by: Liran Schour <

Re: [ovs-dev] [PATCH v3] Scanning only changed entries in the ovnsb

Ben Pfaff <b...@ovn.org> wrote on 07/27/2016 12:30:25 PM: > From: Ben Pfaff <b...@ovn.org> > To: Hui Kang/Watson/IBM@IBMUS > Cc: Hui Kang <hkang.sun...@gmail.com>, dev@openvswitch.org > Date: 07/27/2016 12:30 PM > Subject: Re: [ovs-dev] [PATCH v3] Scanning

[ovs-dev] [PATCH, v4] Scanning only changed entries in the ovnsb

- Improve performance by scanning only changed port binding entries when determining whether to mark the logical switch port up or down v3->v4: - Add an initialization function to scan all entries in Port_binding table when ovn-northd restarts or fails over Signed-off-by: Hui Kang

Re: [ovs-dev] [PATCH v3] Scanning only changed entries in the ovnsb

Russell Bryant <russ...@ovn.org> wrote on 07/26/2016 03:50:44 PM: > From: Russell Bryant <russ...@ovn.org> > To: Hui Kang/Watson/IBM@IBMUS > Cc: Ben Pfaff <b...@ovn.org>, Hui Kang <hkang.sun...@gmail.com>, ovs > dev <dev@openvswitch.org> > Date

Re: [ovs-dev] [PATCH v3] Scanning only changed entries in the ovnsb

"dev" <dev-boun...@openvswitch.org> wrote on 07/26/2016 02:20:27 PM: > From: Ben Pfaff <b...@ovn.org> > To: Hui Kang <hkang.sun...@gmail.com> > Cc: dev@openvswitch.org > Date: 07/26/2016 02:20 PM > Subject: Re: [ovs-dev] [PATCH v3] Scanning only c

Re: [ovs-dev] [PATCH v3] Scanning only changed entries in the ovnsb

"dev" <dev-boun...@openvswitch.org> wrote on 07/26/2016 02:20:27 PM: > From: Ben Pfaff <b...@ovn.org> > To: Hui Kang <hkang.sun...@gmail.com> > Cc: dev@openvswitch.org > Date: 07/26/2016 02:20 PM > Subject: Re: [ovs-dev] [PATCH v3] Scanning only c

Re: [ovs-dev] [PATCH v3] [OVN-northd] Rename nbs/nbr port names tonbsp/nbrp

"dev" <dev-boun...@openvswitch.org> wrote on 07/19/2016 04:15:56 PM: > From: Justin Pettit <jpet...@ovn.org> > To: Hui Kang <hkang.sun...@gmail.com> > Cc: dev@openvswitch.org > Date: 07/19/2016 04:16 PM > Subject: Re: [ovs-dev] [PATCH v3] [OVN-north

[ovs-dev] [PATCH v3] [OVN-northd] Rename nbs/nbr port names to nbsp/nbrp

These variables indicate ports in nb switches or routers. Signed-off-by: Hui Kang <ka...@us.ibm.com> -- v2->v3: - rebase agains master branch v1->v2: - modify commit message --- ovn/northd/ovn-northd.c | 184 1 file changed, 92 inser

[ovs-dev] [PATCH v2] [OVN-northd] Rename nbs/nbr port names to nbsp/nbrp

These variables indicate ports in nb switches or routers. Signed-off-by: Hui Kang <ka...@us.ibm.com> -- v1->v2: - modify commit message --- ovn/northd/ovn-northd.c | 196 1 file changed, 98 insertions(+), 98 deletions(-) diff --git a/o

Re: [ovs-dev] [PATCH] [OVN-northd] Rename nbs/nbr to nbsp/nbrp

To: Justin Pettit <jpet...@ovn.org> >> Cc: dev@openvswitch.org, Hui Kang <hkang.sun...@gmail.com> >> Date: 07/02/2016 12:25 PM >> Subject: Re: [ovs-dev] [PATCH] [OVN-northd] Rename nbs/nbr to nbsp/nbrp >> Sent by: "dev" <dev-boun...@openvswitch.org>

Re: [ovs-dev] [PATCH v2] Scanning only changed entries in the ovnsb

"dev" <dev-boun...@openvswitch.org> wrote on 07/16/2016 10:47:14 PM: > From: Ryan Moats/Omaha/IBM@IBMUS > To: Hui Kang <hkang.sun...@gmail.com> > Cc: dev@openvswitch.org > Date: 07/16/2016 10:47 PM > Subject: Re: [ovs-dev] [PATCH v2] Scanning only changed

[ovs-dev] [PATCH v3] Scanning only changed entries in the ovnsb

Improve performance by scanning only changed port binding entries when determining whether to mark the logical switch port up or down Signed-off-by: Hui Kang <ka...@us.ibm.com> --- ovn/northd/ovn-northd.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ovn/nort

[ovs-dev] [PATCH v2] Scanning only changed entries in the ovnsb

Improve performance by scanning only changed entries in ovnsb Signed-off-by: Hui Kang <ka...@us.ibm.com> --- ovn/northd/ovn-northd.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c index 7ce509d..bb6b853 100644 ---

[ovs-dev] [PATCH] Scanning only changed entries in the ovnsb

Improve performance by scanning only changed entries in ovnsb Signed-off-by: Hui Kang <ka...@us.ibm.com> --- ovn/northd/ovn-northd.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c index 7ce509d..bb6b853 100644 ---

[ovs-dev] [PATCH] Scanning only changed entries in the ovnsb

Improve performance by scanning only changed entries in ovnsb Signed-off-by: Hui Kang <ka...@us.ibm.com> --- ovn/northd/ovn-northd.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c index 7ce509d..8a73a3a 100644 ---

Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd on creating and binding large number of lports

Ryan Moats/Omaha/IBM wrote on 06/25/2016 09:07:39 PM: > From: Ryan Moats/Omaha/IBM > To: Hui Kang/Watson/IBM@IBMUS > Cc: Ben Pfaff <b...@ovn.org>, dev@openvswitch.org > Date: 06/25/2016 09:07 PM > Subject: Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd >

Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd on creating and binding large number of lports

Ryan Moats/Omaha/IBM wrote on 06/26/2016 08:53:19 PM: > From: Ryan Moats/Omaha/IBM > To: Hui Kang/Watson/IBM@IBMUS > Cc: Ben Pfaff <b...@ovn.org>, dev@openvswitch.org > Date: 06/26/2016 08:53 PM > Subject: Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd >

Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd on creating and binding large number of lports

Ryan Moats/Omaha/IBM wrote on 06/25/2016 09:07:39 PM: > From: Ryan Moats/Omaha/IBM > To: Hui Kang/Watson/IBM@IBMUS > Cc: Ben Pfaff <b...@ovn.org>, dev@openvswitch.org > Date: 06/25/2016 09:07 PM > Subject: Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd >

Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd on creating and binding large number of lports

> > > > Actually, I take that back. The cycles/port for all the cases above > > demonstrate only slightly nonlinear scaling: 200/25 is 8 Mcycles/port, > > 1200/125 is 9.6 Mcycles/port. > > > > So the issue is not that it does not scale. The issue is that it is > > slow. > > Er? When I do the

Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd on creating and binding large number of lports

Ryan Moats/Omaha/IBM wrote on 06/24/2016 10:51:10 PM: > From: Ryan Moats/Omaha/IBM > To: Hui Kang/Watson/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 06/24/2016 10:51 PM > Subject: Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd > on creating and binding larg

Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd on creating and binding large number of lports

Ben Pfaff <b...@ovn.org> wrote on 06/24/2016 02:24:12 PM: > From: Ben Pfaff <b...@ovn.org> > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: Hui Kang/Watson/IBM@IBMUS, Justin Pettit <jpet...@ovn.org>, > Yusheng Wang <yshw...@vmware.com>, dev@openvswitch.org > Date:

[ovs-dev] [PATCH] [OVN-northd] Rename nbs/nbr to nbsp/nbrp

These variables indicate ports in nb switches or routers. Signed-off-by: Hui Kang <ka...@us.ibm.com> --- ovn/northd/ovn-northd.c | 196 1 file changed, 98 insertions(+), 98 deletions(-) diff --git a/ovn/northd/ovn-northd.c b/ovn/nort

Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd on creating and binding large number of lports

> > My question is whether there is any particular reason to allocate that > > number > > of (struct ovn_port)? It seems to me there is room in this code to > optimize > > for performance. Thanks. > > > > - Hui > > > > > > [1] > > >

Re: [ovs-dev] [OVN] Potential scalability bug in ovn-northd on creating and binding large number of lports

Hi, Ryan, Thanks for your comments; reply inline. - Hui Ryan Moats/Omaha/IBM wrote on 06/24/2016 08:12:38 AM: > From: Ryan Moats/Omaha/IBM > To: Hui Kang/Watson/IBM@IBMUS, "Ben Pfaff" <b...@ovn.org>, "Justin > Pettit" <jpet...@ovn.org>, &quo

[ovs-dev] [OVN] Potential scalability bug in ovn-northd on creating and binding large number of lports

Hi, In our scalability test for OVN, we observed an in-scalable behaviour of the ovn-northd process: the time binding a logical port increases as # of large port increasing, regardless of whether logical ports belong to the same logical switch. The most suspicious function in causing this issue

Re: [ovs-dev] [PATCH] OVN: initial patch of datalog engine

Yusheng Wang <yshw...@vmware.com> wrote on 06/19/2016 11:12:49 PM: > From: Yusheng Wang <yshw...@vmware.com> > To: Hui Kang/Watson/IBM@IBMUS > Cc: "dev@openvswitch.org" <dev@openvswitch.org> > Date: 06/19/2016 11:13 PM > Subject: Re: [ovs-dev] [

Re: [ovs-dev] [PATCH] OVN: initial patch of datalog engine

ang <yshw...@vmware.com> > To: Hui Kang/Watson/IBM@IBMUS > Cc: "dev@openvswitch.org" <dev@openvswitch.org> > Date: 06/17/2016 04:40 AM > Subject: Re: [ovs-dev] [PATCH] OVN: initial patch of datalog engine > > > The idea is to write ovn-northd in nlog program. In

Re: [ovs-dev] [PATCH] OVN: initial patch of datalog engine

Hui Kang/Watson/IBM wrote on 06/17/2016 11:04:21 AM: > From: Hui Kang/Watson/IBM > To: Yusheng Wang <yshw...@vmware.com> > Cc: Ryan Moats/Omaha/IBM@IBMUS, "dev@openvswitch.org" <dev@openvswitch.org> > Date: 06/17/2016 11:04 AM > Subject: Re: [ovs-dev]

Re: [ovs-dev] [PATCH] OVN: initial patch of datalog engine

"dev" wrote on 06/16/2016 10:58:58 PM: > From: Yusheng Wang > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: "dev@openvswitch.org" > Date: 06/16/2016 10:59 PM > Subject: Re: [ovs-dev] [PATCH] OVN: initial patch of datalog engine >

Re: [ovs-dev] [PATCH] OVN: initial patch of datalog engine

Yusheng Wang <yshw...@vmware.com> wrote on 06/17/2016 04:30:02 AM: > From: Yusheng Wang <yshw...@vmware.com> > To: Hui Kang/Watson/IBM@IBMUS > Cc: "dev@openvswitch.org" <dev@openvswitch.org> > Date: 06/17/2016 04:40 AM > Subject: Re: [ovs-dev] [

Re: [ovs-dev] [PATCH V5] Function tracer to trace all function calls

Hi, Daniele, I am also interesting in profiling the ovs process and would like to participate in the discussion. I am using oprofile-1.1.0 to profiling the ovn-northd process. I believe oprofile-1.1.0 share the same underlying profiling mechanism as Linux perf (correct me if I am wrong).

Re: [ovs-dev] [PATCH monitor_cond V7 00/10] Conditional monitor implementation

"dev" wrote on 06/13/2016 07:19:25 AM: > From: Liran Schour > To: b...@ovn.org > Cc: d...@openvswitch.com > Date: 06/13/2016 08:29 AM > Subject: [ovs-dev] [PATCH monitor_cond V7 00/10] Conditional monitor > implementation > Sent by: "dev"

[ovs-dev] [PATCH] ovn-architecture.7.xml: Fix ovn-controller behavior in VIF life cycle

Signed-off-by: Hui Kang <ka...@us.ibm.com> --- ovn/ovn-architecture.7.xml | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/ovn/ovn-architecture.7.xml b/ovn/ovn-architecture.7.xml index f8a348b..72786bc 100644 --- a/ovn/ovn-architecture.7.xml +++ b/o

[ovs-dev] [PATCH] Display correct ovnsb sock location in the ovnsb-ctl help message

Signed-off-by: Hui Kang <ka...@us.ibm.com> --- ovn/utilities/ovn-sbctl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ovn/utilities/ovn-sbctl.c b/ovn/utilities/ovn-sbctl.c index a257729..a888333 100644 --- a/ovn/utilities/ovn-sbctl.c +++ b/ovn/utilities/ovn-s

Re: [ovs-dev] [PATCH] ovn-architecture.7.xml: Clarify the definition of Binding table

"dev" <dev-boun...@openvswitch.org> wrote on 03/24/2016 03:29:28 PM: > From: Ben Pfaff <b...@ovn.org> > To: Hui Kang <hkang.sun...@gmail.com> > Cc: dev@openvswitch.org > Date: 03/24/2016 03:29 PM > Subject: Re: [ovs-dev] [PATCH] ovn-architecture.7.xml

[ovs-dev] [PATCH] ovn-architecture.7.xml: Clarify the definition of Binding table

For the sake of clarity Signed-off-by: Hui Kang <ka...@us.ibm.com> --- ovn/ovn-architecture.7.xml | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/ovn/ovn-architecture.7.xml b/ovn/ovn-architecture.7.xml index 13acaf5..a588a0e 100644 --- a/ovn/ovn-architecture

Re: [ovs-dev] [PATCH] Use southbound database when saying Binding table

-Ben Pfaff <b...@ovn.org> wrote: - To: Hui Kang <hkang.sun...@gmail.com> From: Ben Pfaff <b...@ovn.org> Date: 03/23/2016 02:22PM Cc: dev@openvswitch.org, Hui Kang/Watson/IBM@IBMUS Subject: Re: [ovs-dev] [PATCH] Use southbound database when saying Binding table On Wed,

[ovs-dev] [PATCH] Use southbound database when saying Binding table

For the sake of clarity Signed-off-by: Hui Kang <ka...@us.ibm.com> --- ovn/ovn-architecture.7.xml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ovn/ovn-architecture.7.xml b/ovn/ovn-architecture.7.xml index 13acaf5..659c599 100644 --- a/ovn/ovn-architecture.7.xml