IETF Logo Mail Archive

Re: [quicwg/base-drafts] Proposed change to #1965 (#2212)

martinduke <notifications@github.com> Thu, 20 December 2018 04:13 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 C5771130F1A for <quic-issues@ietfa.amsl.com>; Wed, 19 Dec 2018 20:13:36 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8.065
X-Spam-Level:
X-Spam-Status: No, score=-8.065 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_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 HMz_KS0WJ2RL for <quic-issues@ietfa.amsl.com>; Wed, 19 Dec 2018 20:13:35 -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 4EBDA130EE5 for <quic-issues@ietf.org>; Wed, 19 Dec 2018 20:13:35 -0800 (PST)
Date: Wed, 19 Dec 2018 20:13:34 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1545279214; bh=qfeV7bdcZK9fBh0RlMYfT7QP2sLuWemw+vumHTK/LWw=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=FOXZ/Icje/vlU83asowqPJSnEl3reZ2waWHLnwgltYwnr3pA0fB7q0sMihxTOIdXh 3rpwrFMwowwbtFbXY8tNGqunix5zUQUyNfUrWa2veJ1M3BUvAsanxbDKaAJbQiZ5LN yzdASjxGt6VnuOLzebPcMoohcizIdEWfJx2CTGNM=
From: martinduke <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4abc626014bcc67f049a5d044bc3ab36997cfa0a4fd92cf000000011832d8ee92a169ce176a63e1@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/2212/c448860464@github.com>
In-Reply-To: <quicwg/base-drafts/pull/2212@github.com>
References: <quicwg/base-drafts/pull/2212@github.com>
Subject: Re: [quicwg/base-drafts] Proposed change to #1965 (#2212)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5c1b16ee11dda_5ce63f964aed45c0178368"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: martinduke
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/QPSMU57Rz-Dgtwhm4cQddctktRw>
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: Thu, 20 Dec 2018 04:13:37 -0000

I like this change, in principle.

I think the current editor's draft suffers from a little conceptual muddle. In TCP, losses, retransmissions, and congestion responses are all tightly coupled. For good reason, the draft backs off on the congestion response bit, and also delays losses, but not always retransmissions. As I indicated in #2103, separating retransmissions and losses creates some ambiguity. I like the idea of just declaring stuff lost. It's conceptually cleaner.

That said, declaring everything lost, rather than the things we're retransmitting, seems like overkill.

-- 
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/2212#issuecomment-448860464

v2.37.1 | Report a Bug | By Email | System Status