Re: [quicwg/base-drafts] Reduce restrictions on valid RTT samples (#2568)
Nick Banks <notifications@github.com> Tue, 09 April 2019 02:33 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 ACB331201B4 for <quic-issues@ietfa.amsl.com>; Mon, 8 Apr 2019 19:33:48 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8.001
X-Spam-Level:
X-Spam-Status: No, score=-8.001 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_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 Fh1EmKmrMetz for <quic-issues@ietfa.amsl.com>; Mon, 8 Apr 2019 19:33:46 -0700 (PDT)
Received: from out-2.smtp.github.com (out-2.smtp.github.com [192.30.252.193]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 8408112019E for <quic-issues@ietf.org>; Mon, 8 Apr 2019 19:33:46 -0700 (PDT)
Date: Mon, 08 Apr 2019 19:33:45 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1554777225; bh=KaI1KyItIF0tjoGeUIL5bqnD32wsGyu0vo8ZbiBf9Kc=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=pFzZ7EEwQ68jEPLF0SpYT/8ggGGpMGzXE7mMsblRGNRMEOgPm2isy2qdQhdRVjaTV bakBE2ay6K94CW8xwv0/8/HstnWJO7ssWdQgU6B/GtAqpuI4gaEBKHEy7AlC+p+Kpd mzOKQzDoF4gOSOk4ADHIQnBJ2+aRBykYFhEahwYo=
From: Nick Banks <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4ab6521eaadbe4b055cd969f37185cc7d4bb325dca992cf0000000118c3c68992a169ce19787a10@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/481078137@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_5cac04898cb5f_52f93f93b60d45c4744c6"; 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/OCT3qiavFaCx-AN5FGllWXWrauo>
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, 09 Apr 2019 02:33:49 -0000
I don't want to have a design that has a certain scenario where things break down, but just ignores it. Also, I don't want to have a design where the receiver has to decide if I can send an ACK with my current state or not. Generally, I expect receivers to only keep track of the time they received the largest packet number; and just use that when they frame an ACK. @marten-seemann as to your issues with keeping track of ACK-only packets I'd argue they are almost entirely the same as normal packets. The ONLY difference is congestion control. As far as the peer trying to attack you by explicitly not ACK'ing the packet, that's no different for any packet. The same logic applies to all packets you send. Eventually, you age out lost packets. IMO, the simplest and most straight forward design is to always send ACK frames for your complete, current state, independent of packet contents. If the receiver of the ACK frame decides the RTT measurement shouldn't be used, that seems to be trivial enough logic to add in that specific ACK processing code. But I still argue that so long as the ACK wasn't delayed too long (~max_ack_delay) then the RTT measurement should be used. You don't want the worst case traffic pattern of each flight/burst of packets ends with an ACK-only packet to cause you never to accept RTT measurements from the correspond ACK you get back. -- 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-481078137
- [quicwg/base-drafts] Reduce restrictions on valid… Jana Iyengar
- Re: [quicwg/base-drafts] Reduce restrictions on v… Nick Banks
- Re: [quicwg/base-drafts] Reduce restrictions on v… ianswett
- Re: [quicwg/base-drafts] Reduce restrictions on v… Nick Banks
- Re: [quicwg/base-drafts] Reduce restrictions on v… ianswett
- Re: [quicwg/base-drafts] Reduce restrictions on v… Nick Banks
- Re: [quicwg/base-drafts] Reduce restrictions on v… ianswett
- Re: [quicwg/base-drafts] Reduce restrictions on v… Nick Banks
- Re: [quicwg/base-drafts] Reduce restrictions on v… Jana Iyengar
- Re: [quicwg/base-drafts] Reduce restrictions on v… Jana Iyengar
- Re: [quicwg/base-drafts] Reduce restrictions on v… MikkelFJ
- Re: [quicwg/base-drafts] Reduce restrictions on v… ianswett
- Re: [quicwg/base-drafts] Reduce restrictions on v… Kazuho Oku
- Re: [quicwg/base-drafts] Reduce restrictions on v… Marten Seemann
- Re: [quicwg/base-drafts] Reduce restrictions on v… ianswett
- Re: [quicwg/base-drafts] Reduce restrictions on v… Marten Seemann
- Re: [quicwg/base-drafts] Reduce restrictions on v… Nick Banks
- Re: [quicwg/base-drafts] Reduce restrictions on v… ianswett
- Re: [quicwg/base-drafts] Reduce restrictions on v… MikkelFJ
- Re: [quicwg/base-drafts] Reduce restrictions on v… Jana Iyengar
- Re: [quicwg/base-drafts] Reduce restrictions on v… Marten Seemann
- Re: [quicwg/base-drafts] Reduce restrictions on v… Jana Iyengar
- Re: [quicwg/base-drafts] Reduce restrictions on v… Jana Iyengar
- Re: [quicwg/base-drafts] Reduce restrictions on v… Marten Seemann
- Re: [quicwg/base-drafts] Reduce restrictions on v… Nick Banks
- Re: [quicwg/base-drafts] Reduce restrictions on v… Jana Iyengar
- Re: [quicwg/base-drafts] Reduce restrictions on v… MikkelFJ
- Re: [quicwg/base-drafts] Reduce restrictions on v… ianswett
- Re: [quicwg/base-drafts] Reduce restrictions on v… Jana Iyengar
- Re: [quicwg/base-drafts] Reduce restrictions on v… Jana Iyengar