Re: [quicwg/base-drafts] RTT variance may be included in threshold (#2213)
ianswett <notifications@github.com> Thu, 20 December 2018 13:36 UTC
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 9CD7D1277D2 for <quic-issues@ietfa.amsl.com>; Thu, 20 Dec 2018 05:36:59 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8.065
X-Spam-Level:
X-Spam-Status: No, score=-8.065 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_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 vorUYoFy1Cd6 for <quic-issues@ietfa.amsl.com>; Thu, 20 Dec 2018 05:36:57 -0800 (PST)
Received: from out-4.smtp.github.com (out-4.smtp.github.com [192.30.252.195]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 784B512426E for <quic-issues@ietf.org>; Thu, 20 Dec 2018 05:36:57 -0800 (PST)
Date: Thu, 20 Dec 2018 05:36:55 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1545313015; bh=xXkSWnLO2b954TX2WWTwVUphDz6wFKtw4ktu7VHUQWk=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=13/xPq5vOuz36ZX6ZHYW0jm6SziiUpK2q21mPn2CzftCWx9m2le0HLvI2W1I7vSHL 1S3sbLcuPci1Y0P1kwaf9R0IKS3xVNSsJsCdBhFAeTBnzJDNmhjh064zC+UK5YH5/N AHm2t5nrEiGRiW0XtH6ShZWCYzyBpTbdoyCkk038=
From: ianswett <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4abba7579b124e444816f4739ac2ef938e1ee15bb1492cf0000000118335cf792a169ce176ad377@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/186979342@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_5c1b9af749a55_b383fc9966d45bc6168"; 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/A-2t7f1H_oFvmIzrW0lSQgZYbs8>
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:37:00 -0000
ianswett commented on this pull request. FWIW, I'm still pretty skeptical that RTTVar and reordering are correlated in a meaningful way. Before we land this, I'd like to see something, anecdotal or otherwise, that indicates that's true sometimes? Or a stronger theoretical argument than in the issue. > @@ -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 If we're going to do this, I'd highly prefer adding variance to the list "using other reordering thresholds, including absolute thresholds, those including RTT variance, or those from similar previous connections. Using a lower multiplier reduces..." I added similar previous connections, because I think that's probably a fairly fruitful direction if someone wanted to work on it. -- 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#pullrequestreview-186979342
- [quicwg/base-drafts] RTT variance may be included… janaiyengar
- Re: [quicwg/base-drafts] RTT variance may be incl… janaiyengar
- Re: [quicwg/base-drafts] RTT variance may be incl… ianswett
- Re: [quicwg/base-drafts] RTT variance may be incl… MikkelFJ
- Re: [quicwg/base-drafts] RTT variance may be incl… ianswett
- Re: [quicwg/base-drafts] RTT variance may be incl… ianswett
- Re: [quicwg/base-drafts] RTT variance may be incl… MikkelFJ
- Re: [quicwg/base-drafts] RTT variance may be incl… janaiyengar
- Re: [quicwg/base-drafts] RTT variance may be incl… MikkelFJ
- Re: [quicwg/base-drafts] RTT variance may be incl… janaiyengar
- Re: [quicwg/base-drafts] RTT variance may be incl… janaiyengar
- Re: [quicwg/base-drafts] RTT variance may be incl… janaiyengar
- Re: [quicwg/base-drafts] RTT variance may be incl… janaiyengar
- Re: [quicwg/base-drafts] RTT variance may be incl… janaiyengar
- Re: [quicwg/base-drafts] RTT variance may be incl… janaiyengar
- Re: [quicwg/base-drafts] RTT variance may be incl… ianswett
- Re: [quicwg/base-drafts] RTT variance may be incl… MikkelFJ
- Re: [quicwg/base-drafts] RTT variance may be incl… ianswett
- Re: [quicwg/base-drafts] RTT variance may be incl… janaiyengar