Re: [quicwg/base-drafts] Reduce restrictions on valid RTT samples (#2568)

ianswett <notifications@github.com> Tue, 02 April 2019 19:03 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 0D2C5120075 for <quic-issues@ietfa.amsl.com>; Tue, 2 Apr 2019 12:03:06 -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 CqC4Zub7sOwv for <quic-issues@ietfa.amsl.com>; Tue, 2 Apr 2019 12:03:04 -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 2B98B12013E for <quic-issues@ietf.org>; Tue, 2 Apr 2019 12:03:04 -0700 (PDT)
Date: Tue, 02 Apr 2019 12:03:02 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1554231782; bh=w5sC4HJsZjqG4EezMhOMon7GRGry1RMHMhnc5Lt6GWM=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=gSW/pFrJiwcX7lgdCI4wx5c7RKp444UvN1h3jnSOngcVCUzmbgn4bCgJ+dtTU6jnw +cRuqvnj0NXVmBSvL7z/dlS5z7eq2ezPtakWEwd6ldVwm8MtUbL7v/jveuFm25w3uD 3ViLuAZSTWEBefXN7Kxs90GTdTZV96y9I/rZDlhQ=
From: ianswett <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4aba54389a2e1964fd4f56a75e56db8e0ded5eee78a92cf0000000118bb73e692a169ce19787a10@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/2568/479150237@github.com>
In-Reply-To: <quicwg/base-drafts/issues/2568@github.com>
References: <quicwg/base-drafts/issues/2568@github.com>
Subject: Re: [quicwg/base-drafts] Reduce restrictions on valid RTT samples (#2568)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5ca3b1e6bd694_5bc43f8f6b2d45bc1519e2"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: ianswett
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/l-uG4KoCvclQa2wKJIAE3rEaTTU>
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, 02 Apr 2019 19:03:06 -0000

@nibanks In both cases, I believe the current algorithm of limiting ack_delay to max_ack_delay makes perfect sense.  In the case of CPU load, RTT should show up in SRTT, RTTVar, or both.

In the second case, the text also advises including any expected timer delay in your advertised max_ack_delay.

This issue is about what happens if the largest acked is an ACK-only packet(not ack-eliciting).  In this case, we do ignore the RTT sample, and I do think that could turn out to be problematic, particularly in very asymmetric flows.

-- 
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/2568#issuecomment-479150237