Re: [PATCH net-next 2/2] fq_codel: implement L4S style ce_threshold_ect1 marking

Jonathan Morton <chromatix99@gmail.com> Sun, 17 October 2021 12:18 UTC

Return-Path: <chromatix99@gmail.com>
X-Original-To: wgchairs@ietfa.amsl.com
Delivered-To: wgchairs@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 5CA963A0D6A for <wgchairs@ietfa.amsl.com>; Sun, 17 Oct 2021 05:18:40 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.848
X-Spam-Level:
X-Spam-Status: No, score=-1.848 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_ENVFROM_END_DIGIT=0.25, FREEMAIL_FROM=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.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 aoSiezWTqxlQ for <wgchairs@ietfa.amsl.com>; Sun, 17 Oct 2021 05:18:35 -0700 (PDT)
Received: from mail-lj1-x229.google.com (mail-lj1-x229.google.com [IPv6:2a00:1450:4864:20::229]) (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 886F63A0D69 for <wgchairs@ietf.org>; Sun, 17 Oct 2021 05:18:35 -0700 (PDT)
Received: by mail-lj1-x229.google.com with SMTP id d13so3438735ljg.0 for <wgchairs@ietf.org>; Sun, 17 Oct 2021 05:18:35 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; bh=Ams7n3RdF+fXSvGvFS6HBjzoePKPkXT/5ewtgnYm2/4=; b=dcHvLZuKFlIjp70b00a0Ctnjao9EvhnRf6FeqkEmTAZtQbiHNXkmmQswhADvOawrhJ b7oMrbrEOGlv10WUVBW55YXfJwX8bpSYWMBXtEB9nCLRixrmkwg8h523fyElkhvD1ULb XewgrxWCQMTiQG20NnMQzArMT8z0HEZA89LWU2nq3l4WLBwmds6YkqJeNjvw8TQUghte 0RjbpLocObwI6MS41Lc8po1asnMQF4sWQcTU8PpGI3QSYWJZdEiMj60RciOy2YDeWoZi 74XqxwLOKdZbbTHNJLSDHu9G3Veunk6KCHqywY7zS/dzeSOPI9s16ZF/4T8gAzJEssgq 2SNQ==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; bh=Ams7n3RdF+fXSvGvFS6HBjzoePKPkXT/5ewtgnYm2/4=; b=Zjuhq04dfRpF4hMPFfWGDSpcnDksVdDTZk3M3oOoGdshzxkwip5u3VhLfjUAxQmLrd /43R0fFi2kH8/QArzBQoTTwxgHA4MpQ7yljBwUd+0a6A1yHazdYDyP/LvkobJVSZODsq jnBWGQOP8f/binIkig5uLcwlva1KvdgHqJQSWURRscnbN3g/95sBQV3QCKl9HsgfkYkL dmMtXqG/n8dvei6Cr1WbO9mLqWeVmcHMbvkMZIZQIqg+kE+uPuLu5RKez0XVaRNMDWur +nb11NtszWia6Dyp4MejUdzfSO+Qu/unD2xpq3jD8CyX5jPqXrqaMeojcG83QMsVgr3z OZXg==
X-Gm-Message-State: AOAM531Ien8kLGw07sOoWAq83zM8wOZInhUtQlQ6B6DsMiXwjmyX2BgX WSu16T50peA4ZtT3fVEgveXThnPSi9k5tumf
X-Google-Smtp-Source: ABdhPJwsnuuOEJ/BnqWwfYpITkwIdH+9GRI0WwKNk0HHc0Fd7a5QPoWxdwhG4kNjRlkDXztBtVIUKA==
X-Received: by 2002:a05:651c:1504:: with SMTP id e4mr25389127ljf.131.1634473113534; Sun, 17 Oct 2021 05:18:33 -0700 (PDT)
Received: from smtpclient.apple (176-93-88-52.bb.dnainternet.fi. [176.93.88.52]) by smtp.gmail.com with ESMTPSA id u16sm1226356lfr.260.2021.10.17.05.18.32 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Sun, 17 Oct 2021 05:18:33 -0700 (PDT)
Content-Type: text/plain; charset="us-ascii"
Mime-Version: 1.0 (Mac OS X Mail 14.0 \(3654.100.0.2.22\))
Subject: Re: [PATCH net-next 2/2] fq_codel: implement L4S style ce_threshold_ect1 marking
From: Jonathan Morton <chromatix99@gmail.com>
In-Reply-To: <9ad3a249-1950-c665-5996-e15352867924@bobbriscoe.net>
Date: Sun, 17 Oct 2021 15:18:30 +0300
Cc: Eric Dumazet <edumazet@google.com>, Eric Dumazet <eric.dumazet@gmail.com>, "David S . Miller" <davem@davemloft.net>, Jakub Kicinski <kuba@kernel.org>, netdev <netdev@vger.kernel.org>, Neal Cardwell <ncardwell@google.com>, Ingemar Johansson S <ingemar.s.johansson@ericsson.com>, Tom Henderson <tomh@tomh.org>, Toke Høiland-Jørgensen <toke@redhat.com>
Content-Transfer-Encoding: quoted-printable
Message-Id: <A2749FBB-6963-4043-9A1D-E950AF4ACE62@gmail.com>
References: <20211014175918.60188-1-eric.dumazet@gmail.com> <20211014175918.60188-3-eric.dumazet@gmail.com> <87wnmf1ixc.fsf@toke.dk> <CANn89iLbJL2Jzot5fy7m07xDhP_iCf8ro8SBzXx1hd0EYVvHcA@mail.gmail.com> <87mtnb196m.fsf@toke.dk> <308C88C6-D465-4D50-8038-416119A3535C@gmail.com> <9ad3a249-1950-c665-5996-e15352867924@bobbriscoe.net>
To: Bob Briscoe <ietf@bobbriscoe.net>
X-Mailer: Apple Mail (2.3654.100.0.2.22)
Archived-At: <https://mailarchive.ietf.org/arch/msg/wgchairs/ZG2kbq2b88ghN28Y4BbNVYAuTUM>
X-Mailman-Approved-At: Mon, 18 Oct 2021 08:43:21 -0700
X-BeenThere: wgchairs@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Working Group Chairs <wgchairs.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/wgchairs>, <mailto:wgchairs-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/wgchairs/>
List-Post: <mailto:wgchairs@ietf.org>
List-Help: <mailto:wgchairs-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/wgchairs>, <mailto:wgchairs-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 17 Oct 2021 12:18:41 -0000

> On 17 Oct, 2021, at 2:22 pm, Bob Briscoe <ietf@bobbriscoe.net> wrote:
> 
>> I'll be blunter:
>> 
>> In its original (and currently stable) form, fq_codel is RFC-compliant.  It conforms, in particular, to RFC-3168 (ECN).  There's a relatively low threshold for adding RFC-compliant network algorithms to Linux, and it is certainly not required to have a published RFC specifically describing each qdisc's operating principles before it can be upstreamed.  It just so happens that fq_codel (and some other notable algorithms such as CUBIC) proved sufficiently useful in practice to warrant post-hoc documentation in RFC form.
>> 
>> However, this patch adds an option which, when enabled, makes fq_codel *non-compliant* with RFC-3168, specifically the requirement to treat ECT(0) and ECT(1) identically, unless conforming to another published RFC which permits different behaviour.
>> 
>> There is a path via RFC-8311 to experiment with alternative ECN semantics in this way, but the way ECT(1) is used by L4S is specifically mentioned as requiring a published RFC for public deployments.  The L4S Internet Drafts have *just failed* an IETF WGLC, which means they are *not* advancing to publication as RFCs in their current form.
> 
> [BB] Clarification of IETF process: A first Working Group Last Call (WGLC) is nearly always the beginning of the end of the IETF's RFC publication process. Usually the majority of detailed comments arrive during a WGLC. Then the draft has to be fixed, and then it goes either directly through to the next stage (in this case, an IETF-wide last call), or to another WGLC.

Further clarification: this is already the second WGLC for L4S.  The one two years previously (at Montreal) yielded a number of major technical objections, which remained unresolved as of this latest WGLC.

>> The primary reason for this failure is L4S' fundamental incompatibility with existing Internet traffic, despite its stated goal of general Internet deployment.
> 
> [BB] s/The primary reason /JM's primary objection /
> There is no ranking of the reasons for more work being needed.  The WG had already developed a way to mitigate this objection. Otherwise, a WGLC would not have been started in the first place. Further work on this issue is now more likely to be wordsmithing.

Given that the objections cited by the TSVWG Chairs were technical in nature, and related specifically to the incompatibility between L4S and existing conventional traffic, it is clear to me that wordsmithing will *not* be sufficient to render L4S publishable in RFC form, nor deployable at Internet scale.  

To quote David Black, one of the aforementioned Chairs and also an author of RFC-8311:

> Two overall conclusions are that a) the WGLC has been productive, and shows significant continuing support for L4S, and b) the L4S drafts should be revised to address the WGLC concerns raised.   The WG chairs strongly suggest that the revisions include limiting the scope and impact of initial L4S experiments on RFC 3168 functionality (both existing usage and potential deployment) to ensure that the L4S experiments are safe to perform on the Internet, paying particular attention to potential impacts on networks and users that are not participating in the L4S experiments.

It is my recommendation to netdev to stay out of this ongoing mess, by rejecting this patch.

 - Jonathan Morton