[ovs-dev] [PATCH 1/4] logging: Lower loggging level for coverage messages due to slow poll loops

Justin Pettit jpettit at nicira.com
Fri Sep 11 16:34:47 UTC 2009

By default, many OVS processes keep track of their time through a poll
loop.  If it takes an unusually long time (measured as some distance
from the mean), the processes will log stats its been keeping about
coverage.  It was doing this at level WARN.

On Xen systems, syslog messages written at level INFO and higher are
written to /var/log/messages synchronously.  This would mean that there
would be dire messages that it took a few dozen milliseconds to go
through the loop, meanwhile, it would take up to 6(!) seconds writing
those.  Meanwhile, the process would do no other processing, which could
be quite serious in the case of a process such as ovs-vswitchd.
This problem was somewhat masked because the time used by this logging
was not used in the calculations for determining how long it was taking
to get through the loop.

This commit lowers the default log level for those coverage messages to
INFO.  On Xen systems, it raises the default level at which messages are
written to syslog to WARN.

Diagnosed and fixed with the help of Ian Campbell.
 lib/timeval.c                |    2 +-
 xenserver/etc_init.d_vswitch |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/timeval.c b/lib/timeval.c
index 4c34a17..2dfde6b 100644
--- a/lib/timeval.c
+++ b/lib/timeval.c
@@ -294,7 +294,7 @@ log_poll_interval(long long int last_wakeup, const struct rusage *last_rusage)
                       rusage.ru_nvcsw - last_rusage->ru_nvcsw,
                       rusage.ru_nivcsw - last_rusage->ru_nivcsw);
-        coverage_log(VLL_WARN, true);
+        coverage_log(VLL_INFO, true);
     /* Update exponentially weighted moving average.  With these parameters, a
diff --git a/xenserver/etc_init.d_vswitch b/xenserver/etc_init.d_vswitch
index 92be421..74d0775 100755
--- a/xenserver/etc_init.d_vswitch
+++ b/xenserver/etc_init.d_vswitch
@@ -49,7 +49,7 @@ BRCOMPATD_RUN_DIR="${BRCOMPATD_RUN_DIR:-/var/xen/vswitch}"

More information about the dev mailing list