Re: [quicwg/base-drafts] Proposal for adding ECN support to QUIC. (#1372)
Kazuho Oku <notifications@github.com> Mon, 18 June 2018 14:59 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 BC264120049 for <quic-issues@ietfa.amsl.com>; Mon, 18 Jun 2018 07:59:45 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.01
X-Spam-Level:
X-Spam-Status: No, score=-3.01 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001, T_DKIMWL_WL_HIGH=-0.01] 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 IbLLXWmn4v_4 for <quic-issues@ietfa.amsl.com>; Mon, 18 Jun 2018 07:59:43 -0700 (PDT)
Received: from o5.sgmail.github.com (o5.sgmail.github.com [192.254.113.10]) (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 AA4D6130DE1 for <quic-issues@ietf.org>; Mon, 18 Jun 2018 07:59:42 -0700 (PDT)
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=rTQwkxlBI2puJhdLxV/BnahVctk=; b=Q3o4getQfNPWoWes cPZ3SVFdt1KRpuS84MhGP2pFle8aiPcIOz+ug7nc7v8nVhI0U+sry2wMWXOS5wvT mVRPFNlGF6ZbrgTEmsiIowNXJwCrrSlwDp3zbPk3JTqiMy8Lozo6MEcFU8Ofc103 Y+nl2bAxRBpUwK4Dow47zXQoj8U=
Received: by filter0124p1iad2.sendgrid.net with SMTP id filter0124p1iad2-13255-5B27C8DD-14 2018-06-18 14:59:41.338404378 +0000 UTC
Received: from github-lowworker-643483b.cp1-iad.github.net (unknown [192.30.252.45]) by ismtpd0003p1iad1.sendgrid.net (SG) with ESMTP id zywohZgmRDCOcLPeicn2XQ for <quic-issues@ietf.org>; Mon, 18 Jun 2018 14:59:41.286 +0000 (UTC)
Received: from github.com (localhost [127.0.0.1]) by github-lowworker-643483b.cp1-iad.github.net (Postfix) with ESMTP id 442456C13D5 for <quic-issues@ietf.org>; Mon, 18 Jun 2018 07:59:41 -0700 (PDT)
Date: Mon, 18 Jun 2018 14:59:41 +0000
From: Kazuho Oku <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4ab8f883871e22d1a6c097b28c97903ac50cfc3a03792cf00000001173f8add92a169ce13656182@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/1372/review/129602064@github.com>
In-Reply-To: <quicwg/base-drafts/pull/1372@github.com>
References: <quicwg/base-drafts/pull/1372@github.com>
Subject: Re: [quicwg/base-drafts] Proposal for adding ECN support to QUIC. (#1372)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5b27c8dd429dc_65ff3ff5d5e64f7852742"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: kazuho
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: l64QuQ2uJCcEyUykJbxN122A6QRmEpucztpreh3Pak13QzsgPs8JcjCrfK06uvcW3+5Ay8XFUzd8eF UtGdFhSG1BErd6yCY2ilAzG8WtlJHbDNzIJeplZO2ui3g//NmoX/jkaMl4yN096yv1IzDK84ZH84ss mBSMzsMutzNB0E1f28YMt65KHlYEACWwvvU4GpiEqbqS5I3By7Imv/JkuXVszbiHixzi5FFe6i1ox5 s=
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/Y5llIT37ftCaMcxy2x5G_IF2uiI>
X-BeenThere: quic-issues@ietf.org
X-Mailman-Version: 2.1.26
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, 18 Jun 2018 14:59:46 -0000
kazuho commented on this pull request. > @@ -2887,6 +3033,95 @@ by a client in protected packets, because it is certain that the server is able to decipher the packet. +## ACK_ECN Frame {#frame-ack-ecn} + +A QUIC connection MUST keep counters for each ECN codepoint, recording +the number of packets that were received with the corresponding ECN +codepoint in the IP header. If the header is not readable from the +application, the codepoint 00 (Not-ECT) MUST be assumed. If any packet +are duplicated by the network then only the value of the ECN field of +the packet copy first received SHALL be included in the counters, the ECN +field value for a duplicate SHALL be ignored. This to prevent the Thank you for the clarification. My two cents go to that this is an unnecessary burden, because IMO while idempotency is required for frame payloads, it is not required for ECN signal that is injected by an on-path device (even though we need some defense against injection attacks). -- 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/1372#discussion_r196110762
- Re: [quicwg/base-drafts] Proposal for adding ECN … Martin Thomson
- Re: [quicwg/base-drafts] Proposal for adding ECN … janaiyengar
- Re: [quicwg/base-drafts] Proposal for adding ECN … Martin Thomson
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Lars Eggert
- Re: [quicwg/base-drafts] Proposal for adding ECN … ianswett
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … ianswett
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Martin Thomson
- Re: [quicwg/base-drafts] Proposal for adding ECN … Kazuho Oku
- Re: [quicwg/base-drafts] Proposal for adding ECN … Martin Thomson
- Re: [quicwg/base-drafts] Proposal for adding ECN … janaiyengar
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … mirjak
- Re: [quicwg/base-drafts] Proposal for adding ECN … ianswett
- Re: [quicwg/base-drafts] Proposal for adding ECN … mirjak
- Re: [quicwg/base-drafts] Proposal for adding ECN … mirjak
- Re: [quicwg/base-drafts] Proposal for adding ECN … ianswett
- Re: [quicwg/base-drafts] Proposal for adding ECN … mirjak
- Re: [quicwg/base-drafts] Proposal for adding ECN … Kazuho Oku
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Kazuho Oku
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … ianswett
- Re: [quicwg/base-drafts] Proposal for adding ECN … mirjak
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … ianswett
- Re: [quicwg/base-drafts] Proposal for adding ECN … Kazuho Oku
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … ianswett
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … ianswett
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Martin Thomson
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Lars Eggert
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … ianswett
- [quicwg/base-drafts] Proposal for adding ECN supp… Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … IngJohEricsson
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Martin Thomson
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Magnus Westerlund
- Re: [quicwg/base-drafts] Proposal for adding ECN … Kazuho Oku
- Re: [quicwg/base-drafts] Proposal for adding ECN … Lars Eggert