Re: [quicwg/base-drafts] Allow alternate restarts after idle (#2187)

janaiyengar <notifications@github.com> Tue, 18 December 2018 22:30 UTC

Return-Path: <bounces+848413-a050-quic-issues=ietf.org@sgmail.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 66FEA126F72 for <quic-issues@ietfa.amsl.com>; Tue, 18 Dec 2018 14:30:13 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.064
X-Spam-Level:
X-Spam-Status: No, score=-3.064 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.065, 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_NONE=-0.0001, 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 Hr7CtBNUJSAH for <quic-issues@ietfa.amsl.com>; Tue, 18 Dec 2018 14:30:11 -0800 (PST)
Received: from o7.sgmail.github.com (o7.sgmail.github.com [167.89.101.198]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 63F33129BBF for <quic-issues@ietf.org>; Tue, 18 Dec 2018 14:30:11 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=github.com; h=from:reply-to:to:cc:in-reply-to:references:subject:mime-version:content-type:content-transfer-encoding:list-id:list-archive:list-post:list-unsubscribe; s=s20150108; bh=u0q3E5STc2CsSQO0vyW9+e3GC6c=; b=HFdzktt4vzn5HX5H ewo1zQ41sOdXfGReQDuqD+WIsULCTzNkKsENgwctGFDKTgQxYUu49yUR4F0iXuez wQ+t0S5r+8BY81yoXdodPirp6lhUy8R0CgnJ0IIipdSLhhn8bUxea8jhAEsTVtBr 8a03JTTB+rnAWvx16UeZeRqZ6Bg=
Received: by filter0283p1iad2.sendgrid.net with SMTP id filter0283p1iad2-4031-5C1974F0-6E 2018-12-18 22:30:08.879161415 +0000 UTC m=+101381.935224778
Received: from github-lowworker-56a5eb2.cp1-iad.github.net (unknown [192.30.252.33]) by ismtpd0020p1iad2.sendgrid.net (SG) with ESMTP id GrX1hgNXSGua7HYSG8A-xQ for <quic-issues@ietf.org>; Tue, 18 Dec 2018 22:30:08.796 +0000 (UTC)
Received: from github.com (localhost [127.0.0.1]) by github-lowworker-56a5eb2.cp1-iad.github.net (Postfix) with ESMTP id C9DCBC0098 for <quic-issues@ietf.org>; Tue, 18 Dec 2018 14:30:08 -0800 (PST)
Date: Tue, 18 Dec 2018 22:30:08 +0000
From: janaiyengar <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4abfef71f11474e2d42a4a86886accf13d1cdaa65aa92cf00000001183136f092a169ce1752fa68@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/2187/review/186313224@github.com>
In-Reply-To: <quicwg/base-drafts/pull/2187@github.com>
References: <quicwg/base-drafts/pull/2187@github.com>
Subject: Re: [quicwg/base-drafts] Allow alternate restarts after idle (#2187)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5c1974f0c83f8_b913fe58f0d45bc139597"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: janaiyengar
X-GitHub-Recipient: quic-issues
X-GitHub-Reason: subscribed
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address: quic-issues@ietf.org
X-SG-EID: l64QuQ2uJCcEyUykJbxN122A6QRmEpucztpreh3Pak3BJjOmdBcrt1QU0hGAgR7EXGc8rE0bly+3Nt mRw5fgszFSverKjXLIBH/H4M+UATIpOiYE11zNBo3EfmIb79vGa+5D5i94lHahw2+eV5zsZ4rcjONZ I98Q2o676jvdOZ8A5EZ+CYaskBUXZeROFZQYmWdrhq3OjJtk0V9ElYI+vzFeZcYnAtLP5rKLx1Gg86 Q=
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/5EyUmGlkLb8P0_tWhDNeURC3tJs>
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, 18 Dec 2018 22:30:13 -0000

janaiyengar commented on this pull request.



> -idle depends upon whether pacing is used.
-
-If the sender uses pacing, the connection should limit the initial burst of
-packets to no more than the initial congestion window and subsequent packets
-SHOULD be paced. The congestion window does not change while the connection
-is idle.
-
-A sender that does not use pacing SHOULD reset its congestion window to the
-minimum of the current congestion window and the initial congestion window.
-This recommendation is based on Section 4.1 of {{?RFC5681}}.
+
+## Sending data after an idle period
+
+A sender becomes idle if it has no bytes in flight.  A sender can become idle
+when it is application limited and has no data to send, or when it marks packets
+as lost (and therefore not in flight).

Yes, that is a fine case ... this is what we've always had, the RTO text was incorrect. Basically the sender goes idle as soon as bytes in flight goes to 0. Maybe that's all we need to say. I've removed more text to make this clearer.

-- 
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/pull/2187#discussion_r242730224