Having a coverage counter tracking the value of the internal seq_next
should help in debugging.
Suggested-by: Justin Pettit <jpet...@ovn.org>
Signed-off-by: Jarno Rajahalme <ja...@ovn.org>
---
lib/seq.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/lib/seq.c b/lib/seq.c
index 271e617..9c3257c 100644
--- a/lib/seq.c
+++ b/lib/seq.c
@@ -20,6 +20,7 @@
#include <stdbool.h>
+#include "coverage.h"
#include "hash.h"
#include "hmap.h"
#include "latch.h"
@@ -27,6 +28,8 @@
#include "ovs-thread.h"
#include "poll-loop.h"
+COVERAGE_DEFINE(seq_change);
+
/* A sequence number object. */
struct seq {
uint64_t value OVS_GUARDED;
@@ -74,6 +77,9 @@ seq_create(void)
seq_init();
seq = xmalloc(sizeof *seq);
+
+ COVERAGE_INC(seq_change);
+
ovs_mutex_lock(&seq_mutex);
seq->value = seq_next++;
hmap_init(&seq->waiters);
@@ -100,6 +106,8 @@ void
seq_change(struct seq *seq)
OVS_EXCLUDED(seq_mutex)
{
+ COVERAGE_INC(seq_change);
+
ovs_mutex_lock(&seq_mutex);
seq->value = seq_next++;
seq_wake_waiters(seq);
--
2.1.4
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
