Re: [quicwg/base-drafts] Add a section on flow control performance (#3793)
Gorry Fairhurst <notifications@github.com> Fri, 26 June 2020 07:54 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 1A7CE3A11B9 for <quic-issues@ietfa.amsl.com>; Fri, 26 Jun 2020 00:54:42 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.697
X-Spam-Level:
X-Spam-Status: No, score=-1.697 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_28=1.404, 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 yWjkJZVK9x2n for <quic-issues@ietfa.amsl.com>; Fri, 26 Jun 2020 00:54:40 -0700 (PDT)
Received: from out-20.smtp.github.com (out-20.smtp.github.com [192.30.252.203]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id C208A3A11B6 for <quic-issues@ietf.org>; Fri, 26 Jun 2020 00:54:40 -0700 (PDT)
Received: from github-lowworker-d1d6e31.ash1-iad.github.net (github-lowworker-d1d6e31.ash1-iad.github.net [10.56.105.50]) by smtp.github.com (Postfix) with ESMTP id DF7B88C0B57 for <quic-issues@ietf.org>; Fri, 26 Jun 2020 00:54:39 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1593158079; bh=KGyHPtXO3jepGi/iH+j2YJsLMAxNsmoyOxGidnO/H9g=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=zcA+qyaBVu7fq6uEhb/hTox6LsS9AmEsbOam37lVvmuecX+wqXWh2FAxS6dwYC37B 7BQ+xM07LIp1hjvVTHXCTlntI+nIbjnlzjr1TcarikC7o1x0A94d/55gAfdN432cNH Rjl2ddsHM8F+cHOS89dRTMiFl/kJ9OzJxRXWWr2s=
Date: Fri, 26 Jun 2020 00:54:39 -0700
From: Gorry Fairhurst <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJK4JCIQBXXF6GLBPYA55AGFL7EVBNHHCNAR3C4@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/3793/review/438086760@github.com>
In-Reply-To: <quicwg/base-drafts/pull/3793@github.com>
References: <quicwg/base-drafts/pull/3793@github.com>
Subject: Re: [quicwg/base-drafts] Add a section on flow control performance (#3793)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5ef5a9bfcfc93_37d3ffd038cd968492697"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: gorryfair
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/u6xBhLL1ejyf37EaX2h0sS6cBQU>
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: Fri, 26 Jun 2020 07:54:42 -0000
@gorryfair commented on this pull request. > @@ -968,6 +968,24 @@ signal before advertising additional credit, since doing so will mean that the peer will be blocked for at least an entire round trip, and potentially for longer if the peer chooses to not send STREAMS_BLOCKED frames. +## Flow Control Performance + +Unlike TCP, QUIC decouples flow control from congestion control. This can +release pressure on implementing a good flow control algorithm that governs both +how limits are increased and when new limits are advertised. An endpoint can +use flow control to constrain resource commitments without an optimized +algorithm by allocating buffers for receiving data that are significantly larger +than the bandwidth-delay product (BDP). That is, by a factor of 2 or more. + +An endpoint that is unable to ensure that a peer has flow control credit in the +order of the current BDP will have receive throughput limited by flow control +and not other limiting factors like congestion control. Timely sending of +updates to flow control limits can improve performance, however an excessive +rate of updates can also adversely affect performance. I tend to agree, the present working sounds like a "significant" warning, but the penalty of updating too infrequently is also severe - perhaps and example of frequently might help? such as 1/8 RTT? -- 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/3793#discussion_r446026721
- Re: [quicwg/base-drafts] Add a section on flow co… Gorry Fairhurst
- [quicwg/base-drafts] Add a section on flow contro… Martin Thomson
- Re: [quicwg/base-drafts] Add a section on flow co… Marten Seemann
- Re: [quicwg/base-drafts] Add a section on flow co… Lars Eggert
- Re: [quicwg/base-drafts] Add a section on flow co… evolodina
- Re: [quicwg/base-drafts] Add a section on flow co… evolodina
- Re: [quicwg/base-drafts] Add a section on flow co… evolodina
- Re: [quicwg/base-drafts] Add a section on flow co… Nick Banks
- Re: [quicwg/base-drafts] Add a section on flow co… Martin Thomson
- Re: [quicwg/base-drafts] Add a section on flow co… krasic
- Re: [quicwg/base-drafts] Add a section on flow co… Mike Bishop
- Re: [quicwg/base-drafts] Add a section on flow co… Martin Thomson
- Re: [quicwg/base-drafts] Add a section on flow co… ianswett
- Re: [quicwg/base-drafts] Add a section on flow co… Martin Thomson
- Re: [quicwg/base-drafts] Add a section on flow co… ianswett
- Re: [quicwg/base-drafts] Add a section on flow co… Martin Thomson
- Re: [quicwg/base-drafts] Add a section on flow co… Martin Thomson
- Re: [quicwg/base-drafts] Add a section on flow co… ianswett
- Re: [quicwg/base-drafts] Add a section on flow co… Martin Thomson
- Re: [quicwg/base-drafts] Add a section on flow co… Martin Thomson