IETF Logo Mail Archive

Re: [quicwg/base-drafts] Persistent congestion period: text and pseudo-code don't agree (#3972)

Marten Seemann <notifications@github.com> Thu, 30 July 2020 02:13 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 E832A3A0B9A for <quic-issues@ietfa.amsl.com>; Wed, 29 Jul 2020 19:13:24 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.009
X-Spam-Level:
X-Spam-Status: No, score=-2.009 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_16=1.092, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_NONE=0.001, 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 niklAYjOwPCD for <quic-issues@ietfa.amsl.com>; Wed, 29 Jul 2020 19:13:23 -0700 (PDT)
Received: from out-22.smtp.github.com (out-22.smtp.github.com [192.30.252.205]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 3E6133A0B98 for <quic-issues@ietf.org>; Wed, 29 Jul 2020 19:13:23 -0700 (PDT)
Received: from github-lowworker-c73936b.ash1-iad.github.net (github-lowworker-c73936b.ash1-iad.github.net [10.56.112.13]) by smtp.github.com (Postfix) with ESMTP id 673F8560BFB for <quic-issues@ietf.org>; Wed, 29 Jul 2020 19:13:22 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1596075202; bh=bmjC4kbYZEd1pxni7xMVV5/flSN/Ix56EGs2O8U0+cE=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=z4JEtpsI5nQFEE01BklqiyLzlcTnKu2s4yqLe7SGOy45tJEpB8CLp/oxbBCHKb8I9 eOJ6BNsfO/oVh3yqh2I2ZjRWZII73tNVTjbIVMkxLzFlZYwdpdqRL/Y+aZ1pjs6Rzq hWgcSCsVx9Z4HBgHvrnbgDArUp4nNBRzq4tD+HqI=
Date: Wed, 29 Jul 2020 19:13:22 -0700
From: Marten Seemann <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJK7KQC52L6OKV34KG2N5FYG4FEVBNHHCPUC7GA@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/3972/666040269@github.com>
In-Reply-To: <quicwg/base-drafts/issues/3972@github.com>
References: <quicwg/base-drafts/issues/3972@github.com>
Subject: Re: [quicwg/base-drafts] Persistent congestion period: text and pseudo-code don't agree (#3972)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5f222cc256dcf_688516f8440284"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: marten-seemann
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/PYJtUWt2H0pJUKjD0Z_civeft-0>
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, 30 Jul 2020 02:13:25 -0000

I agree that making the change to the pseudo-code seems to be the more correct solution.

That being said, I'm not planning to implement persistent congestion detection during the early phases of the handshake (but only after confirmation of the handshake), since I don't see clear benefits of the added complexity of cross-packet-number-space persistent congestion detection.

-- 
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/issues/3972#issuecomment-666040269

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