[ovs-dev] [PATCH] bridge: Omit alerts on the cfm_remote_opstate column.

Ethan Jackson ethan at nicira.com
Thu Sep 20 02:27:10 UTC 2012


This column should be write only, otherwise every call to update it
has to make a trip to the database.  Since this column is updated
every time through the run loop as part of refresh_instant_stats(),
this patch fixes a significant performance degradation.

Signed-off-by: Ethan Jackson <ethan at nicira.com>
---
 vswitchd/bridge.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c
index 53bb7b9..940e5e7 100644
--- a/vswitchd/bridge.c
+++ b/vswitchd/bridge.c
@@ -305,6 +305,7 @@ bridge_init(const char *remote)
     ovsdb_idl_omit_alert(idl, &ovsrec_interface_col_cfm_fault_status);
     ovsdb_idl_omit_alert(idl, &ovsrec_interface_col_cfm_remote_mpids);
     ovsdb_idl_omit_alert(idl, &ovsrec_interface_col_cfm_health);
+    ovsdb_idl_omit_alert(idl, &ovsrec_interface_col_cfm_remote_opstate);
     ovsdb_idl_omit_alert(idl, &ovsrec_interface_col_lacp_current);
     ovsdb_idl_omit(idl, &ovsrec_interface_col_external_ids);
 
-- 
1.7.12




More information about the dev mailing list