Re: [quicwg/base-drafts] Implicit acknowledgment via header block acknowledgment (#1399)

Mike Bishop <notifications@github.com> Wed, 30 May 2018 17:58 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 AE56512426E for <quic-issues@ietfa.amsl.com>; Wed, 30 May 2018 10:58:40 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8.01
X-Spam-Level:
X-Spam-Status: No, score=-8.01 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] 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 H3wO53dP5RQg for <quic-issues@ietfa.amsl.com>; Wed, 30 May 2018 10:58:38 -0700 (PDT)
Received: from out-3.smtp.github.com (out-3.smtp.github.com [192.30.252.194]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 3705012D88A for <quic-issues@ietf.org>; Wed, 30 May 2018 10:58:38 -0700 (PDT)
Date: Wed, 30 May 2018 10:58:37 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1527703117; bh=+dYxPesuw3jVR0dqE5QZRR5VhMG+rsiR26AbiMmJDuo=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=X0haAb/bJa2ueX9PH8ZUU3OAgVuDLczKGpzy48cqY42IEj8iaejeaW1yIhBT2CvUc HPEu5ZK6R4kwZ5n2HEv7qEhe/GarVpBYdFmv2hyNRpFeRi70a3EXusauE6Fxr4xTX5 F7EA8chDzAmp53Xj08YHfdS7v1v8SYXohGIRtNCQ=
From: Mike Bishop <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4abd8d188ba6cf94c40ef4d63dc1ec8d05faceeacf792cf000000011726a84d92a169ce13860808@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/1399/review/124522166@github.com>
In-Reply-To: <quicwg/base-drafts/pull/1399@github.com>
References: <quicwg/base-drafts/pull/1399@github.com>
Subject: Re: [quicwg/base-drafts] Implicit acknowledgment via header block acknowledgment (#1399)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5b0ee64d44253_6f513f83e001ef84965ed"; 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/NyQL_3CrKogcd2jOCB1CIJyoQ7Q>
X-BeenThere: quic-issues@ietf.org
X-Mailman-Version: 2.1.22
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: Wed, 30 May 2018 17:58:41 -0000

MikeBishop requested changes on this pull request.

Before this change, the encoder simply kept an "Acknowledged" index that started at zero and incremented by the specified amount each time it received a TSS.

Now it needs to keep two things:  "Explicitly Acknowledged" and "Acknowledged".  When you receive a TSS, you increment Explicitly Acknowledged as before.  The actual Acknowledged state is the largest-seen value of Explicitly Acknowledged or an acknowledged Largest Reference.

We probably need to explicitly say that these implicit acknowledgements don't affect that the number in TSS is still since the last TSS.  (Unless we want the decoder to track what the encoder will have gathered from its acks, in which case it could.)

> @@ -523,6 +530,12 @@ blocks within a stream have been fully processed.
 ~~~~~~~~~~
 {:#fig-header-ack title="Header Acknowledgement"}
 
+An encoder MUST treat receipt of a Header Acknowledgment as also acknowledging
+any dynamic table entries that the header block referenced.  That is, this
+instruction is also processed as a Table Size Synchronize instruction with a
+value matching the Largest Reference of the corresponding header block.

Not quite, because TSS is a count of the inserts received since the last TSS, instead of an absolute index.

-- 
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/1399#pullrequestreview-124522166