Re: [quicwg/base-drafts] Remove ack_delay_exponent TP (#2670)

MikkelFJ <notifications@github.com> Thu, 09 May 2019 17:34 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 AF60612004A for <quic-issues@ietfa.amsl.com>; Thu, 9 May 2019 10:34:00 -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 pzQtuyxfUJcs for <quic-issues@ietfa.amsl.com>; Thu, 9 May 2019 10:33:59 -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 3882312003E for <quic-issues@ietf.org>; Thu, 9 May 2019 10:33:59 -0700 (PDT)
Date: Thu, 09 May 2019 10:33:58 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1557423238; bh=3SxUFYXh4iLlSxwwYSQa0N0ZGd3sKWoSDZOu1KCYto0=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=LnBKx806jLH58K1wbE16d5EfUWUWaMhj6Eiz52p5WuMFVibmpSOuwn6kBHuecBwwX kqWWFYmRQS914LD7QPWKCoclVqVoUF28699TcKqfOj9FzaJ7k5KUNLa796Gt0K5fW9 BvSGETjOGOHOvJe5s77i91cTzmsgubgCtQxCN7mg=
From: MikkelFJ <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJK436IZDYHZM5E6OQLV24GLQNEVBNHHBUTIZ2M@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/490996306@github.com>
In-Reply-To: <quicwg/base-drafts/issues/2670@github.com>
References: <quicwg/base-drafts/issues/2670@github.com>
Subject: Re: [quicwg/base-drafts] Remove ack_delay_exponent TP (#2670)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5cd464861b24b_4fe23f91a30cd9648107c"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: mikkelfj
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/-V6vV4bUmU1uJqr12X8VmLY-pAs>
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, 09 May 2019 17:34:01 -0000

Well, yes, you will never get it better than now resolution. But that resolution is much higher than ms if you use timespec_get calls. https://en.cppreference.com/w/c/chrono/timespec_get
It is more likely you get thrown by page cache misses. But sure, systems differ. All I'm saying is that if you are able, and can be bothered, exponents do not loose additional precision after computing a time delta. That said, I'd only call now occasionally because it can be expensive, so there you immediately loose precision. But that might be 300ns, not 1ms.

-- 
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#issuecomment-490996306