Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

In-flight not outstanding #2120

Merged
merged 8 commits into from Dec 13, 2018
Merged

In-flight not outstanding #2120

Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
6763689
In-flight not outstanding
ianswett Dec 12, 2018
d4c3b58
Update draft-ietf-quic-recovery.md
janaiyengar Dec 13, 2018
11ff347
Update draft-ietf-quic-recovery.md
janaiyengar Dec 13, 2018
dfdf208
Update draft-ietf-quic-recovery.md
janaiyengar Dec 13, 2018
023db98
Update draft-ietf-quic-recovery.md
janaiyengar Dec 13, 2018
b82ab25
Update draft-ietf-quic-recovery.md
janaiyengar Dec 13, 2018
2e3fea6
Update draft-ietf-quic-recovery.md
janaiyengar Dec 13, 2018
de32595
Update draft-ietf-quic-recovery.md
janaiyengar Dec 13, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
14 changes: 7 additions & 7 deletions draft-ietf-quic-recovery.md
View file
Open in desktop
Expand Up @@ -400,7 +400,7 @@ On each consecutive expiration of the crypto timer without receiving an
acknowledgement for a new packet, the sender SHOULD double the crypto
retransmission timeout and set a timer for this period.

When crypto packets are outstanding, the TLP and RTO timers are not active.
When crypto packets are in flight, the TLP and RTO timers are not active.

#### Retry and Version Negotiation

Expand Down Expand Up @@ -436,9 +436,9 @@ conditions:
PTO SHOULD be scheduled for min(RTO, PTO).

QUIC includes MaxAckDelay in all probe timeouts, because it assumes the ack
delay may come into play, regardless of the number of packets outstanding.
TCP's TLP assumes if at least 2 packets are outstanding, acks will not be
delayed.
delay may come into play, regardless of the number of ack-eliciting
packets in flight. TCP's TLP assumes if at least 2 ack-eliciting packets are
in flight, acks will not be delayed.

A PTO value of at least 1.5*SRTT ensures that the ACK is overdue. The 1.5 is
based on {{?TLP}}, but implementations MAY experiment with other constants.
Expand Down Expand Up @@ -796,7 +796,7 @@ Pseudocode for SetLossDetectionTimer follows:
loss_detection_timer.cancel()
return

if (crypto packets are outstanding):
if (crypto packets are in flight):
// Crypto retransmission timer.
if (smoothed_rtt == 0):
timeout = 2 * kInitialRtt
Expand Down Expand Up @@ -837,7 +837,7 @@ Pseudocode for OnLossDetectionTimeout follows:

~~~
OnLossDetectionTimeout():
if (crypto packets are outstanding):
if (crypto packets are in flight):
// Crypto retransmission timeout.
RetransmitUnackedCryptoData()
crypto_count++
Expand Down Expand Up @@ -1043,7 +1043,7 @@ kMaxDatagramSize:
windows. The RECOMMENDED value is 1200 bytes.

kInitialWindow:
: Default limit on the initial amount of outstanding data in bytes. Taken from
: Default limit on the initial amount of data in flight, in bytes. Taken from
{{?RFC6928}}. The RECOMMENDED value is the minimum of 10 * kMaxDatagramSize
and max(2* kMaxDatagramSize, 14600)).

Expand Down