[tor-commits] [tor/master] Add another 8387 diagnostic

nickm at torproject.org nickm at torproject.org
Wed Jul 16 09:10:23 UTC 2014


commit 32495ee3099c3fb512a986a8e5db32ebf153b439
Author: Nick Mathewson <nickm at torproject.org>
Date:   Wed Jul 16 10:05:00 2014 +0200

    Add another 8387 diagnostic
    
    When we run into bug 8387 (if we run into it again), report when we
    last called circuit_expire_old_circuits_clientside().  This will let
    us know -- if my fix for 8387 doesn't work -- whether my diagnosis
    was at least correct.
    
    Suggested by Andrea.
---
 src/or/circuituse.c |    9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/src/or/circuituse.c b/src/or/circuituse.c
index 600aede..714754a 100644
--- a/src/or/circuituse.c
+++ b/src/or/circuituse.c
@@ -783,6 +783,10 @@ circuit_expire_building(void)
   }
 }
 
+/** For debugging #8387: track when we last called
+ * circuit_expire_old_circuits_clientside. */
+static time_t last_expired_clientside_circuits = 0;
+
 /**
  * As a diagnostic for bug 8387, log information about how many one-hop
  * circuits we have around that have been there for at least <b>age</b>
@@ -894,6 +898,10 @@ circuit_log_ancient_one_hop_circuits(int age)
     }
   } SMARTLIST_FOREACH_END(ocirc);
 
+  log_notice(LD_HEARTBEAT, "It has been %ld seconds since I last called "
+             "circuit_expire_old_circuits_clientside().",
+             (long)(now - last_expired_clientside_circuits));
+
  done:
   smartlist_free(log_these);
 }
@@ -1220,6 +1228,7 @@ circuit_expire_old_circuits_clientside(void)
 
   tor_gettimeofday(&now);
   cutoff = now;
+  last_expired_clientside_circuits = now.tv_sec;
 
   if (! circuit_build_times_disabled() &&
       circuit_build_times_needs_circuits(get_circuit_build_times())) {





More information about the tor-commits mailing list