I have been looking at the FreeBSD congestion control code  and had some 
questions on the SACK enabled code flow .  
With SACK the congestion window is retained as 1 MSS when the Fast Retransmit 
is exited and Fast Recovery  is entered,so cwnd growth seems to be  much slower 
than slow start, as we we are re transmitting drops from the sack hole or new 
data.  But in non-SACK case the congestion window is restored  to cwnd/2 
+(dupacks-snd_limited)*MSS when Fast retransmit is exited  as per RFC3782.   
Could some one familiar with the code please throw some light on why we fall 
back to a cwnd of 1MSS as we enter fast recovery with SACK enabled.

                                        /*Congestion signal before ack. */
                                        cc_cong_signal(tp, th, CC_NDUPACK);
                                        cc_ack_received(tp, th, CC_DUPACK);
                                        tcp_timer_activate(tp, TT_REXMT, 0);
                                        tp->t_rtttime = 0;
                                        if (tp->t_flags & TF_SACK_PERMIT) {
                                                TCPSTAT_INC(
                                                    tcps_sack_recovery_episode);
                                                tp->sack_newdata = tp->snd_nxt;
                                                tp->snd_cwnd = tp->t_maxseg; 
<<<<<<<<<
                                                (void) tcp_output(tp);
                                                goto drop;                      
                <<<<<<<<<<
                                        }
                                        tp->snd_nxt = th->th_ack;
                                        tp->snd_cwnd = tp->t_maxseg;
                                        (void) tcp_output(tp);
                                        KASSERT(tp->snd_limited <= 2,
                                            ("%s: tp->snd_limited too big",
                                            __func__));
                                        tp->snd_cwnd = tp->snd_ssthresh +
                                             tp->t_maxseg *
                                             (tp->t_dupacks - tp->snd_limited); 
<<<<<<<<<

Bejoy Thomas
_______________________________________________
freebsd-net@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-net
To unsubscribe, send any mail to "freebsd-net-unsubscr...@freebsd.org"

Reply via email to