Re: [quicwg/base-drafts] Add further guidance related to unidirectional stream TPs (#2612)
Kazuho Oku <notifications@github.com> Mon, 15 April 2019 03:27 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 E491512032F for <quic-issues@ietfa.amsl.com>; Sun, 14 Apr 2019 20:27:33 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8.001
X-Spam-Level:
X-Spam-Status: No, score=-8.001 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_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_HI=-5, 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 vjQr8dqZi1Nz for <quic-issues@ietfa.amsl.com>; Sun, 14 Apr 2019 20:27:32 -0700 (PDT)
Received: from out-6.smtp.github.com (out-6.smtp.github.com [192.30.252.197]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 2522F12032A for <quic-issues@ietf.org>; Sun, 14 Apr 2019 20:27:32 -0700 (PDT)
Date: Sun, 14 Apr 2019 20:27:30 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1555298850; bh=8Iilg0mPW6D2/qC+9uNGFR2WYXOhabe7iDdjEgswZoE=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=ZqO7Tdv28Edf9xHJP/Nwk+5DgxDmQwbSQIDWrwfgoScj+cVaAy/4pCh4Mf3E2inlW 4Gjt1kFNA21lOTBHgg4BMBmhKjpktPkwR3s5OH8nBrwhg9w3X8MAbD5hjZxv4T99OP 9BvtOyZBlCLTxLjXz+b8FtdMW8qmctDZBmnIPtZA=
From: Kazuho Oku <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4ab848e883798fa7ad868dcdafbee44dd35cd26732592cebac12ca292a169ce19c7a23a@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/2612/review/226441557@github.com>
In-Reply-To: <quicwg/base-drafts/pull/2612@github.com>
References: <quicwg/base-drafts/pull/2612@github.com>
Subject: Re: [quicwg/base-drafts] Add further guidance related to unidirectional stream TPs (#2612)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5cb3fa22c5392_37a93faabc0d45c0650959"; 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/yaS7bkw6WQlxZ5FK8HNAt1RqCQA>
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, 15 Apr 2019 03:27:34 -0000
kazuho commented on this pull request. > @@ -321,8 +321,19 @@ defined in this document: control streams ({{control-streams}}) and push streams ({{push-streams}}). Other stream types can be defined by extensions to HTTP/3; see {{extensions}} for more details. -Both clients and servers SHOULD send a value of three or greater for the QUIC -transport parameter `initial_max_uni_streams`. +The performance of HTTP/3 connections in the early phase of their lifetime is +sensitive to the creation and exchange of data on unidirectional streams. +Endpoints that set low values for the QUIC transport parameters +`initial_max_uni_streams` and `initial_max_stream_data_uni` will increase the +chance that the remote peer reaches the limit early. In particular, the value +chosen for `initial_max_uni_streams` should consider that remote peers may wish +to exercise reserved stream behaviour ({{stream-grease}}). Limits can be +increased post handshake at the cost of additional RTT, for example by +exchanging `STREAMS_BLOCKED` and `MAX_STREAMS` QUIC frames. To reduce the +possibility of this situation occuring, both clients and servers SHOULD send a +value of three or greater for the QUIC transport parameter +`initial_max_uni_streams`, and a value of eight or greater for the QUIC +transport `initial_max_stream_data_uni`. While I understand the motivation to suggest something for `initial_max_stream_data_uni`, 8 bytes sounds strangely small. I'd argue that we should recommend a value that is greater than the size of SETTINGS frames that most endpoints would send. Also, considering the fact that QUIC endpoint needs to be capable of processing at least one full-sized packet at a time, I do not think that even the most restricted endpoints would set `initial_max_stream_data_uni` below one MTU. Therefore, I might propose something like "SHOULD be at least 1,024 bytes." -- 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/2612#pullrequestreview-226441557
- [quicwg/base-drafts] Add further guidance related… Lucas Pardue
- Re: [quicwg/base-drafts] Add further guidance rel… ianswett
- Re: [quicwg/base-drafts] Add further guidance rel… Lucas Pardue
- Re: [quicwg/base-drafts] Add further guidance rel… ianswett
- Re: [quicwg/base-drafts] Add further guidance rel… Lucas Pardue
- Re: [quicwg/base-drafts] Add further guidance rel… Marten Seemann
- Re: [quicwg/base-drafts] Add further guidance rel… Lucas Pardue
- Re: [quicwg/base-drafts] Add further guidance rel… Kazuho Oku
- Re: [quicwg/base-drafts] Add further guidance rel… Lucas Pardue
- Re: [quicwg/base-drafts] Add further guidance rel… Lucas Pardue
- Re: [quicwg/base-drafts] Add further guidance rel… Jana Iyengar
- Re: [quicwg/base-drafts] Add further guidance rel… Lucas Pardue
- Re: [quicwg/base-drafts] Add further guidance rel… Mike Bishop
- Re: [quicwg/base-drafts] Add further guidance rel… Mike Bishop