bridge: Omit alerts on the cfm_remote_opstate column.
authorEthan Jackson <ethan@nicira.com>
Thu, 20 Sep 2012 02:21:06 +0000 (19:21 -0700)
committerEthan Jackson <ethan@nicira.com>
Thu, 20 Sep 2012 16:17:40 +0000 (09:17 -0700)
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@nicira.com>
vswitchd/bridge.c

index 53bb7b90517a0d4fbee3b235fdc3a46ef7e3870e..940e5e7f5895ec8b49a9a6782b2ae3c9389a7419 100644 (file)
@@ -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);