timeval: Call coverage_clear() before coverage_log() in time_poll().
authorBen Pfaff <blp@nicira.com>
Mon, 23 Sep 2013 17:13:54 +0000 (10:13 -0700)
committerBen Pfaff <blp@nicira.com>
Thu, 10 Oct 2013 00:02:57 +0000 (17:02 -0700)
commite2afa7cd6820821e4c652818722d6120d7912399
tree8147459f1ea5df92464b79ead5c110e358cee9e6
parent10c44245ad673e03c50c03b711f6c4dea1d7f2e2
timeval: Call coverage_clear() before coverage_log() in time_poll().

time_poll() calls log_poll_interval(), which in some circumstances calls
coverage_log().  Before this commit, time_poll() also called
coverage_clear() after log_poll_interval().  This made sense before commit
857165b5fd26 (coverage: Make thread-safe.), because coverage_log() would
log the most recent main loop's coverage counters separately and calling
coverage_clear() beforehand would zero out those counters.  However, it
doesn't make sense any longer because the most recent loop's counters are
no longer separately logged and in fact this practice now means that the
most recent loop's counters are omitted from the logged counters.

Therefore, this commit moves the call to coverage_clear() earlier, so that
the most recent loop's counters are included.

Signed-off-by: Ben Pfaff <blp@nicira.com>
lib/timeval.c