Re: [quicwg/base-drafts] kGranularity and max_ack_delay possibly should be added after exponential backoff (#2393)

Praveen Balasubramanian <notifications@github.com> Mon, 25 March 2019 21:56 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 53E25120112 for <quic-issues@ietfa.amsl.com>; Mon, 25 Mar 2019 14:56:12 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8
X-Spam-Level:
X-Spam-Status: No, score=-8 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, HTML_IMAGE_ONLY_32=0.001, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_HI=-5, 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 IkwHRhyQFgJD for <quic-issues@ietfa.amsl.com>; Mon, 25 Mar 2019 14:56:10 -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 362DF120106 for <quic-issues@ietf.org>; Mon, 25 Mar 2019 14:56:10 -0700 (PDT)
Date: Mon, 25 Mar 2019 14:56:09 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1553550969; bh=Q6QMN844rZP9XB3DTeiaLp7Fr+v7b0PVuqowLl4lZSo=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=TcbB8kiFvgIHBBTWa9v5GA+f9qkha6J3peysUQ6WuOtoJ2jKBL0NvudGIhrXfzY25 GivcvRCBBMO834pTEdrutx4SxWxDvrDcJjap2iK/PnU3YlEANpxz67vksDlrEoqI0P fKqQKardcyXyYp+wTFtt1Esz8A8+A8QoyWDRxXAU=
From: Praveen Balasubramanian <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4abd9ac26be3227a78dea64494c548166333a6ddff592cf0000000118b1107992a169ce1823a471@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/2393/476393056@github.com>
In-Reply-To: <quicwg/base-drafts/issues/2393@github.com>
References: <quicwg/base-drafts/issues/2393@github.com>
Subject: Re: [quicwg/base-drafts] kGranularity and max_ack_delay possibly should be added after exponential backoff (#2393)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5c994e791ab94_6c903f80bf2d45b829312e"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: pravb
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/b6KXSrNeJRxZrx0z1uzUKYjS4-A>
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: Mon, 25 Mar 2019 21:56:12 -0000

TCP does not use kGranularity because minRTO already accounts for it indirectly i.e. minRTO values have never been allowed to be set lower than the timer granularity. But yes in TCP minRTO is included in the exponential backoff so it makes sense to keep the equation as is. I think we are effectively using kGranularity as minRTO here. 

In any case I think max_ack_delay is going to be the dominant value here in practice unless delayed ACKs are turned off. I wonder if another option is this:
PTO = smoothed_rtt + max(4*rttvar) + max(max_ack_delay, kGranularity)


-- 
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/2393#issuecomment-476393056