Return-Path: <noreply@github.com>
X-Original-To: quic-issues@ietfa.amsl.com
Delivered-To: quic-issues@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1])
 by ietfa.amsl.com (Postfix) with ESMTP id 2829C130E4A
 for <quic-issues@ietfa.amsl.com>; Thu, 20 Dec 2018 05:54:07 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8.064
X-Spam-Level: 
X-Spam-Status: No, score=-8.064 tagged_above=-999 required=5
 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.065, DKIM_SIGNED=0.1,
 DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_IMAGE_ONLY_32=0.001,
 HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_HI=-5,
 SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key)
 header.d=github.com
Received: from mail.ietf.org ([4.31.198.44])
 by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024)
 with ESMTP id w86LdgghaejG for <quic-issues@ietfa.amsl.com>;
 Thu, 20 Dec 2018 05:54:05 -0800 (PST)
Received: from out-2.smtp.github.com (out-2.smtp.github.com [192.30.252.193])
 (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))
 (No client certificate requested)
 by ietfa.amsl.com (Postfix) with ESMTPS id 38980130DFC
 for <quic-issues@ietf.org>; Thu, 20 Dec 2018 05:54:05 -0800 (PST)
Date: Thu, 20 Dec 2018 05:54:04 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
 s=pf2014; t=1545314044;
 bh=+07r1gEQyEwhmlv7OWfuMcpE25A8IFkT5V5L3AtKRcI=;
 h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID:
 List-Archive:List-Post:List-Unsubscribe:From;
 b=VvwgztqHX2cpP89VXY1i12yqkx9qcll1Y+Y+BVqRtlM8a6OYqstHryp20JJZjgDG6
 zcY/xUnDKapJ6h6PZqgJ1LJsgCfA9V0SCluDUX7tDpLlz6b0HwtqYEdDwdD/qYmzBd
 UEPZywTq353kotCIxxEbpMJLf7jG1bSmBgs5l1TM=
From: ianswett <notifications@github.com>
Reply-To: quicwg/base-drafts
 <reply+0166e4abb323a1fcca48ddb036a4ad8aac0c7711a7a88aa892cf00000001183360fc92a169ce176ad377@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/2213/review/187005001@github.com>
In-Reply-To: <quicwg/base-drafts/pull/2213@github.com>
References: <quicwg/base-drafts/pull/2213@github.com>
Subject: Re: [quicwg/base-drafts] RTT variance may be included in threshold
 (#2213)
Mime-Version: 1.0
Content-Type: multipart/alternative;
 boundary="--==_mimepart_5c1b9efc2c940_7cd63fb11ccd45bc1371aa";
 charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: ianswett
X-GitHub-Recipient: quic-issues
X-GitHub-Reason: subscribed
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address: quic-issues@ietf.org
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/9Ogj0PhHoWOxjxJ-gkoIoaPKDOQ>
X-BeenThere: quic-issues@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Notification list for GitHub issues related to the QUIC WG
 <quic-issues.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/quic-issues>,
 <mailto:quic-issues-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/quic-issues/>
List-Post: <mailto:quic-issues@ietf.org>
List-Help: <mailto:quic-issues-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/quic-issues>,
 <mailto:quic-issues-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 20 Dec 2018 13:54:07 -0000


----==_mimepart_5c1b9efc2c940_7cd63fb11ccd45bc1371aa
Content-Type: text/plain;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

ianswett commented on this pull request.



> @@ -362,10 +362,10 @@ Using max(SRTT, latest_RTT) protects from the two following cases:
 * the latest RTT sample is higher than the SRTT, perhaps due to a sustained
   increase in the actual RTT, but the smoothed SRTT has not yet caught up.
 
-Implementers MAY experiment with using other reordering thresholds, including
-absolute thresholds, bearing in mind that a lower multiplier reduces reordering
-resilience and increases spurious retransmissions, and a higher multiplier
-increases loss detection delay.
+Implementers MAY experiment with including RTT variance in the threshold, or

To make this slightly harder still, Chromium's early experiments showed that loss was not correlated, but when an application is using the network, evidence shows loss is very correlated.  So I don't think we can understand the prevalence and nature of reordering in a network without using the network in a realistic way, such as HTTP.

-- 
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/pull/2213#discussion_r243278596
----==_mimepart_5c1b9efc2c940_7cd63fb11ccd45bc1371aa
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

<p><b>@ianswett</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/quicwg/base-drafts/pull/2213#discussion_r243278596">draft-ietf-quic-recovery.md</a>:</p>
<pre style='color:#555'>&gt; @@ -362,10 +362,10 @@ Using max(SRTT, latest_RTT) protects from the two following cases:
 * the latest RTT sample is higher than the SRTT, perhaps due to a sustained
   increase in the actual RTT, but the smoothed SRTT has not yet caught up.
 
-Implementers MAY experiment with using other reordering thresholds, including
-absolute thresholds, bearing in mind that a lower multiplier reduces reordering
-resilience and increases spurious retransmissions, and a higher multiplier
-increases loss detection delay.
+Implementers MAY experiment with including RTT variance in the threshold, or
</pre>
<p>To make this slightly harder still, Chromium's early experiments showed that loss was not correlated, but when an application is using the network, evidence shows loss is very correlated.  So I don't think we can understand the prevalence and nature of reordering in a network without using the network in a realistic way, such as HTTP.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/quicwg/base-drafts/pull/2213#discussion_r243278596">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AWbkq4c0ft8AgpZmCWbGjKD1zOE08JOvks5u65Z8gaJpZM4Zbc4n">mute the thread</a>.<img src="https://github.com/notifications/beacon/AWbkq9X20TxmGKBwtOPcz81jyF-rrrKZks5u65Z8gaJpZM4Zbc4n.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/quicwg/base-drafts","title":"quicwg/base-drafts","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/quicwg/base-drafts"}},"updates":{"snippets":[{"icon":"PERSON","message":"@ianswett commented on #2213"}],"action":{"name":"View Pull Request","url":"https://github.com/quicwg/base-drafts/pull/2213#discussion_r243278596"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/quicwg/base-drafts/pull/2213#discussion_r243278596",
"url": "https://github.com/quicwg/base-drafts/pull/2213#discussion_r243278596",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>
----==_mimepart_5c1b9efc2c940_7cd63fb11ccd45bc1371aa--

