From: Ben Pfaff <blp@nicira.com>
Date: Mon, 23 Sep 2013 17:24:05 +0000 (-0700)
Subject: ofproto-dpif-upcall: Make miss handlers accumulate coverage counters.
X-Git-Tag: sliver-openvswitch-2.0.90-1~8^2~19
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=de80e4b65ba14c8d21b537470678fe0562ca79a8;p=sliver-openvswitch.git

ofproto-dpif-upcall: Make miss handlers accumulate coverage counters.

The miss handler threads do not call poll_block(), which calls
coverage_clear() indirectly.  This meant that coverage counters incremented
by miss handler threads never got integrated into the global coverage
counters and therefore did not show up in coverage logging or
"ovs-appctl coverage/show" output.  This commit fixes the problem.

Signed-off-by: Ben Pfaff <blp@nicira.com>
---

diff --git a/ofproto/ofproto-dpif-upcall.c b/ofproto/ofproto-dpif-upcall.c
index 74e8e9273..5222c11e1 100644
--- a/ofproto/ofproto-dpif-upcall.c
+++ b/ofproto/ofproto-dpif-upcall.c
@@ -420,6 +420,8 @@ udpif_upcall_handler(void *arg)
         ovs_mutex_unlock(&handler->mutex);
 
         handle_upcalls(handler->udpif, &misses);
+
+        coverage_clear();
     }
 }