[ovs-dev] [OVN] DB backup and restore

2020-07-29 Thread Tony Liu
Hi, There is any guidance to backup and restore OVN nb-db and sb-db? Is /var/lib/openvswitch/ovn-[ns]b/ovn[ns]b.db the only database file? For 3-node DB cluster, is replication 3 (the data is replicated onto All 3 nodes)? Are DB files on 3 nodes identical? If I stop a DB follower and empty

Re: [ovs-dev] [PATCH ovn] expr.c: Fix argument type of expr_write_scope().

2020-07-29 Thread Numan Siddique
On Thu, Jul 30, 2020 at 9:58 AM Han Zhou wrote: > On Wed, Jul 29, 2020 at 9:03 PM Ankur Sharma > wrote: > > > > Hi Han, > > > > Changes look fine. > > > > Acked-by: Ankur Sharma > > > > Regards, > > Ankur > > Thanks Ankur, I applied this to master. > > > > >

Re: [ovs-dev] [PATCH net-next v3 2/2] net: openvswitch: make masks cache size configurable

2020-07-29 Thread Tonghao Zhang
On Wed, Jul 29, 2020 at 9:27 PM Eelco Chaudron wrote: > > This patch makes the masks cache size configurable, or with > a size of 0, disable it. > > Reviewed-by: Paolo Abeni > Signed-off-by: Eelco Chaudron > --- > Changes in v3: > - Use is_power_of_2() function > - Use array_size() function >

Re: [ovs-dev] [PATCH ovn] expr.c: Fix argument type of expr_write_scope().

2020-07-29 Thread Han Zhou
On Wed, Jul 29, 2020 at 9:03 PM Ankur Sharma wrote: > > Hi Han, > > Changes look fine. > > Acked-by: Ankur Sharma > > Regards, > Ankur Thanks Ankur, I applied this to master. > > From: dev on behalf of Han Zhou < hz...@ovn.org> > Sent: Wednesday, July 29, 2020

Re: [ovs-dev] [PATCH ovn] expr.c: Fix argument type of expr_write_scope().

2020-07-29 Thread Ankur Sharma
Hi Han, Changes look fine. Acked-by: Ankur Sharma Regards, Ankur From: dev on behalf of Han Zhou Sent: Wednesday, July 29, 2020 6:44 PM To: d...@openvswitch.org Cc: Han Zhou Subject: [ovs-dev] [PATCH ovn] expr.c: Fix argument type of expr_write_scope().

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

2020-07-29 Thread Ankur Sharma
Hi Numan, Daniel, Lucas, Thank  you so much for the feedback and providing your inputs. I went the through the bug that was being referenced and Numan's inputs and following is my take on it. Its a lengthy email, if you just want to look at the suggestions for fix then please scroll to the

[ovs-dev] [PATCH ovn] expr.c: Fix argument type of expr_write_scope().

2020-07-29 Thread Han Zhou
There is compile error introduced by the commit 2054d01247. *** ../lib/expr.c:3328:6: error: symbol 'expr_type_check' redeclared with different type (originally declared at ../include/ovn/expr.h:481) - incompatible argument 4 (different signedness) Makefile:1971: recipe for target 'lib/expr.lo'

Re: [ovs-dev] [PATCH ovn v3 0/5] Add ECMP symmetric replies

2020-07-29 Thread Mark Michelson
On 7/29/20 9:43 AM, Numan Siddique wrote: On Wed, Jul 29, 2020 at 1:54 AM Mark Michelson > wrote: In some cases, it is important that traffic that arrived over an ECMP route has its replies sent out over that same route. This series of commits adds

[ovs-dev] [PATCH branch-2.13 1/2] Set release date for 2.13.1.

2020-07-29 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 11 ++- debian/changelog | 2 +- 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/NEWS b/NEWS index ac7dc670c..fc4304985 100644 --- a/NEWS +++ b/NEWS @@ -1,9 +1,10 @@ -v2.13.1 - xx xxx +v2.13.1 - 30 Jul 2020

[ovs-dev] [PATCH branch-2.12 1/2] Set release date for 2.12.1.

2020-07-29 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 ++- debian/changelog | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index d00cc22df..2c7aa9b59 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,6 @@ -v2.12.1 - xx xxx +v2.12.1 - 30 Jul 2020

[ovs-dev] [PATCH branch-2.13 2/2] Prepare for 2.13.2.

2020-07-29 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 +++ configure.ac | 2 +- debian/changelog | 6 ++ 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index fc4304985..128db0f61 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v2.13.2 - xx xxx

[ovs-dev] [PATCH branch-2.11 2/2] Prepare for 2.11.5.

2020-07-29 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 +++ configure.ac | 2 +- debian/changelog | 6 ++ 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index 68224b4fa..87f24b52c 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v2.11.5 - xx xxx

[ovs-dev] [PATCH branch-2.10 0/2] Release patches for v2.10.5.

2020-07-29 Thread Ilya Maximets
Ilya Maximets (2): Set release date for 2.10.5. Prepare for 2.10.6. NEWS | 6 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 13 insertions(+), 3 deletions(-) -- 2.25.4 ___ dev mailing list

[ovs-dev] [PATCH branch-2.10 2/2] Prepare for 2.10.6.

2020-07-29 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 +++ configure.ac | 2 +- debian/changelog | 6 ++ 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index 14721913e..87665d047 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v2.10.6 - xx xxx

[ovs-dev] [PATCH branch-2.10 1/2] Set release date for 2.10.5.

2020-07-29 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 ++- debian/changelog | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index 8715349ba..14721913e 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,6 @@ -v2.10.5 - xx xxx +v2.10.5 - 30 Jul 2020

[ovs-dev] [PATCH branch-2.11 0/2] Release patches for v2.11.4.

2020-07-29 Thread Ilya Maximets
Ilya Maximets (2): Set release date for 2.11.4. Prepare for 2.11.5. NEWS | 6 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 13 insertions(+), 3 deletions(-) -- 2.25.4 ___ dev mailing list

[ovs-dev] [PATCH branch-2.12 0/2] Release patches for v2.12.1.

2020-07-29 Thread Ilya Maximets
Ilya Maximets (2): Set release date for 2.12.1. Prepare for 2.12.2. NEWS | 6 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 13 insertions(+), 3 deletions(-) -- 2.25.4 ___ dev mailing list

[ovs-dev] [PATCH branch-2.11 1/2] Set release date for 2.11.4.

2020-07-29 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 ++- debian/changelog | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index cd0c889c1..68224b4fa 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,6 @@ -v2.11.4 - xx xxx +v2.11.4 - 30 Jul 2020

[ovs-dev] [PATCH branch-2.13 0/2] Release patches for v2.13.1.

2020-07-29 Thread Ilya Maximets
Ilya Maximets (2): Set release date for 2.13.1. Prepare for 2.13.2. NEWS | 14 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 17 insertions(+), 7 deletions(-) -- 2.25.4 ___ dev mailing

[ovs-dev] [PATCH branch-2.12 2/2] Prepare for 2.12.2.

2020-07-29 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 +++ configure.ac | 2 +- debian/changelog | 6 ++ 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index 2c7aa9b59..459cded4d 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v2.12.2 - xx xxx

[ovs-dev] [PATCH branch-2.9 1/2] Set release date for 2.9.7.

2020-07-29 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 ++- debian/changelog | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index a6f1586ac..887dda24b 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,6 @@ -v2.9.7 - xx xxx +v2.9.7 - 30 Jul 2020

[ovs-dev] [PATCH branch-2.9 2/2] Prepare for 2.9.8.

2020-07-29 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 +++ configure.ac | 2 +- debian/changelog | 6 ++ 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index 887dda24b..dacd064cf 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v2.9.8 - xx xxx

[ovs-dev] [PATCH branch-2.9 0/2] Release patches for v2.9.7.

2020-07-29 Thread Ilya Maximets
Ilya Maximets (2): Set release date for 2.9.7. Prepare for 2.9.8. NEWS | 6 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 13 insertions(+), 3 deletions(-) -- 2.25.4 ___ dev mailing list

[ovs-dev] [PATCH branch-2.8 2/2] Prepare for 2.8.10.

2020-07-29 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 +++ configure.ac | 2 +- debian/changelog | 6 ++ 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index 10a15b561..8771a23e0 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v2.8.10 - xx xxx

[ovs-dev] [PATCH branch-2.6 2/2] Prepare for 2.6.9.

2020-07-29 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 +++ configure.ac | 2 +- debian/changelog | 6 ++ 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index 090ae04cd..e9b00b4bf 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v2.6.9 - xx xxx

[ovs-dev] [PATCH branch-2.8 1/2] Set release date for 2.8.9.

2020-07-29 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 ++- debian/changelog | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index 1844d0c22..10a15b561 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,6 @@ -v2.8.9 - xx xxx +v2.8.9 - 30 Jul 2020

[ovs-dev] [PATCH branch-2.7 2/2] Prepare for 2.7.12.

2020-07-29 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 +++ configure.ac | 2 +- debian/changelog | 6 ++ 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index dfee36de1..2bf2b03c6 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v2.7.12 - xx xxx

[ovs-dev] [PATCH branch-2.8 0/2] Release patches for v2.8.9.

2020-07-29 Thread Ilya Maximets
Ilya Maximets (2): Set release date for 2.8.9. Prepare for 2.8.10. NEWS | 6 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 13 insertions(+), 3 deletions(-) -- 2.25.4 ___ dev mailing list

[ovs-dev] [PATCH branch-2.7 1/2] Set release date for 2.7.11.

2020-07-29 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 ++- debian/changelog | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index 365513542..dfee36de1 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,6 @@ -v2.7.11 - xx xxx +v2.7.11 - 30 Jul 2020

[ovs-dev] [PATCH branch-2.7 0/2] Release patches for v2.7.11.

2020-07-29 Thread Ilya Maximets
Ilya Maximets (2): Set release date for 2.7.11. Prepare for 2.7.12. NEWS | 6 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 13 insertions(+), 3 deletions(-) -- 2.25.4 ___ dev mailing list

[ovs-dev] [PATCH branch-2.5 2/2] Prepare for 2.5.11.

2020-07-29 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 +++ configure.ac | 2 +- debian/changelog | 6 ++ 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index 9c1749fbb..828b5ef65 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v2.5.11 - xx xxx

[ovs-dev] [PATCH branch-2.6 1/2] Set release date for 2.6.8.

2020-07-29 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 ++- debian/changelog | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index a083ca3fd..090ae04cd 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,6 @@ -v2.6.8 - xx xxx +v2.6.8 - 30 Jul 2020

[ovs-dev] [PATCH branch-2.5 0/2] Release patches for v2.5.10.

2020-07-29 Thread Ilya Maximets
Ilya Maximets (2): Set release date for 2.5.10. Prepare for 2.5.11. NEWS | 6 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 13 insertions(+), 3 deletions(-) -- 2.25.4 ___ dev mailing list

[ovs-dev] [PATCH branch-2.5 1/2] Set release date for 2.5.10.

2020-07-29 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- NEWS | 3 ++- debian/changelog | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index 999c124e6..9c1749fbb 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,6 @@ -v2.5.10 - xx xxx +v2.5.10 - 30 Jul 2020

[ovs-dev] [PATCH branch-2.6 0/2] Release patches for v2.6.8.

2020-07-29 Thread Ilya Maximets
Ilya Maximets (2): Set release date for 2.6.8. Prepare for 2.6.9. NEWS | 6 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 13 insertions(+), 3 deletions(-) -- 2.25.4 ___ dev mailing list

Re: [ovs-dev] [PATCH v2] datapath-windows: Update flow key in SET action

2020-07-29 Thread Alin Serdean
Applied on master. Thank you for the fixes and testing! Alin. From: Jinjun Gao Sent: Wednesday, July 29, 2020 6:29 AM To: d...@openvswitch.org; Alin Serdean;

Re: [ovs-dev] [PATCH ovn v2] Avoid nb_cfg update notification flooding

2020-07-29 Thread Han Zhou
On Wed, Jul 29, 2020 at 2:46 AM Lucas Alvares Gomes wrote: > > Hi Han, > > > > > Just to follow up. I didn't find any new versions for this patch and it > > doesn't seem to be merged. Did you send any new versions or do you still > > plan to work on this? > > > > Thanks for this follow up. > > So

Re: [ovs-dev] [ovs-discuss] OVN: configuration in Neutron DB?

2020-07-29 Thread Tony Liu
Hi Lucas, Is it OK to discuss OpenStack integration here? Otherwise, please let me know which OpenStack email list we can use. I am running networking scaling test. The target is 4K isolated private networks with external/public access via logical routers. The test in my previous email is to add

Re: [ovs-dev] [PATCH ovn 7/7] ovn-northd.c: Support optionally disabling neighbor learning from ARP request/NS.

2020-07-29 Thread Numan Siddique
On Thu, Jul 23, 2020 at 10:57 AM Han Zhou wrote: > Support a new logical router option "always_learn_from_arp_request" that > controls > behavior when handling ARP requests or IPv4 ND-NS packets. > > "true" - Always learn the MAC/IP binding and add a new MAC_Binding entry > (default behavior) >

Re: [ovs-dev] [PATCH ovn 6/7] ovn-northd.c: Remove the use of the REGBIT_SKIP_LOOKUP_NEIGHBOR bit.

2020-07-29 Thread Numan Siddique
On Thu, Jul 23, 2020 at 10:57 AM Han Zhou wrote: > In LR ingress stage LOOKUP_NEIGHBOR and LEARN_NEIGHBOR, the flag > REGBIT_SKIP_LOOKUP_NEIGHBOR was used to indicate if mac-binding > lookup can be skipped. This patch avoid using the bit by combining > it with the REGBIT_LOOKUP_NEIGHBOR_RESULT

Re: [ovs-dev] [PATCH ovn 5/7] ovn-northd.8.xml: Fix reg9 bits documentation.

2020-07-29 Thread Numan Siddique
On Thu, Jul 23, 2020 at 10:57 AM Han Zhou wrote: > Update the reg9 bits according to current implementation. > > Fixes: 2dc7869436de ("ovn-northd: Address scale issues with DNAT flows.") > Signed-off-by: Han Zhou > Acked-by: Numan Siddique Thanks Numan > --- > northd/ovn-northd.8.xml | 16

Re: [ovs-dev] [PATCH ovn 3/7] actions: Rename xxx_lookup_mac to xxx_lookup_mac_bind.

2020-07-29 Thread Numan Siddique
On Thu, Jul 23, 2020 at 10:57 AM Han Zhou wrote: > For the functions related to lookup_arp/lookup_nd, renaming them to > avoid confusion, because those functions checks both mac and ip in > mac-bindings. This patch renames them so that a future patch can > add a function that only looks up by ip

Re: [ovs-dev] [PATCH ovn 2/7] tests: Fix get_arp/get_nd tests mac-binding table id.

2020-07-29 Thread Numan Siddique
On Thu, Jul 23, 2020 at 10:57 AM Han Zhou wrote: > The table id used in test is not the same as the one used in > real implementation. Although it doesn't affect correctness, it > may cause confusion when people are studying test cases. > > Signed-off-by: Han Zhou > Acked-by: Numan Siddique

Re: [ovs-dev] [PATCH ovn 1/7] ovn-northd: Support optionally avoid static neighbor flows in routers.

2020-07-29 Thread Numan Siddique
On Thu, Jul 23, 2020 at 10:57 AM Han Zhou wrote: > Support option:dynamic_neigh_routers for logical routers, so that in > particular use cases static neighbor flows are not prepopulated IP > addresses belonging to neighbor router ports, to avoid flow exploding > problem reported for

[ovs-dev] Your help

2020-07-29 Thread Mrs. Elizabeth Smith
May the peace of God be with you and your family.I know it will be a great surprise reading from me today but consider this a divine intervention as a pastor explained to my understanding. My name is Mrs. Elizabeth Smith .G, a widow from Oklahoma USA and am writing you from my sick bed because i

Re: [ovs-dev] [PATCH v3] odp-util: Fix clearing match mask if set action is partially unnecessary.

2020-07-29 Thread Adrian Moreno
Verified that v3 also solves the issue reported. Tested-by: Adrián Moreno On 7/29/20 12:37 PM, Eli Britstein wrote: > Thanks! > > Acked-by: Eli Britstein > > On 7/29/2020 12:13 PM, Ilya Maximets wrote: >> While committing set() actions, commit() could wildcard all the fields >> that are same

Re: [ovs-dev] [PATCH ovn v3 0/5] Add ECMP symmetric replies

2020-07-29 Thread Numan Siddique
On Wed, Jul 29, 2020 at 1:54 AM Mark Michelson wrote: > In some cases, it is important that traffic that arrived over an ECMP > route has its replies sent out over that same route. This series of > commits adds some prerequisite changes in order to facilitate using > conntrack labels to store

[ovs-dev] Altium - Proposal

2020-07-29 Thread Kelsey Cooper via dev
Hi,   Would you like to send in your Business Proposals/Newsletter to key decision Makers from companies currently using Altium Software?   Titles Like:  IT Decision Makers, C-level, Managers and other job titles as per your requirement.   Kindly let me know the Job Titles & Geography that

[ovs-dev] [PATCH net-next v3 2/2] net: openvswitch: make masks cache size configurable

2020-07-29 Thread Eelco Chaudron
This patch makes the masks cache size configurable, or with a size of 0, disable it. Reviewed-by: Paolo Abeni Signed-off-by: Eelco Chaudron --- Changes in v3: - Use is_power_of_2() function - Use array_size() function - Fix remaining sparse errors Changes in v2: - Fix sparse warnings -

[ovs-dev] [PATCH net-next v3 1/2] net: openvswitch: add masks cache hit counter

2020-07-29 Thread Eelco Chaudron
Add a counter that counts the number of masks cache hits, and export it through the megaflow netlink statistics. Reviewed-by: Paolo Abeni Reviewed-by: Tonghao Zhang Signed-off-by: Eelco Chaudron --- include/uapi/linux/openvswitch.h |2 +- net/openvswitch/datapath.c |5 -

[ovs-dev] [PATCH net-next v3 0/2] net: openvswitch: masks cache enhancements

2020-07-29 Thread Eelco Chaudron
This patchset adds two enhancements to the Open vSwitch masks cache. Signed-off-by: Eelco Chaudron Changes in v3 [patch 2/2 only]: - Use is_power_of_2() function - Use array_size() function - Fix remaining sparse errors Changes in v2 [patch 2/2 only]: - Fix sparse warnings - Fix netlink

[ovs-dev] [PATCH v2 2/3] dpif-netdev/avx512: add -fPIC flag to enable shared builds

2020-07-29 Thread Harry van Haaren
In certain scenarios with OVS built with --enable-shared and DPDK enabled as shared build too, Position Independant Code is required to link the avx512.a file into the relocatable .so that it must be linked into. Signed-off-by: Harry van Haaren --- v3: - Note this patch fixes a potential build

[ovs-dev] [PATCH v2 3/3] configure: explicitly disable avx512 if bintuils check fails

2020-07-29 Thread Harry van Haaren
This commit explicitly disables avx512f if the binutils assembler check fails to correctly assemble its input. Without this fix, there is a possibility that users can see undefined behaviour when compiling with -march=native on a CPU which supports avx512 and with a buggy binutils version (v2.30

[ovs-dev] [PATCH v2 1/3] dpif-netdev/avx512: avoid compiling avx512 code if binutils check fails

2020-07-29 Thread Harry van Haaren
This commit avoids compiling and linking of avx512 code into the vswitch_la library if the binutils check fails. This avoids compiling code into OVS that will not be executed due to binutils issue. Signed-off-by: Harry van Haaren --- v3: - Reword commit title & description --- lib/automake.mk

Re: [ovs-dev] [PATCH v3] odp-util: Fix clearing match mask if set action is partially unnecessary.

2020-07-29 Thread Eli Britstein
Thanks! Acked-by: Eli Britstein On 7/29/2020 12:13 PM, Ilya Maximets wrote: While committing set() actions, commit() could wildcard all the fields that are same in match key and in the set action. This leads to situation where mask after commit could actually contain less bits than it was

Re: [ovs-dev] [PATCH ovn v2] Avoid nb_cfg update notification flooding

2020-07-29 Thread Lucas Alvares Gomes
Hi Han, > > Just to follow up. I didn't find any new versions for this patch and it > doesn't seem to be merged. Did you send any new versions or do you still > plan to work on this? > Thanks for this follow up. So I am not currently working on it but the work is still in our backlog. Many of

Re: [ovs-dev] [PATCH v2] odp-util: Fix clearing match mask if set action is partially unnecessary.

2020-07-29 Thread Ilya Maximets
On 7/29/20 6:55 AM, Eli Britstein wrote: > > On 7/28/2020 7:32 PM, Ilya Maximets wrote: >> On 7/28/20 4:16 PM, Eli Britstein wrote: >>> it is not "wildcarded". "wildcarded" means it had a match and it was >>> removed. the case here it was only not "expanded". >>> It actually had a match on

Re: [ovs-dev] [ovs-discuss] OVN: configuration in Neutron DB?

2020-07-29 Thread Lucas Alvares Gomes
Hi, On Wed, Jul 29, 2020 at 7:42 AM Numan Siddique wrote: > > Adding Daniel and Lucas. Maybe you can also include opendev ML to get > appropriate responses from the OpenStack side. > > Please see below for few comments. > > > On Wed, Jul 29, 2020 at 12:02 PM Tony Liu wrote: > > > Quick update.

[ovs-dev] [PATCH v3] odp-util: Fix clearing match mask if set action is partially unnecessary.

2020-07-29 Thread Ilya Maximets
While committing set() actions, commit() could wildcard all the fields that are same in match key and in the set action. This leads to situation where mask after commit could actually contain less bits than it was before. And if set action was partially committed, all the fields that were the

Re: [ovs-dev] [ovs-discuss] OVN: configuration in Neutron DB?

2020-07-29 Thread Tony Liu
Thanks Numan! Tony From: Numan Siddique Sent: July 28, 2020 11:41 PM To: Tony Liu Cc: ovs-disc...@openvswitch.org ; ovs-dev@openvswitch.org ; Daniel Alvarez Sanchez ; Lucas Alvares Gomes Martins Subject: Re: [ovs-discuss] OVN: configuration in Neutron DB?

Re: [ovs-dev] [ovs-discuss] OVN: configuration in Neutron DB?

2020-07-29 Thread Numan Siddique
Adding Daniel and Lucas. Maybe you can also include opendev ML to get appropriate responses from the OpenStack side. Please see below for few comments. On Wed, Jul 29, 2020 at 12:02 PM Tony Liu wrote: > Quick update. I changed the script to create 256 routers first, then set > each of them as

Re: [ovs-dev] OVN: configuration in Neutron DB?

2020-07-29 Thread Tony Liu
Quick update. I changed the script to create 256 routers first, then set each of them as gateway. There is no create and set back to back. It seems working fine now. It would be good someone can clarify my questions. It seems that it's not guaranteed that the object is ready when client get OK

[ovs-dev] OVN: configuration in Neutron DB?

2020-07-29 Thread Tony Liu
Hi, In case of integration with OpenStack, for example, when a client requests to create a network, is this network configuration saved in both Neutron DB and OVN DB, or OVN DB only? Also, when a client gets a network from Neutron API, is the configuration read from Neutron DB or OVN DB?