IETF Logo Mail Archive

Re: [quicwg/base-drafts] New paths may seed initial_rtt (#2687)

ianswett <notifications@github.com> Tue, 14 May 2019 09:57 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 5A81F1200F8 for <quic-issues@ietfa.amsl.com>; Tue, 14 May 2019 02:57:35 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.009
X-Spam-Level:
X-Spam-Status: No, score=-3.009 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, 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, SPF_PASS=-0.001, T_DKIMWL_WL_HIGH=-0.01] 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 jx3f_cLmUmkR for <quic-issues@ietfa.amsl.com>; Tue, 14 May 2019 02:57:33 -0700 (PDT)
Received: from out-21.smtp.github.com (out-21.smtp.github.com [192.30.252.204]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 8AC69120091 for <quic-issues@ietf.org>; Tue, 14 May 2019 02:57:33 -0700 (PDT)
Date: Tue, 14 May 2019 02:57:32 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1557827852; bh=2wQm7RMoIaPucxFtBvvOEAhguCgsWRSxdX5ttGB88GM=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=Io64JTUkhpPxY1cDvcCIAILqww5ku1lQreuYNxt0hNoL4eS7zfBqiQxLmHcFRORQh m5UpnaukuiJAqyWo+RPVYup7xvlSzwdGWGmJyU1hisipAjNrEp5l0WszyviK41D6ml 9Zp75xD8XTAddFWRb/ly0bThdEbnwkYbNP5EsVCM=
From: ianswett <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJKYJ2N6PS3TXULHXPMN247BYZEVBNHHBUYRXKM@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/2687/c492171320@github.com>
In-Reply-To: <quicwg/base-drafts/pull/2687@github.com>
References: <quicwg/base-drafts/pull/2687@github.com>
Subject: Re: [quicwg/base-drafts] New paths may seed initial_rtt (#2687)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5cda910c33fb2_5f233f8077ecd95c1408160"; 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/R0Ik1V9xKr8D7ibrdoeudWlAuAs>
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: Tue, 14 May 2019 09:57:35 -0000

1. I think this is OK as long as we limit how many path responses we send immediately?  Realistically, it's not that different from a generate_204, but it's something to be aware of.
2. I think we need to add a whole section on paths and connection migration in recovery. See #2684   I can do that in this PR if people want? 
3. #2367 decreases the value of this, since you may only see one half of the path.  It might still be better than nothing?  Initial RTT doesn't need to be correct, but getting it within a factor of 2 is helpful.

-- 
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/2687#issuecomment-492171320

v2.23.0 | Report a Bug | By Email