[ovs-dev] [PATCH 4/6] timeval: Wake up all threads when time is warped.
Alex Wang
alexw at nicira.com
Mon Oct 7 23:03:39 UTC 2013
This commit makes the main thread wake up all other threads when time is
warped.
Signed-off-by: Alex Wang <alexw at nicira.com>
---
lib/poll-loop.c | 5 +++++
lib/timeval.c | 28 ++++++++++++++++++++++++++++
lib/timeval.h | 2 ++
3 files changed, 35 insertions(+)
diff --git a/lib/poll-loop.c b/lib/poll-loop.c
index 4eb1187..97e4a2a 100644
--- a/lib/poll-loop.c
+++ b/lib/poll-loop.c
@@ -226,6 +226,11 @@ poll_block(void)
COVERAGE_INC(poll_zero_timeout);
}
+ /* For threads other than main thread, wait for timewarp. */
+ if (ovsthread_id_self()) {
+ timewarp_wait();
+ }
+
retval = time_poll(loop->pollfds, loop->n_waiters,
loop->timeout_when, &elapsed);
if (retval < 0) {
diff --git a/lib/timeval.c b/lib/timeval.c
index 64ae845..cb4ce7c 100644
--- a/lib/timeval.c
+++ b/lib/timeval.c
@@ -33,6 +33,7 @@
#include "hmap.h"
#include "ovs-thread.h"
#include "signals.h"
+#include "seq.h"
#include "unixctl.h"
#include "util.h"
#include "vlog.h"
@@ -57,6 +58,14 @@ static struct clock wall_clock; /* CLOCK_REALTIME. */
/* The monotonic time at which the time module was initialized. */
static long long int boot_time;
+/* True only when timeval_dummy_register() is called. */
+static bool timewarp_enabled;
+/* Reference to the seq struct. Threads other than main thread can
+ * wait on timewarp_seq and be waken up when time is warped. */
+static struct seq *timewarp_seq;
+/* Last value of 'timewarp_seq'. */
+DEFINE_STATIC_PER_THREAD_DATA(uint64_t, last_seq, 0);
+
/* Monotonic time in milliseconds at which to die with SIGALRM (if not
* LLONG_MAX). */
static long long int deadline = LLONG_MAX;
@@ -79,6 +88,7 @@ init_clock(struct clock *c, clockid_t id)
ovs_mutex_init(&c->mutex);
atomic_init(&c->slow_path, false);
xclock_gettime(c->id, &c->cache);
+ timewarp_seq = seq_create();
}
static void
@@ -313,6 +323,19 @@ xclock_gettime(clock_t id, struct timespec *ts)
}
}
+/* Makes threads wait on timewarp_seq and be waken up when time is warped.
+ * This function will be no-op unless timeval_dummy_register() is called. */
+void
+timewarp_wait(void)
+{
+ if (timewarp_enabled) {
+ uint64_t *last_seq = last_seq_get();
+
+ *last_seq = seq_read(timewarp_seq);
+ seq_wait(timewarp_seq, *last_seq);
+ }
+}
+
static long long int
timeval_diff_msec(const struct timeval *a, const struct timeval *b)
{
@@ -512,12 +535,17 @@ timeval_warp_cb(struct unixctl_conn *conn,
timespec_add(&monotonic_clock.warp, &monotonic_clock.warp, &ts);
ovs_mutex_unlock(&monotonic_clock.mutex);
+ if (timewarp_enabled) {
+ /* Changes 'timewarp_seq' to wake up all waiting thread. */
+ seq_change(timewarp_seq);
+ }
unixctl_command_reply(conn, "warped");
}
void
timeval_dummy_register(void)
{
+ timewarp_enabled = true;
unixctl_command_register("time/stop", "", 0, 0, timeval_stop_cb, NULL);
unixctl_command_register("time/warp", "MSECS", 1, 1,
timeval_warp_cb, NULL);
diff --git a/lib/timeval.h b/lib/timeval.h
index 99b3af0..1bbfd5c 100644
--- a/lib/timeval.h
+++ b/lib/timeval.h
@@ -69,6 +69,8 @@ int get_cpu_usage(void);
long long int time_boot_msec(void);
+void timewarp_wait(void);
+
#ifdef __cplusplus
}
#endif
--
1.7.9.5
More information about the dev
mailing list