[ovs-dev] [PATCH 1/3] ovn: Fix check on existing encap row.

Russell Bryant rbryant at redhat.com
Tue Oct 27 09:05:33 UTC 2015


This code does some checking to validate the existing encaps for a
chassis to see if they need to be updated.  This typo resulted in
ovn-controller re-creating its encap(s) every time this code ran, making
ovn-controller and ovsdb-server eat up a CPU in my testing.

Signed-off-by: Russell Bryant <rbryant at redhat.com>
---
 ovn/controller/chassis.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ovn/controller/chassis.c b/ovn/controller/chassis.c
index 6617bf3..f0dfaa5 100644
--- a/ovn/controller/chassis.c
+++ b/ovn/controller/chassis.c
@@ -95,7 +95,7 @@ chassis_run(struct controller_ctx *ctx, const char *chassis_id)
         bool same = true;
         for (int i = 0; i < chassis_rec->n_encaps; i++) {
             cur_tunnels |= get_tunnel_type(chassis_rec->encaps[i]->type);
-            same = same && strcmp(chassis_rec->encaps[i]->ip, encap_ip);
+            same = same && !strcmp(chassis_rec->encaps[i]->ip, encap_ip);
         }
         same = same && req_tunnels == cur_tunnels;
 
-- 
2.4.3




More information about the dev mailing list