Re: [quicwg/base-drafts] Clarify ACK Delay and reference host-delay (#3971)

Martin Thomson <notifications@github.com> Thu, 30 July 2020 02:07 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 A37703A0C02 for <quic-issues@ietfa.amsl.com>; Wed, 29 Jul 2020 19:07:09 -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 2pl8-V8K8CY3 for <quic-issues@ietfa.amsl.com>; Wed, 29 Jul 2020 19:07:08 -0700 (PDT)
Received: from out-25.smtp.github.com (out-25.smtp.github.com [192.30.252.208]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D467F3A0BA8 for <quic-issues@ietf.org>; Wed, 29 Jul 2020 19:07:07 -0700 (PDT)
Received: from github-lowworker-2ef7ba1.ac4-iad.github.net (github-lowworker-2ef7ba1.ac4-iad.github.net [10.52.16.66]) by smtp.github.com (Postfix) with ESMTP id 40C7E840E57 for <quic-issues@ietf.org>; Wed, 29 Jul 2020 19:07:06 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1596074826; bh=2a7wq/6jjQ4KVue6j7CICMyNwMUFTbboIeu+2risZlU=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=03hSiIiC/XrKkfODDaiMMmuqfKvfOOMZIFK/B7YyJDMhZEor96NBFGihnHsg7Lto2 sKB7n49wenFF638eGemOdjXahNK/vsRhZsZUMukuErg5Enre/zt4JpUEt/ZnAeUdRd 8h08IbUilSX07IsItfJ9rPVPHYBswoPqWZ25kKrg=
Date: Wed, 29 Jul 2020 19:07:06 -0700
From: Martin Thomson <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJK6TCI6R3SQ4WGJKOIF5FYGEVEVBNHHCPTRR6M@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/3971/review/458022836@github.com>
In-Reply-To: <quicwg/base-drafts/pull/3971@github.com>
References: <quicwg/base-drafts/pull/3971@github.com>
Subject: Re: [quicwg/base-drafts] Clarify ACK Delay and reference host-delay (#3971)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5f222b4a310e9_7de016f8290185"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: martinthomson
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/VDry1yvIhUm-haT1KMvTM4lihFg>
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: Thu, 30 Jul 2020 02:07:16 -0000

@martinthomson commented on this pull request.



> +: A variable-length integer representing the intentional delay in microseconds
+  between when the largest acknowledged packet, as indicated in the Largest
+  Acknowledged field, was received, and when the ACK frame was sent;
+  see {{host-delay}}.  The value of the ACK Delay field is scaled by

Including the encoding/unit details in the description makes this a very hard sentence to swallow.  I can only think of ways to make this longer though...

This value represents the length of the delay between when the largest acknowledged packet, as indicated in the Largest Acknowledged field, was received, and when the ACK frame was sent; see {{host-delay}}.  ACK Delay is encoded as a variable-length integer with a value that is derived by dividing the delay in microseconds by 2 to the power of the value of the ack_delay_exponent transport parameter set by the sender of the ACK frame; see {{transport-parameter-definitions}}.  To decode the value, the encoded variable-length integer value is multiplied by 2 to the power of the ack_delay_exponent transport parameter to obtain a value in microseconds.  Scaling in this fashion allows ...

-- 
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/3971#pullrequestreview-458022836