Skip to content

Commit

Permalink
Limit delayed ack timer to 1 RTT
Browse files Browse the repository at this point in the history
Fixes #1662
  • Loading branch information
ianswett committed Aug 15, 2018
1 parent 3bd3efa commit 5521fab
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions draft-ietf-quic-transport.md
Expand Up @@ -3407,9 +3407,11 @@ Implementations MUST NOT generate packets that only contain ACK frames in
response to packets which only contain ACK frames. However, they MUST
acknowledge packets containing only ACK frames when sending ACK frames in
response to other packets. Implementations MUST NOT send more than one packet
containing only ACK frames per received packet that contains frames other than
ACK frames. Packets containing non-ACK frames MUST be acknowledged immediately
or when a delayed ack timer expires.
containing only an ACK frame per received packet that contains frames other than
an ACK frame. Packets containing non-ACK frames MUST be acknowledged
immediately or when a delayed ack timer expires. The delayed ack timer should
never delay an ACK for longer than an RTT, which ensures an ACK frame is sent
at least once per RTT.

This comment has been minimized.

Copy link
@mirjak

mirjak Aug 15, 2018

Contributor

Should probably say
"... which ensures an ACK frame is sent at least once per RTT if new data was received in the last RTT."


To limit ACK blocks to those that have not yet been received by the sender, the
receiver SHOULD track which ACK frames have been acknowledged by its peer. Once
Expand Down

0 comments on commit 5521fab

Please sign in to comment.