Re: [quicwg/base-drafts] allow PRIORITY frames referring to placeholders exceeding `SETTING_NUM_PLACEHOLDERS` (#2761)
Lucas Pardue <notifications@github.com> Fri, 31 May 2019 15:39 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 D436F12008A for <quic-issues@ietfa.amsl.com>; Fri, 31 May 2019 08:39:27 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.391
X-Spam-Level:
X-Spam-Status: No, score=-6.391 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_IMAGE_ONLY_24=1.618, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_HI=-5, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_DKIMWL_WL_HIGH=-0.01] 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 ojBisgVnTaVn for <quic-issues@ietfa.amsl.com>; Fri, 31 May 2019 08:39:26 -0700 (PDT)
Received: from out-13.smtp.github.com (out-13.smtp.github.com [192.30.254.196]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 0EF6212003E for <quic-issues@ietf.org>; Fri, 31 May 2019 08:39:26 -0700 (PDT)
Date: Fri, 31 May 2019 08:39:25 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1559317165; bh=UDWWrT56lU1GwjLtfvdkYd0kgZLA5ZAvZWsIy1e3thM=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=HFI6bqAzMuf/1L30bUi1IwDqtNV5g7vksAeSFJRy1ggpokTGTk2Aali1EXzyJInKz WWSS+PfVD61gvgbYYv49mX741GwucBOTBjnfwkUFiMXWgfGgER8+fJKh5T6+p575+V vDDIkIy8ZbCt/2YHtpFRNeM20mdSZ+sjTHkQ5rGg=
From: Lucas Pardue <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJK3VGXFZUE5VDTMZJDV27Z6S3EVBNHHBVW7INY@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/2761/c497756420@github.com>
In-Reply-To: <quicwg/base-drafts/pull/2761@github.com>
References: <quicwg/base-drafts/pull/2761@github.com>
Subject: Re: [quicwg/base-drafts] allow PRIORITY frames referring to placeholders exceeding `SETTING_NUM_PLACEHOLDERS` (#2761)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5cf14aad517d0_324a3f9a8f6cd96414926c"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: LPardue
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/JHDBCN4IyxAXV19c6UuVEzK4LlI>
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, 31 May 2019 15:39:28 -0000
A general comment on this is that things might get racy depending on how things get resolved with the initial value of SETTINGS_NUM_PLACEHOLDERS. If the default is low and a SETTINGS frame would increase it, then the client that attempts to set placeholder priority too eagerly may not get the result they expect. In practice what we might see is that different servers implement some internal limit to num placeholders and manage the tree that way, settling on some value that works well for most clients. I think that *is* the spirit of this change. This change allows some looser behaviour and unlike HTTP/2 phantom streams, doesn't tie up stream credits for prioritisation structures. -- 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/2761#issuecomment-497756420
- [quicwg/base-drafts] allow PRIORITY frames referr… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Lucas Pardue
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Lucas Pardue
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Lucas Pardue
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Lucas Pardue
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Alessandro Ghedini
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Lucas Pardue
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Alessandro Ghedini
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Lucas Pardue
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Robin Marx
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Robin Marx
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Mike Bishop
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Mike Bishop
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… ianswett
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Martin Thomson
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Mike Bishop
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Mike Bishop
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… ianswett
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Robin Marx
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… ianswett
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… ianswett
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Mike Bishop
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… ianswett
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Mike Bishop
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Mike Bishop
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Martin Thomson
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Martin Thomson
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Mike Bishop
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Kazuho Oku
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Mike Bishop
- Re: [quicwg/base-drafts] allow PRIORITY frames re… Mike Bishop