Re: [quicwg/base-drafts] Change idle_timeout to max_idle_timeout (#3099)
Martin Thomson <notifications@github.com> Sun, 03 November 2019 23: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 C24AC12004D for <quic-issues@ietfa.amsl.com>; Sun, 3 Nov 2019 15:56:10 -0800 (PST)
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_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_HI=-5, SPF_HELO_NONE=0.001, 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 LzQc7FV_BhHP for <quic-issues@ietfa.amsl.com>; Sun, 3 Nov 2019 15:56:08 -0800 (PST)
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 2E05512003E for <quic-issues@ietf.org>; Sun, 3 Nov 2019 15:56:08 -0800 (PST)
Date: Sun, 03 Nov 2019 15:56:07 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1572825367; bh=NBMV6fjXFVLrC8YGWJBAQLO8BVvoapWN5RHOibRWxyg=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=ttbwwJDfIRC4Jpk9rNNY02kjejfQPxurr4kz90OB2updxhbQFlVcB02PJdHUHL1cD 07phGBy1aEKwcnP7Xfkmnt22eXnDHz6Qy3mmWi3qpicisMhjZPewGGcs1bbkrUY0L7 YQI3U7xa7nM0TmZa+/bt5SuNyhJvyzuL+Obvlwg4=
From: Martin Thomson <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJKZT5NO7IOLYL5JM3KF3ZSNZPEVBNHHB4R4NZY@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/3099/review/310849894@github.com>
In-Reply-To: <quicwg/base-drafts/pull/3099@github.com>
References: <quicwg/base-drafts/pull/3099@github.com>
Subject: Re: [quicwg/base-drafts] Change idle_timeout to max_idle_timeout (#3099)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5dbf691726fde_227a3ffc766cd95c5646a2"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: martinthomson
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/KocmXjObjZklR1tcE77YyJ-fr0s>
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: Sun, 03 Nov 2019 23:56:11 -0000
martinthomson approved this pull request. > current Probe Timeout (PTO). -Each endpoint advertises its own idle timeout to its peer. An endpoint -restarts any timer it maintains when a packet from its peer is received and -processed successfully. The timer is also restarted when sending an +Each endpoint advertises a max_idle_timeout, but the effective value +at an endpoint is computed as the minimum of the two advertised values. By announcing +a max_idle_timeout, an endpoint commits to initiating an immediate close > a max_idle_timeout, an endpoint declares that after being idle for longer than the advertised time it will not send any packets and it could generate a stateless reset ({{stateless-reset}}) in reaction to any received packet. An endpoint commits to initiating an immediate close [...] > - -The value for an idle timeout can be asymmetric. The value advertised by an -endpoint is only used to determine whether the connection is live at that -endpoint. An endpoint that sends packets near the end of the idle timeout -period of a peer risks having those packets discarded if its peer enters the -draining state before the packets arrive. If a peer could timeout within a -Probe Timeout (PTO; see Section 6.3 of {{QUIC-RECOVERY}}), it is advisable to -test for liveness before sending any data that cannot be retried safely. Note -that it is likely that only applications or application protocols will -know what information can be retried. +Each endpoint advertises a different max_idle_timeout, but the effective value +is determined by taking the minimum of the two advertised values. By announcing +a max_idle_timeout, endpoints commit to initiating an immediate close +({{immediate-close}}) if it abandons a connection prior to the effective value. + +An endpoint restarts any timer it maintains when a packet from its peer is I don't think that we need new names. New names are best when you need to refer to them in many places, but idle timeout logic is all constrained to this one section. -- 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/3099#pullrequestreview-310849894
- [quicwg/base-drafts] Idle timeout indicates you w… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… Jana Iyengar
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… Martin Thomson
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… Kazuho Oku
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… Nick Banks
- Re: [quicwg/base-drafts] Idle timeout indicates y… Martin Thomson
- Re: [quicwg/base-drafts] Idle timeout indicates y… Mike Bishop
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… Jana Iyengar
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Idle timeout indicates y… ianswett
- Re: [quicwg/base-drafts] Change to max_idle_timeo… ianswett
- Re: [quicwg/base-drafts] Change idle_timeout to m… ianswett
- Re: [quicwg/base-drafts] Change idle_timeout to m… Martin Thomson
- Re: [quicwg/base-drafts] Change idle_timeout to m… Marten Seemann
- Re: [quicwg/base-drafts] Change idle_timeout to m… Martin Thomson
- Re: [quicwg/base-drafts] Change idle_timeout to m… Jana Iyengar
- Re: [quicwg/base-drafts] Change idle_timeout to m… Jana Iyengar
- Re: [quicwg/base-drafts] Change idle_timeout to m… ianswett
- Re: [quicwg/base-drafts] Change idle_timeout to m… ianswett
- Re: [quicwg/base-drafts] Change idle_timeout to m… ianswett
- Re: [quicwg/base-drafts] Change idle_timeout to m… ianswett
- Re: [quicwg/base-drafts] Change idle_timeout to m… ianswett
- Re: [quicwg/base-drafts] Change idle_timeout to m… ianswett
- Re: [quicwg/base-drafts] Change idle_timeout to m… Martin Thomson
- Re: [quicwg/base-drafts] Change idle_timeout to m… Jana Iyengar
- Re: [quicwg/base-drafts] Change idle_timeout to m… Martin Thomson
- Re: [quicwg/base-drafts] Change idle_timeout to m… Martin Thomson
- Re: [quicwg/base-drafts] Change idle_timeout to m… Martin Thomson