Re: [quicwg/base-drafts] Persistent Congestion Time Threshold (#2365)
ianswett <notifications@github.com> Mon, 11 February 2019 19:54 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 26BA913117B for <quic-issues@ietfa.amsl.com>; Mon, 11 Feb 2019 11:54:20 -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_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 a_HW0H0BGSHe for <quic-issues@ietfa.amsl.com>; Mon, 11 Feb 2019 11:54:18 -0800 (PST)
Received: from out-1.smtp.github.com (out-1.smtp.github.com [192.30.252.192]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 4C351131178 for <quic-issues@ietf.org>; Mon, 11 Feb 2019 11:54:18 -0800 (PST)
Date: Mon, 11 Feb 2019 11:54:17 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1549914857; bh=bj33oh1StEKx+/oNGz7W3Ju6w/HTL9HUaH6401zZPC8=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=qtyA6b8nhLaofDZwubc078iudlOjAsuLU1+BcYh1np8RYKSTwYg/B2K7XDWy1g9JZ zjlKwaqPdfx7HdzyEFyR1ApBnuJrFnfho6q1VraLFAOwcll2yPcQ92wdWhXz+PD+eg p7Q/5tA70mmrCGqsyHQLU5ihhm485MnXLBjlISkM=
From: ianswett <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4ab810e6d55bf294aa7d8cba08456fd5881f68cccc892cf00000001187994e992a169ce17fab1f6@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/2365/review/202306589@github.com>
In-Reply-To: <quicwg/base-drafts/pull/2365@github.com>
References: <quicwg/base-drafts/pull/2365@github.com>
Subject: Re: [quicwg/base-drafts] Persistent Congestion Time Threshold (#2365)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5c61d2e96030c_5d983febd16d45b428984a"; 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/Dfyz-EqtzmNse_FmcU4GDR0VPTU>
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, 11 Feb 2019 19:54:21 -0000
ianswett commented on this pull request. > + rttvar = 0 + max_ack_delay = 0 + kPersistentCongestionThreshold = 2 + +If an eck-eliciting packet is sent at time = 0, the following scenario would +illustrate persistent congestion: + + t=0 | Send Pkt #1 (App Data) + t=1 | Send Pkt #2 (PTO 1) + t=3 | Send Pkt #3 (PTO 2) + t=7 | Send Pkt #4 (PTO 3) + t=8 | Recv ACK of Pkt #4 + +The first three packets are determined to be lost when the ACK of packet 4 is +received at t=8. The congestion period is calculated as the time between the +oldest and newest lost packets: (3 - 0) = 3. The duration for persistent I think this example works, but the pseudocode below is not quite right for this reason. The easiest way to fix it would be to assume no packet number gaps and check if the number of packets lost is equal to largest - smallest + 1. -- 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/2365#discussion_r255668357
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- [quicwg/base-drafts] Persistent Congestion Time T… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… MikkelFJ
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… MikkelFJ
- Re: [quicwg/base-drafts] Persistent Congestion Ti… MikkelFJ
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… MikkelFJ
- Re: [quicwg/base-drafts] Persistent Congestion Ti… MikkelFJ
- Re: [quicwg/base-drafts] Persistent Congestion Ti… MikkelFJ
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… MikkelFJ
- Re: [quicwg/base-drafts] Persistent Congestion Ti… MikkelFJ
- Re: [quicwg/base-drafts] Persistent Congestion Ti… MikkelFJ
- Re: [quicwg/base-drafts] Persistent Congestion Ti… MikkelFJ
- Re: [quicwg/base-drafts] Persistent Congestion Ti… MikkelFJ
- Re: [quicwg/base-drafts] Persistent Congestion Ti… MikkelFJ
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Marten Seemann
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Marten Seemann
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Jana Iyengar
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Jana Iyengar
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Jana Iyengar
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Jana Iyengar
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Jana Iyengar
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Jana Iyengar
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Jana Iyengar
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Marten Seemann
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Marten Seemann
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Marten Seemann
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Marten Seemann
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Jana Iyengar
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Jana Iyengar
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Jana Iyengar
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett
- Re: [quicwg/base-drafts] Persistent Congestion Ti… Nick Banks
- Re: [quicwg/base-drafts] Persistent Congestion Ti… ianswett