Re: [quicwg/base-drafts] Improve ACK_ECN frame encoding (e.g., use bit-vector) (#1439)
Magnus Westerlund <notifications@github.com> Thu, 14 June 2018 10:23 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 BC5A5130EEC for <quic-issues@ietfa.amsl.com>; Thu, 14 Jun 2018 03:23:06 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8.009
X-Spam-Level:
X-Spam-Status: No, score=-8.009 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_HI=-5, SPF_PASS=-0.001, T_DKIMWL_WL_HIGH=-0.01, URIBL_BLOCKED=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 DvsG28WQChFk for <quic-issues@ietfa.amsl.com>; Thu, 14 Jun 2018 03:23:04 -0700 (PDT)
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 B1C1F130EBE for <quic-issues@ietf.org>; Thu, 14 Jun 2018 03:23:04 -0700 (PDT)
Date: Thu, 14 Jun 2018 03:23:03 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1528971783; bh=scQ9rsCGipd8GgOD6NjekNSoIy660LdIl9sS5pJONyM=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=pU/wF15/Zw1bqv6egDVA5n+qw57t6rRM+7PtrYSW7hVYiG2iaT+8Jd++V/Y4lro78 Sq9MPSo0p4O4m7UrjqvHhTj46DELA1GpcpeGIVUduKinvg/UsVjyhLK9FbXXsjbIqL h87ygi5QsutpjArvP7UlQr2pcxUtecAwcjf7uXW8=
From: Magnus Westerlund <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4ab9ac6a70514f0194c8f2982aabc6d1c2710790d9c92cf00000001173a040792a169ce13c0caa7@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/1439/397246970@github.com>
In-Reply-To: <quicwg/base-drafts/issues/1439@github.com>
References: <quicwg/base-drafts/issues/1439@github.com>
Subject: Re: [quicwg/base-drafts] Improve ACK_ECN frame encoding (e.g., use bit-vector) (#1439)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5b224207c0bb5_5f953ff865acaf88333388"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: gloinul
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/VrY6w-mKa0soaOj08IaQtMmEzOM>
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: Thu, 14 Jun 2018 10:23:07 -0000
@kazuho I think I better understand your thinking, but I am still uncertain that I fully understand. So lets see if I understand correctly how the receiver would act in the case of a packet duplications happening. So the receiver at the point of sending an ACK has received PN=2,3,4 where 4 has a CE mark. Then the ACK will say: Largest PN=4, length=3, CE vector = 001. But then arrives PN=4, 5, 5 before it is time to send the next ACK. So the duplicate of PN=4 has ECN=ECT(0), and first 5 has ECT(0) and second ECN-CE set. Thus the ACK will be: Largest PN=5, length 4, CE vector 0001? In this case the sender doesn't see that PN=5 was duplicated with different CE values. Or do you say that this must be sent as two ACKs, where the first is: Largest PN=5, length 4, CE vector 0000 and the next: Largest PN=5, length 4, CE vector 0001 So any duplication must trigger ACKing? The issues I see with this, if I have correctly understood it are: * Packet duplication results in additional ACKs * In the first case where the CE marks are in the first arrival and then not in the late coming duplication an ACK loss of the first will mask the CE mark. * Also in the case of ACK loss of the first PN=5 ACK will result in that the sender can't know if this CE mark was present in the lost ACK or not, thus resulting in an uncertainty for a sender if an CE mark is more likely an attack packet. As late remarked packets a bigger risk for the attack included in the security consideration. When it comes to bleaching, we are not only considering ECN-CE to ECT but also ECT to Not-ECT. The later more relevant when it comes to the capability check as networks that don't handle ECN correctly usually sets all the ECN bits to Not-ECT independently of value. Thus we do need to determine also when some packets has not been marked at the receiver. So, I see your proposed format work for the purpose of the sender learn the received value as long as there are no packet duplication. I still don't see this format helping with the issues that arise with packet duplication. Then you would need yet another set of modes saying duplicate with which of the 4 ECN values. >From my perspective receiver suppressing duplicated packets are a work saver for the receiver. It doesn't need to do more than PN decryption for the packet. The ECN field of the later packets are of very limited value. And the receiver requirement for implementing this is very low. And if you are not suppressing duplicated packets early, then you will have to consider what you do with each individual frame type. Is it safe to process a duplicate or do you need to suppress it at frame type level. To me it opens up the implementation to much more 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/issues/1439#issuecomment-397246970
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Martin Thomson
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… MikkelFJ
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Kazuho Oku
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Kazuho Oku
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Magnus Westerlund
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Kazuho Oku
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Magnus Westerlund
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Kazuho Oku
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Kazuho Oku
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… MikkelFJ
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Magnus Westerlund
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Kazuho Oku
- [quicwg/base-drafts] Improve ACK_ECN frame encodi… Kazuho Oku
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… MikkelFJ
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Kazuho Oku
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… ianswett
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Antoine Delignat-Lavaud
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… MikkelFJ
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Kazuho Oku
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… ianswett
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… MikkelFJ
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… MikkelFJ
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… MikkelFJ
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Kazuho Oku
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… MikkelFJ
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Kazuho Oku
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… MikkelFJ
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… MikkelFJ
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Magnus Westerlund
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Kazuho Oku
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Martin Thomson
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Kazuho Oku
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Martin Thomson
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… MikkelFJ
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Lars Eggert
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Magnus Westerlund
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Magnus Westerlund
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… MikkelFJ
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Kazuho Oku
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Kazuho Oku
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Magnus Westerlund
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Magnus Westerlund
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Martin Thomson
- Re: [quicwg/base-drafts] Improve ACK_ECN frame en… Martin Thomson