From 8cbfed717229511b8a50f7c046d39ddd6a93617b Mon Sep 17 00:00:00 2001 From: ianswett Date: Fri, 10 May 2019 06:13:26 -0400 Subject: [PATCH 1/5] New paths may seed initial_rtt Fixes #2644 --- draft-ietf-quic-recovery.md | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/draft-ietf-quic-recovery.md b/draft-ietf-quic-recovery.md index 4490ef975e..84e045f5a7 100644 --- a/draft-ietf-quic-recovery.md +++ b/draft-ietf-quic-recovery.md @@ -521,6 +521,10 @@ is available, or if the network changes, the initial RTT SHOULD be set to 500ms, resulting in a 1 second initial handshake timeout as recommended in {{?RFC6298}}. +A new path on an existing connection MAY use the delay between sending a +PATH_CHALLENGE and receiving a PATH_RESPONSE to seed initial_rtt for a new path, +but MUST not be considered an RTT sample. + When a crypto packet is sent, the sender MUST set a timer for twice the smoothed RTT. This timer MUST be updated when a new crypto packet is sent and when an acknowledgement is received which computes a new RTT sample. Upon timeout, From 5eb311bba9a4ccaf61c167eabb619238ce6353c9 Mon Sep 17 00:00:00 2001 From: ianswett Date: Sat, 11 May 2019 11:26:54 -0400 Subject: [PATCH 2/5] Update draft-ietf-quic-recovery.md --- draft-ietf-quic-recovery.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/draft-ietf-quic-recovery.md b/draft-ietf-quic-recovery.md index 84e045f5a7..fd7be7016e 100644 --- a/draft-ietf-quic-recovery.md +++ b/draft-ietf-quic-recovery.md @@ -521,9 +521,9 @@ is available, or if the network changes, the initial RTT SHOULD be set to 500ms, resulting in a 1 second initial handshake timeout as recommended in {{?RFC6298}}. -A new path on an existing connection MAY use the delay between sending a -PATH_CHALLENGE and receiving a PATH_RESPONSE to seed initial_rtt for a new path, -but MUST not be considered an RTT sample. +A connection MAY use the delay between sending a PATH_CHALLENGE and receiving +a PATH_RESPONSE to seed initial_rtt for a new path, but MUST not be considered +an RTT sample. When a crypto packet is sent, the sender MUST set a timer for twice the smoothed RTT. This timer MUST be updated when a new crypto packet is sent and when From c35d5f1357c04739a44b84dbdb5a45aacbb9f0f8 Mon Sep 17 00:00:00 2001 From: ianswett Date: Mon, 13 May 2019 10:13:20 -0400 Subject: [PATCH 3/5] Update draft-ietf-quic-recovery.md --- draft-ietf-quic-recovery.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/draft-ietf-quic-recovery.md b/draft-ietf-quic-recovery.md index fd7be7016e..fd9ade874d 100644 --- a/draft-ietf-quic-recovery.md +++ b/draft-ietf-quic-recovery.md @@ -522,8 +522,8 @@ resulting in a 1 second initial handshake timeout as recommended in {{?RFC6298}}. A connection MAY use the delay between sending a PATH_CHALLENGE and receiving -a PATH_RESPONSE to seed initial_rtt for a new path, but MUST not be considered -an RTT sample. +a PATH_RESPONSE to seed initial_rtt for a new path, but the delay MUST not be +considered an RTT sample. When a crypto packet is sent, the sender MUST set a timer for twice the smoothed RTT. This timer MUST be updated when a new crypto packet is sent and when From 7beb5ef2b03ed4281e1d12cad18531d7bbb47269 Mon Sep 17 00:00:00 2001 From: ianswett Date: Tue, 28 May 2019 14:57:02 -0400 Subject: [PATCH 4/5] Update draft-ietf-quic-recovery.md Co-Authored-By: Jana Iyengar --- draft-ietf-quic-recovery.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/draft-ietf-quic-recovery.md b/draft-ietf-quic-recovery.md index fd9ade874d..d7701a0fe6 100644 --- a/draft-ietf-quic-recovery.md +++ b/draft-ietf-quic-recovery.md @@ -522,7 +522,7 @@ resulting in a 1 second initial handshake timeout as recommended in {{?RFC6298}}. A connection MAY use the delay between sending a PATH_CHALLENGE and receiving -a PATH_RESPONSE to seed initial_rtt for a new path, but the delay MUST not be +a PATH_RESPONSE to seed initial_rtt for a new path, but the delay MUST NOT be considered an RTT sample. When a crypto packet is sent, the sender MUST set a timer for twice the smoothed From 5a6461ef8016071d5474563e93453287c842be33 Mon Sep 17 00:00:00 2001 From: ianswett Date: Tue, 28 May 2019 15:03:39 -0400 Subject: [PATCH 5/5] MUST NOT to SHOULD NOT --- draft-ietf-quic-recovery.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/draft-ietf-quic-recovery.md b/draft-ietf-quic-recovery.md index d7701a0fe6..3c32bbb260 100644 --- a/draft-ietf-quic-recovery.md +++ b/draft-ietf-quic-recovery.md @@ -522,8 +522,8 @@ resulting in a 1 second initial handshake timeout as recommended in {{?RFC6298}}. A connection MAY use the delay between sending a PATH_CHALLENGE and receiving -a PATH_RESPONSE to seed initial_rtt for a new path, but the delay MUST NOT be -considered an RTT sample. +a PATH_RESPONSE to seed initial_rtt for a new path, but the delay SHOULD NOT +be considered an RTT sample. When a crypto packet is sent, the sender MUST set a timer for twice the smoothed RTT. This timer MUST be updated when a new crypto packet is sent and when