Re: [quicwg/base-drafts] QPACK improvement: wrap absolute index values (#1644)
Mike Bishop <notifications@github.com> Thu, 09 August 2018 22: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 9B134130E8B for <quic-issues@ietfa.amsl.com>; Thu, 9 Aug 2018 15:39:18 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8.009
X-Spam-Level:
X-Spam-Status: No, score=-8.009 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, 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, T_DKIMWL_WL_HIGH=-0.01, URIBL_BLOCKED=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 AnVkWk6YvxM3 for <quic-issues@ietfa.amsl.com>; Thu, 9 Aug 2018 15:39:16 -0700 (PDT)
Received: from out-9.smtp.github.com (out-9.smtp.github.com [192.30.254.192]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id C8CC6130ED4 for <quic-issues@ietf.org>; Thu, 9 Aug 2018 15:39:16 -0700 (PDT)
Date: Thu, 09 Aug 2018 15:39:16 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1533854356; bh=yx9BMmPCP2sxzt5jsdjDO64KHY7lO4ilVvY4A4+QTGA=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=oT6xJG8UM6QJ6Jofj2bJQquBjqcFIpHiSE52u/ce+jHJ4PGGz33Lu2DtkkcVYsWQF A+AzuF6BcmuS79ZbvcHPSAWZd3Udmiw2Fqb+aXvXsEDwCX7FJ06nGXuXIEfEkTFEDT ygFNaYM6I2Sm7jWH288oE3b/T8MLHxzSFlRQ31o8=
From: Mike Bishop <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4ab7ef5d3b26a01b453986251861c5421662acef3af92cf000000011784849492a169ce14cf3287@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/1644/411920382@github.com>
In-Reply-To: <quicwg/base-drafts/issues/1644@github.com>
References: <quicwg/base-drafts/issues/1644@github.com>
Subject: Re: [quicwg/base-drafts] QPACK improvement: wrap absolute index values (#1644)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5b6cc29424968_38a53fa73bebe624557ac"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: MikeBishop
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/Xb2Lom0NR2eNAbrW67YYawU869o>
X-BeenThere: quic-issues@ietf.org
X-Mailman-Version: 2.1.27
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, 09 Aug 2018 22:39:19 -0000
Okay, that helps a lot. I see where you're coming from. My perspective was that MaxEntries is an absolute bound on the size of the table, so if entries are off the end of the table (the "Error area" in your diagram) the correct interpretation would be that they're in the future. That always allows some room to insert without acknowledgement, because of the difference between real entries and the zero-size entries used to calculate MaxEntries. I don't think it leads to confusion since you have to reference the table on a stream to have anything to get confused by 😉 and once you've referenced an entry you have a new pin in how far the table can advance without acknowledgement. In any scheme, you can't reuse a slot until the *eviction* of the item which previously occupied it has been acknowledged. If the window is 2*MaxEntries or more, that's at least a full turnover of the table and the limit on unacknowledged entries automatically implies that requirement. If the window is MaxEntries, you have to explicitly track that dependency which will create the limit on how many unacknowledged entries can exist. I think either is easy enough to reason about, but yours tracks a single value rather than a ring buffer of references, which is a decided advantage. Okay -- I'm sufficiently convinced. -- 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/1644#issuecomment-411920382
- [quicwg/base-drafts] QPACK improvement: wrap abso… Dmitri Tikhonov
- Re: [quicwg/base-drafts] QPACK improvement: wrap … Mike Bishop
- Re: [quicwg/base-drafts] QPACK improvement: wrap … Dmitri Tikhonov
- Re: [quicwg/base-drafts] QPACK improvement: wrap … Mike Bishop
- Re: [quicwg/base-drafts] QPACK improvement: wrap … Dmitri Tikhonov
- Re: [quicwg/base-drafts] QPACK improvement: wrap … Mike Bishop
- Re: [quicwg/base-drafts] QPACK improvement: wrap … Dmitri Tikhonov
- Re: [quicwg/base-drafts] QPACK improvement: wrap … Mike Bishop
- Re: [quicwg/base-drafts] QPACK improvement: wrap … Martin Thomson
- Re: [quicwg/base-drafts] QPACK improvement: wrap … Dmitri Tikhonov
- Re: [quicwg/base-drafts] QPACK improvement: wrap … Mike Bishop
- Re: [quicwg/base-drafts] QPACK improvement: wrap … Dmitri Tikhonov
- Re: [quicwg/base-drafts] QPACK improvement: wrap … Mike Bishop
- Re: [quicwg/base-drafts] QPACK improvement: wrap … Martin Thomson
- Re: [quicwg/base-drafts] QPACK improvement: wrap … Dmitri Tikhonov
- Re: [quicwg/base-drafts] QPACK improvement: wrap … Martin Thomson
- Re: [quicwg/base-drafts] QPACK improvement: wrap … Dmitri Tikhonov
- Re: [quicwg/base-drafts] QPACK improvement: wrap … afrind
- Re: [quicwg/base-drafts] QPACK improvement: wrap … afrind
- Re: [quicwg/base-drafts] QPACK improvement: wrap … Dmitri Tikhonov
- Re: [quicwg/base-drafts] QPACK improvement: wrap … afrind