From d0fbbb909ebbf80ba60e61444377a0761a3edc13 Mon Sep 17 00:00:00 2001 From: Willy Tarreau Date: Tue, 23 Mar 2021 08:45:42 +0100 Subject: [PATCH] MINOR: time: also provide a global, monotonic global_now_ms timer The period-based freq counters need the global date in milliseconds, so better calculate it and expose it rather than letting all call places incorrectly retrieve it. Here what we do is that we maintain a new globally monotonic timer, global_now_ms, which ought to be very close to the global_now one, but maintains the monotonic approach of now_ms between all threads in that global_now_ms is always ahead of any now_ms. This patch is made simple to ease backporting (it will be needed for a subsequent fix), but it also opens the way to some simplifications on the time handling: instead of computing the local time and trying to force it to the global one, we should soon be able to proceed in the opposite way, that is computing the new global time an making the local one just the latest snapshot of it. This will bring the benefit of making sure that the global time is always ahead of the local one. (cherry picked from commit 6064b34be0e761de881a5bfca287d01f69122602) Signed-off-by: Willy Tarreau (cherry picked from commit 3a9bc55350e18a92538ffa7295a204420fdba683) Signed-off-by: Willy Tarreau (cherry picked from commit 29d0329b8af90af42d37007c8c89872203fba1a3) [wt: minor adjustments to includes (common/ instead of haproxy/)] Signed-off-by: Willy Tarreau --- include/common/time.h | 1 + src/time.c | 12 ++++++++++++ 2 files changed, 13 insertions(+) diff --git a/include/common/time.h b/include/common/time.h index 9f04815..7d5b09b 100644 --- a/include/common/time.h +++ b/include/common/time.h @@ -64,6 +64,7 @@ extern struct timeval start_date; /* the process's start date */ extern THREAD_LOCAL struct timeval before_poll; /* system date before calling poll() */ extern THREAD_LOCAL struct timeval after_poll; /* system date after leaving poll() */ extern volatile unsigned long long global_now; +extern volatile unsigned int global_now_ms; /**** exported functions *************************************************/ diff --git a/src/time.c b/src/time.c index 719d124..1d74ed3 100644 --- a/src/time.c +++ b/src/time.c @@ -16,6 +16,7 @@ #include #include +#include #include #include @@ -31,6 +32,7 @@ THREAD_LOCAL struct timeval after_poll; /* system date after leaving poll() static THREAD_LOCAL struct timeval tv_offset; /* per-thread time ofsset relative to global time */ volatile unsigned long long global_now; /* common date between all threads (32:32) */ +volatile unsigned int global_now_ms; /* common date in milliseconds (may wrap) */ static THREAD_LOCAL unsigned int iso_time_sec; /* last iso time value for this thread */ static THREAD_LOCAL char iso_time_str[28]; /* ISO time representation of gettimeofday() */ @@ -179,6 +181,7 @@ REGPRM2 void tv_update_date(int max_wait, int interrupted) { struct timeval adjusted, deadline, tmp_now, tmp_adj; unsigned int curr_sec_ms; /* millisecond of current second (0..999) */ + unsigned int old_now_ms, new_now_ms; unsigned long long old_now; unsigned long long new_now; @@ -262,6 +265,15 @@ REGPRM2 void tv_update_date(int max_wait, int interrupted) */ ms_left_scaled = (999U - curr_sec_ms) * 4294967U; now_ms = now.tv_sec * 1000 + curr_sec_ms; + + /* update the global current millisecond */ + old_now_ms = global_now_ms; + do { + new_now_ms = old_now_ms; + if (tick_is_lt(new_now_ms, now_ms)) + new_now_ms = now_ms; + } while (!_HA_ATOMIC_CAS(&global_now_ms, &old_now_ms, new_now_ms)); + return; } -- 1.7.10.4