Re: [quicwg/base-drafts] MUST pace or limit bursts to 10 packets (#3106)

Martin Thomson <> Sat, 26 October 2019 01:31 UTC

Return-Path: <>
Received: from localhost (localhost []) by (Postfix) with ESMTP id 2EB17120090 for <>; Fri, 25 Oct 2019 18:31:39 -0700 (PDT)
X-Virus-Scanned: amavisd-new at
X-Spam-Flag: NO
X-Spam-Score: -6.596
X-Spam-Status: No, score=-6.596 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, HTML_IMAGE_ONLY_28=1.404, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_HI=-5, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: (amavisd-new); dkim=pass (1024-bit key)
Received: from ([]) by localhost ( []) (amavisd-new, port 10024) with ESMTP id 0aTZ2diAyMiu for <>; Fri, 25 Oct 2019 18:31:37 -0700 (PDT)
Received: from ( []) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by (Postfix) with ESMTPS id B3656120026 for <>; Fri, 25 Oct 2019 18:31:37 -0700 (PDT)
Date: Fri, 25 Oct 2019 18:31:37 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=pf2014; t=1572053497; bh=8a6Qv2PO1IE/uyf+p+oU1Ggdi/Nwu0VIS6RxnrIYWVo=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=v5a5/pL/Se6pm4cE3DIb/8Vc4VANYyYw5Zp8zjd0Z5r8hAvF9xCr/pLCzHOULH9Ss +DbHwPx/cmuAYGgZ7JMkIHQC8WBml7qfFLf151uAapDNEh750j36ayVmyXDZyCsalR Ie9ABUaZJ/EEq+gPKk1cm+PX6NqKy7Zs5g+b8LAI=
From: Martin Thomson <>
Reply-To: quicwg/base-drafts <>
To: quicwg/base-drafts <>
Cc: Subscribed <>
Message-ID: <quicwg/base-drafts/pull/3106/review/>
In-Reply-To: <quicwg/base-drafts/pull/>
References: <quicwg/base-drafts/pull/>
Subject: Re: [quicwg/base-drafts] MUST pace or limit bursts to 10 packets (#3106)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5db3a1f9521d_65223f811f6cd95c6140a6"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: martinthomson
X-GitHub-Recipient: quic-issues
X-GitHub-Reason: subscribed
X-Auto-Response-Suppress: All
Archived-At: <>
X-Mailman-Version: 2.1.29
List-Id: Notification list for GitHub issues related to the QUIC WG <>
List-Unsubscribe: <>, <>
List-Archive: <>
List-Post: <>
List-Help: <>
List-Subscribe: <>, <>
X-List-Received-Date: Sat, 26 Oct 2019 01:31:39 -0000

martinthomson commented on this pull request.

> @@ -753,6 +753,12 @@ delivery of ACK frames is important for efficient loss recovery. Packets
 containing only ACK frames should therefore not be paced, to avoid delaying
 their delivery to the peer.
+Sending multiple packets into the network without any delay between them
+creates a packet burst that might cause short-term congestion and losses.
+Implementations MUST either use pacing or limit such bursts to minimum
+of 10 * kMaxDatagramSize and max(2* kMaxDatagramSize, 14720)), the same
+as the recommended initial congestion window.

I think that explicitly saying "Initial congestion window" here as the primary target of the MUST requirement would be better.

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub: