[or-cvs] [tor/master] Fix various comment typos in ewma patch; found by arma.

Nick Mathewson nickm at seul.org
Sat Dec 19 03:34:46 UTC 2009


Author: Nick Mathewson <nickm at torproject.org>
Date: Tue, 15 Dec 2009 13:20:22 -0500
Subject: Fix various comment typos in ewma patch; found by arma.
Commit: 1292a9ddfeef5d9560f4493d6767a10b181f031e

---
 src/or/or.h    |    4 ++--
 src/or/relay.c |   15 +++++++++------
 2 files changed, 11 insertions(+), 8 deletions(-)

diff --git a/src/or/or.h b/src/or/or.h
index 8be8491..fd913ce 100644
--- a/src/or/or.h
+++ b/src/or/or.h
@@ -1083,7 +1083,7 @@ typedef struct or_connection_t {
    * cell_ewma algorithm for choosing circuits, we can remove active_circuits.
    */
   smartlist_t *active_circuit_pqueue;
-  /** The tick on which the ciell_ewma_t's in active_circuit_pqueue last had
+  /** The tick on which the cell_ewma_ts in active_circuit_pqueue last had
    * their ewma values rescaled. */
   unsigned active_circuit_pqueue_last_recalibrated;
   struct or_connection_t *next_with_same_id; /**< Next connection with same
@@ -2018,7 +2018,7 @@ typedef struct {
   unsigned last_adjusted_tick;
   /** The EWMA of the cell count. */
   double cell_count;
-  /** True iff this is a the cell count for a circuit's previous
+  /** True iff this is the cell count for a circuit's previous
    * connection. */
   unsigned int is_for_p_conn : 1;
   /** The position of the circuit within the or connection's priority
diff --git a/src/or/relay.c b/src/or/relay.c
index 19bb687..4a27df6 100644
--- a/src/or/relay.c
+++ b/src/or/relay.c
@@ -1787,9 +1787,10 @@ cell_ewma_to_circuit(cell_ewma_t *ewma)
 
    Conceptually, we take an exponentially weighted mean average of the number
    of cells a circuit has sent, and allow active circuits (those with cells to
-   relay) to send cells in order of their exponentially-weighted mean average
-   (EWMA) cell count.  [That is, a cell sent N seconds ago 'counts' F^N times
-   as much as a cell sent now, for 0<F<1.0.]
+   relay) to send cells in reverse order of their exponentially-weighted mean
+   average (EWMA) cell count.  [That is, a cell sent N seconds ago 'counts'
+   F^N times as much as a cell sent now, for 0<F<1.0, and we favor the
+   circuit that has sent the fewest cells]
 
    If 'double' had infinite precision, we could do this simply by counting a
    cell sent at startup as having weight 1.0, and a cell sent N seconds later
@@ -1816,8 +1817,10 @@ cell_ewma_to_circuit(cell_ewma_t *ewma)
  * consensus or a configuration setting. */
 #define EWMA_DEFAULT_SCALE_FACTOR 0.9
 
-/** Given a timeval 'now', compute the cell_ewma tick in which it occurs
- * and the fraction of the tick that has elapsed before 
+/** Given a timeval <b>now</b>, compute the cell_ewma tick in which it occurs
+ * and the fraction of the tick that has elapsed between the start of the tick
+ * and <b>now</b>.  Return the former and store the latter in
+ * *<b>remainder_out</b>.
  *
  * These tick values are not meant to be shared between Tor instances, or used
  * for other purposes. */
@@ -2205,7 +2208,7 @@ connection_or_flush_from_first_active_circuit(or_connection_t *conn, int max,
   if (streams_blocked && queue->n <= CELL_QUEUE_LOWWATER_SIZE)
     set_streams_blocked_on_circ(circ, conn, 0); /* unblock streams */
 
-  /* Did we just ran out of cells on this circuit's queue? */
+  /* Did we just run out of cells on this circuit's queue? */
   if (queue->n == 0) {
     log_debug(LD_GENERAL, "Made a circuit inactive.");
     make_circuit_inactive_on_conn(circ, conn);
-- 
1.5.6.5




More information about the tor-commits mailing list