Re: [quicwg/base-drafts] More ECN verification failure modes (#2198)
Magnus Westerlund <notifications@github.com> Mon, 17 December 2018 12:57 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 22DC712872C for <quic-issues@ietfa.amsl.com>; Mon, 17 Dec 2018 04:57:56 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -9.459
X-Spam-Level:
X-Spam-Status: No, score=-9.459 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-1.46, 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 mXSZItuDIP7y for <quic-issues@ietfa.amsl.com>; Mon, 17 Dec 2018 04:57:53 -0800 (PST)
Received: from out-5.smtp.github.com (out-5.smtp.github.com [192.30.252.196]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id A65A91286E7 for <quic-issues@ietf.org>; Mon, 17 Dec 2018 04:57:53 -0800 (PST)
Date: Mon, 17 Dec 2018 04:57:52 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1545051472; bh=PtbyNS84+FiqfWY99cGVT6tyVpxWfvZA+ubjtxmWw3g=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=McwyL3Q/O4aYCZxan3PxnVIB4K3aQ0dRvta7kXsJ+XMYCjqRCmbRhCLOcnymlq1BP nzT4SNFByLnZmkyFOmLMv0H4LYu83Jmzjtx/ja/8+p4Wa1Sgb4AbMD5n7JPY+WLJ00 QYEL1/IbK47fc4zY4BC9F14kR5fQNCGT47FXnG9M=
From: Magnus Westerlund <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4abea7d28404d3e90f7d5620f0e26ea038d29052e6292cf00000001182f5f5092a169ce17566533@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/2198/447836873@github.com>
In-Reply-To: <quicwg/base-drafts/issues/2198@github.com>
References: <quicwg/base-drafts/issues/2198@github.com>
Subject: Re: [quicwg/base-drafts] More ECN verification failure modes (#2198)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5c179d50468bf_2adc3fe3c86d45c01226f0"; 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/pxzG0SC3YDDSMgRMHHQGuQ7aR-o>
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, 17 Dec 2018 12:57:56 -0000
Maybe we need to bring back my proposed text to actually look that the PN of the packet containing the ACK frame is the highest received so far, else do not perform ECN verification. It requires a single state variable, and likely one per packet status boolean (wasReordered) to use during processing. This I think got removed for more shorter and "elegant" text. That way all these reording things goes away. Without the reordering the ECN counters will never go backwards, even if an ACK, only ACKs what was an gap in what been acked so far. -- 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/2198#issuecomment-447836873
- [quicwg/base-drafts] More ECN verification failur… Martin Thomson
- Re: [quicwg/base-drafts] More ECN verification fa… Magnus Westerlund
- Re: [quicwg/base-drafts] More ECN verification fa… Magnus Westerlund
- Re: [quicwg/base-drafts] More ECN verification fa… Magnus Westerlund
- Re: [quicwg/base-drafts] More ECN verification fa… ianswett
- Re: [quicwg/base-drafts] More ECN verification fa… Martin Thomson