[quicwg/base-drafts] Remove ack_delay_exponent TP (#2670)
Nick Banks <notifications@github.com> Tue, 07 May 2019 14:20 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 CF74D120151 for <quic-issues@ietfa.amsl.com>; Tue, 7 May 2019 07:20:40 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.464
X-Spam-Level:
X-Spam-Status: No, score=-6.464 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_20=1.546, 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 XlgEnnsbhTAn for <quic-issues@ietfa.amsl.com>; Tue, 7 May 2019 07:20:39 -0700 (PDT)
Received: from out-4.smtp.github.com (out-4.smtp.github.com [192.30.252.195]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id DE68D12013C for <quic-issues@ietf.org>; Tue, 7 May 2019 07:20:38 -0700 (PDT)
Date: Tue, 07 May 2019 07:20:37 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1557238837; bh=Sk7+IrDPaaEyOJTNVwz0zVCKfTXnLQpUzbeXuSqDW8A=; h=Date:From:Reply-To:To:Cc:Subject:List-ID:List-Archive:List-Post: List-Unsubscribe:From; b=FaV8OlQnATy6EHtur0/WGgaiLpygPARVcdPcyQO7OxgGcwUoTIxlGRi1H4RyTa6zb G2/DmM4sgulJpJbqPEVjqW/TLivrNA8FIECo7eKpZpuGqu4pVwnUUr7SmxCLEcA7I+ GYk5HuwB1Jpqr79mRwTlrmInxyuFWtFUVqkIlhnI=
From: Nick Banks <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJKYDJLMZR7OMZXMQMG5233DLLEVBNHHBUTIZ2M@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/2670@github.com>
Subject: [quicwg/base-drafts] Remove ack_delay_exponent TP (#2670)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5cd19435b280c_2a503f88cfacd96418736e"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: nibanks
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/J774DfabTC5dhcNRP5YkILQJUhI>
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: Tue, 07 May 2019 14:20:41 -0000
I asked the question on Slack and got quite a bit of feedback so I figured I'd open an issue. Right now, it seems all but one implementation (quicly) is using the default ack_delay_exponent value of 8. Quicly is using an even higher value of 10 because they don't care about more than millisecond timer resolution. This leads me to believe that having this be a configurable value (in the QUIC TP) is an over design, and we should simplify this. Pick a value (8 or 10, I personally don't care which) and just leave it at that. The only argument I've heard for keeping it configurable is that intra-DC environments might want to have a smaller exponent to have more fine grain ACK delays. I'm not sure that's a great reason, personally. At that small of time granularity, most OS timers aren't going to be very accurate. So, even if you can calculate an extremely accurrate RTT, it's not going to help your loss recovery timers any. -- 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/2670
- [quicwg/base-drafts] Remove ack_delay_exponent TP… Nick Banks
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… ianswett
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Marten Seemann
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… ianswett
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Nick Banks
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Nick Banks
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… ianswett
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Nick Banks
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Kazuho Oku
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Jana Iyengar
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Nick Banks
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Kazuho Oku
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Jana Iyengar
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Martin Thomson
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… MikkelFJ
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… ianswett
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Nick Banks
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… MikkelFJ
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Nick Banks
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… MikkelFJ
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… ianswett
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Kazuho Oku
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… MikkelFJ
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Kazuho Oku
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… ianswett
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… MikkelFJ
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Kazuho Oku
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… MikkelFJ
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Kazuho Oku
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… MikkelFJ
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… MikkelFJ
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Nick Banks
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Jana Iyengar
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Christian Huitema
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Martin Thomson
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Nick Banks
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… ianswett
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Marten Seemann
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… MikkelFJ
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Christian Huitema
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Kazuho Oku
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… MikkelFJ
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Martin Thomson
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Marten Seemann
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Martin Thomson
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… ianswett
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Kazuho Oku
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Martin Thomson
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Martin Thomson
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… MikkelFJ
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… MikkelFJ
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Martin Thomson
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Jana Iyengar
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Lars Eggert
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Lars Eggert
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Martin Thomson
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… MikkelFJ
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Christian Huitema
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… MikkelFJ
- Re: [quicwg/base-drafts] Remove ack_delay_exponen… Martin Thomson