Re: [quicwg/base-drafts] congestion window increase on every ACKed packet could result in bursty sends (#3094)

[Vidhi Goel <[e-mail]>]

@junhochoi, I think we need to do 2MSS clamping per ACK received and not per packet acked.

I am thinking that we can remove OnPacketAckedCC and instead do OnAckReceivedCC. Pseudocode change something like this:
```
    OnAckReceived(ack, pn_space):

    Set total_bytes_acked to 0
    for acked_packet in newly_acked_packets:
        total_bytes_acked += acked_packet.size
        OnPacketAcked(acked_packet.packet_number, pn_space)

    OnAckReceivedCC(total_bytes_acked)

OnAckReceivedCC(total_bytes_acked):
    bytes_in_flight -= total_bytes_acked
    if (congestion_window < ssthresh):
         // Slow start.
        congestion_window += min(total_bytes_acked, 2* kMaxDatagramSize)
   else
       congestion_window += kMaxDatagramSize * total_bytes_acked
           / congestion_window
    
```



-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/quicwg/base-drafts/issues/3094#issuecomment-542965003