Re: [quicwg/base-drafts] Gorry's ECN rewrite (#4059)
ianswett <notifications@github.com> Fri, 11 September 2020 17:48 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 995473A0EE3 for <quic-issues@ietfa.amsl.com>; Fri, 11 Sep 2020 10:48:05 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 3.517
X-Spam-Level: ***
X-Spam-Status: No, score=3.517 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, DKIM_VALID_EF=-0.1, GB_SUMOF=5, HTML_IMAGE_ONLY_24=1.618, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] autolearn=no 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 jdUfuQu2sW6m for <quic-issues@ietfa.amsl.com>; Fri, 11 Sep 2020 10:48:04 -0700 (PDT)
Received: from out-23.smtp.github.com (out-23.smtp.github.com [192.30.252.206]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 646593A0EBF for <quic-issues@ietf.org>; Fri, 11 Sep 2020 10:48:04 -0700 (PDT)
Received: from github-lowworker-ca235ff.ash1-iad.github.net (github-lowworker-ca235ff.ash1-iad.github.net [10.56.110.15]) by smtp.github.com (Postfix) with ESMTP id EB9B46001F1 for <quic-issues@ietf.org>; Fri, 11 Sep 2020 10:48:02 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1599846482; bh=0E52FMnaUO8gBRcQXygROc/SeUbc2QNHxKZeBr6N/5A=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=2KBfiEHbpmNwjr7Gvf2F06larAheZARE5RLOdO0JY1sk9B2L/zLFdOOu11rFLuu2Q IhWZu9E3g5NCqWCHg/sfyPJAxe/MSzicdkr8laXqYfm7fEr6zxPyCLd3Tkx6fzf8Gx OCzj5R6BYQXdP7SVBddjVJ5jSyBq0303ohkDq1xI=
Date: Fri, 11 Sep 2020 10:48:02 -0700
From: ianswett <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJK44ZLAEFWXG5E3AWZ55M6MVFEVBNHHCR5CFZA@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/4059/review/487004615@github.com>
In-Reply-To: <quicwg/base-drafts/pull/4059@github.com>
References: <quicwg/base-drafts/pull/4059@github.com>
Subject: Re: [quicwg/base-drafts] Gorry's ECN rewrite (#4059)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5f5bb852dbd9a_5fd119f03146a9"; 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/zo_3_hV9rKgxW3S5Rd0qrJgW9wM>
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: Fri, 11 Sep 2020 17:48:06 -0000
@ianswett commented on this pull request. > with an ECT(0) marking. Similarly, ECN validation fails if the sum of the increases to ECT(1) and ECN-CE counts is less than the number of newly acknowledged packets sent with an ECT(1) marking. These checks can detect -removal of ECN markings in the network. +remarking of ECN-CE markings by the network. If I'm reading correctly, the preceding two sentences describe how to detect if ECN bits have been set to 0. Detecting remarking would indicate to me that you can also detect a bit that was 0 set to 1, but the next paragraph explains why it's ok for the counts to be larger than expected. That being said, it sounds like people like the word 'remarked' when talking about ECN, so I'm fine to keep it as is. -- 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/4059#discussion_r487197110
- [quicwg/base-drafts] Gorry's ECN rewrite (#4059) Martin Thomson
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Martin Thomson
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Jana Iyengar
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Gorry Fairhurst
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Gorry Fairhurst
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Gorry Fairhurst
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Lars Eggert
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… mirjak
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Gorry Fairhurst
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Lars Eggert
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Lars Eggert
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Gorry Fairhurst
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Gorry Fairhurst
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Gorry Fairhurst
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… mirjak
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… mirjak
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… mirjak
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Gorry Fairhurst
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Lars Eggert
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Gorry Fairhurst
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Martin Thomson
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Martin Thomson
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Martin Thomson
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Martin Thomson
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Martin Thomson
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… mirjak
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… mirjak
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… mirjak
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… mirjak
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… mirjak
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Gorry Fairhurst
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Gorry Fairhurst
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Jana Iyengar
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Martin Thomson
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… ianswett
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Jana Iyengar
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… ianswett
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… ianswett
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Gorry Fairhurst
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Martin Thomson
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Martin Thomson
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Martin Thomson
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Gorry Fairhurst
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Martin Thomson
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Gorry Fairhurst
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Martin Thomson
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Gorry Fairhurst
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Martin Thomson
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Jana Iyengar
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Jana Iyengar
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… mirjak
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Gorry Fairhurst
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… ianswett
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… ianswett
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… ianswett
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… ianswett
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Martin Thomson
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Martin Thomson
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Lars Eggert
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… ianswett
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… ianswett
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… ianswett
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Martin Thomson
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Martin Thomson
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… ianswett
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Martin Thomson
- Re: [quicwg/base-drafts] Gorry's ECN rewrite (#40… Jana Iyengar