cfm: Reduce "long delay" message from WARN to INFO, to match BFD behavior.
authorBen Pfaff <blp@nicira.com>
Fri, 25 Jul 2014 16:19:17 +0000 (09:19 -0700)
committerBen Pfaff <blp@nicira.com>
Fri, 25 Jul 2014 16:19:25 +0000 (09:19 -0700)
These messages can cause the testsuite to fail on a busy build machine
since the testsuite treats WARN or ERR log messages as failures.  BFD
uses an INFO message instead of WARN, so this just changes CFM to match.

Alternatively, the testsuite could ignore "long delay" messages (it ignores
some other categories of messages).  In that case I'd expect that we'd
want to change BFD to match CFM since I don't know of a reason why they
should log differently.

Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Justin Pettit <jpettit@nicira.com>
lib/cfm.c

index 7784369..d83a68c 100644 (file)
--- a/lib/cfm.c
+++ b/lib/cfm.c
@@ -613,7 +613,7 @@ cfm_compose_ccm(struct cfm *cfm, struct ofpbuf *packet,
     if (cfm->last_tx) {
         long long int delay = time_msec() - cfm->last_tx;
         if (delay > (cfm->ccm_interval_ms * 3 / 2)) {
-            VLOG_WARN("%s: long delay of %lldms (expected %dms) sending CCM"
+            VLOG_INFO("%s: long delay of %lldms (expected %dms) sending CCM"
                       " seq %"PRIu32, cfm->name, delay, cfm->ccm_interval_ms,
                       cfm->seq);
         }