tor

commit c9f07f36bf986f274ac9fc56b5c61bbcc669b22e
parent 285e7c98fdca8eda97683c0b96f86645ee3f0546
Author: Nick Mathewson <nickm@torproject.org>
Date:   Thu,  3 May 2018 12:23:55 -0400

Mark the 1-per-sec update_current_time() call as redundant.

We still do this time update here, since we do it from all
callbacks, but it is no longer a reason to keep the once-per-second
callback enabled.

Closes ticket 26009.

Diffstat:
A
changes/ticket26009
 | 
9
+++++++++
M
src/or/main.c
 | 
6
+++++-

2 files changed, 14 insertions(+), 1 deletion(-)
diff --git a/changes/ticket26009 b/changes/ticket26009
@@ -0,0 +1,9 @@
+  o Minor features (mainloop):
+   - Move responsibility for
+     keeping track of Tor's uptime
+      from a nce-per-second callback to a callback that is only scheduled as
+      needed.  Once enough items are removed from our once-per-second
+      callback, we can eliminate it entirely to conserve CPU when idle.
+      Closes ticket
+      26009.
+
diff --git a/src/or/main.c b/src/or/main.c
@@ -2576,8 +2576,12 @@ second_elapsed_callback(periodic_timer_t *timer, void *arg)
 
   n_libevent_errors = 0;
 
-  /* log_notice(LD_GENERAL, "Tick."); */
   now = time(NULL);
+
+  /* We don't need to do this once-per-second any more: time-updating is
+   * only in this callback _because it is a callback_. It should be fine
+   * to disable this callback, and the time will still get updated.
+   */
   update_current_time(now);
 
   /* the second has rolled over. check more stuff. */