IETF Logo Mail Archive

[quicwg/base-drafts] update kInitialRtt to 333ms, use that constant (#3823)

Kazuho Oku <notifications@github.com> Mon, 06 July 2020 01:46 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 90B313A0D4E for <quic-issues@ietfa.amsl.com>; Sun, 5 Jul 2020 18:46:45 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.099
X-Spam-Level:
X-Spam-Status: No, score=-3.099 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, HTML_IMAGE_ONLY_32=0.001, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=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 LipYmC8CcvS3 for <quic-issues@ietfa.amsl.com>; Sun, 5 Jul 2020 18:46:43 -0700 (PDT)
Received: from out-17.smtp.github.com (out-17.smtp.github.com [192.30.252.200]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id C31833A0D49 for <quic-issues@ietf.org>; Sun, 5 Jul 2020 18:46:43 -0700 (PDT)
Received: from github-lowworker-ca235ff.ash1-iad.github.net (github-lowworker-ca235ff.ash1-iad.github.net [10.56.110.15]) by smtp.github.com (Postfix) with ESMTP id D217F6E210B for <quic-issues@ietf.org>; Sun, 5 Jul 2020 18:46:42 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1594000002; bh=xNXATLf0e8hNbS9+tTs5DFB3ZtfDar6UhHL6JYifHNg=; h=Date:From:Reply-To:To:Cc:Subject:List-ID:List-Archive:List-Post: List-Unsubscribe:From; b=QkeY3nJsrHy7RKpRN2/0nkLWHdCWWqv5Drff+mYVp2YBkIH7mTyUx7ZQDCDpBvCO+ NPv2STvTcq0l7tVDvZUTchl+rKojl4cNsO4KkzbS6Jp4+s4bAFLPnq4tgMh6a5CWQO WomNeni1bjVVkpbNUay5aCgSZp95M38kKWajMDOc=
Date: Sun, 05 Jul 2020 18:46:42 -0700
From: Kazuho Oku <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJK3ZPZGIYAQMMML6WI55BZRYFEVBNHHCNUDN6I@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/3823@github.com>
Subject: [quicwg/base-drafts] update kInitialRtt to 333ms, use that constant (#3823)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5f028282c30b1_6f993f8129ccd96016635"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: kazuho
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/hDLQ_YgcYebMrnHMStuJHPFearQ>
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: Mon, 06 Jul 2020 01:46:46 -0000

In #3525, we changed the definition of initial PTO from 2x initial RTT to 3x, and therefore initial RTT had to be changed from 500ms to 333ms so that the initial PTO remains at the same value (1 second).

Unfortunately, the PR introduced a new variable called initial_rtt, and left kInitialRtt unmodified.

This PR fixes the editorial issue.

Closes #3822.
You can view, comment on, or merge this pull request online at:

  https://github.com/quicwg/base-drafts/pull/3823

-- Commit Summary --

  * update kInitialRtt to 333ms, use that constant

-- File Changes --

    M draft-ietf-quic-recovery.md (6)

-- Patch Links --

https://github.com/quicwg/base-drafts/pull/3823.patch
https://github.com/quicwg/base-drafts/pull/3823.diff

-- 
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/3823

v2.26.15 | Report a Bug | By Email | System Status