[tor-commits] [torspec] 04/04: Merge branch 'tor-gitlab/mr/83'

[gitolite role]

This is an automated email from the git hooks/post-receive script.

dgoulet pushed a commit to branch main
in repository torspec.

commit 0bacc73d6baee7c2dfd9a7a864163e26f6c121ef
Merge: 773fc92 aab9efe
Author: David Goulet <dgoulet at torproject.org>
AuthorDate: Tue Jan 10 16:17:25 2023 -0500

    Merge branch 'tor-gitlab/mr/83'

 proposals/324-rtt-congestion-control.txt | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --cc proposals/324-rtt-congestion-control.txt
index 7701230,c617983..952d140
--- a/proposals/324-rtt-congestion-control.txt
+++ b/proposals/324-rtt-congestion-control.txt
@@@ -169,14 -167,16 +169,18 @@@ reduce the effects of packet jitter
  This smoothing is performed using N_EWMA[27], which is an Exponential
  Moving Average with alpha = 2/(N+1):
  
-   N_EWMA = BDP*2/(N+1) + N_EWMA_prev*(N-1)/(N+1).
+   N_EWMA = BDP*2/(N+1) + N_EWMA_prev*(N-1)/(N+1)
+          = (BDP*2 + N_EWMA_prev*(N-1))/(N+1).
+ 
+ Note that the second rearranged form MUST be used in order to ensure that
+ rounding errors are handled in the same manner as other implementations.
  
 -Flow control rate limiting uses this function
 +Flow control rate limiting uses this function.
 +
 +During Slow Start, N is set to `cc_ewma_ss`, for RTT estimation.
  
 -For both RTT and SENDME BDP estimation, N is the number of SENDME acks
 -between congestion window updates, divided by the value of consensus
 +After Slow Start, for both RTT and SENDME BDP estimation, N is the number
 +of SENDME acks between congestion window updates, divided by the value of consensus
  parameter 'cc_ewma_cwnd_pct', and then capped at a max of 'cc_ewma_max',
  but always at least 2:
  

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.