From 61a7b1e6fee6c81b0d8551898d2b108de2b42536 Mon Sep 17 00:00:00 2001 From: Ethan Jackson Date: Mon, 15 Oct 2012 17:05:35 -0700 Subject: [PATCH] timeval: Block SIGALRM when sleeping. Commit 00a16895 (timeval: Don't require signals for time_alarm().) Incorrectly disabled signals when when CACHE_TIME was disabled. In fact, the reverse was correct. As a result of this bug, OVS would wake once every 100ms unnecessarily. It shouldn't have affected correctness otherwise. Signed-off-by: Ethan Jackson --- lib/timeval.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/timeval.c b/lib/timeval.c index ea050179f..302224ac1 100644 --- a/lib/timeval.c +++ b/lib/timeval.c @@ -375,7 +375,7 @@ time_poll(struct pollfd *pollfds, int n_pollfds, long long int timeout_when, break; } - if (!blocked && !CACHE_TIME) { + if (!blocked && CACHE_TIME && !backtrace_conn) { block_sigalrm(&oldsigs); blocked = true; } -- 2.43.0