Re: [auth48] [C350] AUTH48: RFC-to-be 9332 <draft-ietf-tsvwg-aqm-dualq-coupled-25> for your review
rfc-editor@rfc-editor.org Thu, 24 November 2022 06:19 UTC
Return-Path: <wwwrun@rfcpa.amsl.com>
X-Original-To: auth48archive@ietfa.amsl.com
Delivered-To: auth48archive@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 3AF9FC14F722; Wed, 23 Nov 2022 22:19:06 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.946
X-Spam-Level:
X-Spam-Status: No, score=-3.946 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, CTE_8BIT_MISMATCH=0.001, HEADER_FROM_DIFFERENT_DOMAINS=0.25, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, URIBL_DBL_BLOCKED_OPENDNS=0.001, URIBL_ZEN_BLOCKED_OPENDNS=0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([50.223.129.194]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id c_pWrEioJbpS; Wed, 23 Nov 2022 22:19:02 -0800 (PST)
Received: from rfcpa.amsl.com (rfc-editor.org [50.223.129.200]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 0BE89C14F74F; Wed, 23 Nov 2022 22:19:02 -0800 (PST)
Received: by rfcpa.amsl.com (Postfix, from userid 499) id E40D788A7E; Wed, 23 Nov 2022 22:19:01 -0800 (PST)
To: koen.de_schepper@nokia.com, ietf@bobbriscoe.net, g.white@cablelabs.com
From: rfc-editor@rfc-editor.org
Cc: rfc-editor@rfc-editor.org, tsvwg-ads@ietf.org, tsvwg-chairs@ietf.org, wes@mti-systems.com, auth48archive@rfc-editor.org
Content-type: text/plain; charset="UTF-8"
Message-Id: <20221124061901.E40D788A7E@rfcpa.amsl.com>
Date: Wed, 23 Nov 2022 22:19:01 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/auth48archive/B4xYuIkaRnhCeOyIimgpZMA1MIw>
Subject: Re: [auth48] [C350] AUTH48: RFC-to-be 9332 <draft-ietf-tsvwg-aqm-dualq-coupled-25> for your review
X-BeenThere: auth48archive@rfc-editor.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: "Archiving AUTH48 exchanges between the RFC Production Center, the authors, and other related parties" <auth48archive.rfc-editor.org>
List-Unsubscribe: <https://mailman.rfc-editor.org/mailman/options/auth48archive>, <mailto:auth48archive-request@rfc-editor.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/auth48archive/>
List-Post: <mailto:auth48archive@rfc-editor.org>
List-Help: <mailto:auth48archive-request@rfc-editor.org?subject=help>
List-Subscribe: <https://mailman.rfc-editor.org/mailman/listinfo/auth48archive>, <mailto:auth48archive-request@rfc-editor.org?subject=subscribe>
X-List-Received-Date: Thu, 24 Nov 2022 06:19:06 -0000
Authors, While reviewing this document during AUTH48, please resolve (as necessary) the following questions, which are also in the XML file. 1) <!--[rfced] FYI, we have updated this document with the changes shown in the -26 file sent by the authors. The diff from the approved I-D is here: https://www.rfc-editor.org/in-notes/authors/draft-ietf-tsvwg-aqm-dualq-coupled-25-26-rfcdiff.html The primary change was the removal of informative reference [DCttH19]. --> 2) <!--[rfced] Please note that the title has been updated as follows. Also, should "Algorithms" be included for clarity? Please review. Original: DualQ Coupled AQMs for Low Latency, Low Loss and Scalable Throughput (L4S) Current: Dual Queue Coupled Active Queue Management (AQM) for Low Latency, Low Loss, and Scalable Throughput (L4S) Perhaps: Dual Queue Coupled Active Queue Management (AQM) Algorithms for Low Latency, Low Loss, and Scalable Throughput (L4S) --> 3) <!-- [rfced] Some author comments are present in the XML. Please confirm that no updates related to these comments are outstanding. Note that the comments will be deleted prior to publication. --> 4) <!-- [rfced] Please insert any keywords (beyond those that appear in the title) for use on https://www.rfc-editor.org/search. --> 5) <!--[rfced] Regarding the expansion of L4S upon first mention in the abstract, would you like to update it to match the title? Or, add the expansion upon second mention? Current: These are designed for consistently very Low queuing Latency, very Low congestion Loss, and Scaling of per-flow throughput (L4S) ... Perhaps: These are designed for consistently very Low Latency, Low Loss, and Scalable Throughput (L4S) ... --> 6) <!--[rfced] Is the intention to include both "many" and "(most?)" in the following, or should one be deleted? (A similar question was asked regarding RFC-to-be 9331.) Original: Latency is becoming the critical performance factor for many (most?) Internet applications, e.g. interactive Web, Web services, voice... --> 7) <!-- [rfced] We note the use of <em> in some places. In the txt output, the text enclosed in <em> appears with an underscore before and after. In the html and pdf outputs, the text enclosed in <em> is output in italics. Please review carefully and let us know if the output is acceptable or if any updates are desired. --> 8) <!--[rfced] We have updated the Terminology section to match that of RFC-to-be 9331. Note that the terms are not in the same order. Please review the following diff file (created using a reordered list from this document). This file compares definitions in 9331 vs. 9332: https://www.rfc-editor.org/authors/rfc9331_vs_9332_term_diff.html Differences that remain (please let us know any updates): - This document includes the abbreviations (C and L); 9331 does not. - For Scalable Congestion Control, the last sentence is different. - This document does not reference [CTCP]. - This document has an additional sentence ("The DualQ Coupled AQM behaviour is ...") - This document does not include "Site" in the terminology section, although it mentions 9330's definition in Section 1.2. --> 9) <!--[rfced] May we add single quotes on the first term, as the text is referring to the term itself? (This appears in all 3 documents.) Current: Reno-friendly is used here in place of 'TCP-friendly' Suggested: 'Reno-friendly' is used here in place of 'TCP-friendly' --> 10) <!--[rfced] For the ease of the reader, would you like to include the section number (perhaps Section 6.1) of the L4S architecture? Original: Subjective testing using very demanding high bandwidth low latency applications over a single shared access link is also described in [L4Sdemo16] and summarized in the section about applications in the L4S architecture [I-D.ietf-tsvwg-l4s-arch]. Perhaps: Subjective testing using very demanding high-bandwidth low-latency applications over a single shared access link is also described in [L4Sdemo16] and summarized in Section 6.1 of the L4S architecture [RFC9330]. --> 11) <!--[rfced] Please review each artwork element in the XML file. Specifically, should any artwork element be tagged as sourcecode or another element? --> 12) <!--[rfced] We could not locate "MUST NOT alter these non-ECN identifiers" in draft-ietf-tsvwg-ecn-l4s-id (or RFC-to-be 9331). Please let us know how this quote may be updated for accuracy. Original: [I-D.ietf-tsvwg-ecn-l4s-id] says it "MUST NOT alter these non-ECN identifiers". Section 5.4.1.1 of RFC-to-be 9331 (https://www.rfc-editor.org/authors/rfc9331.html#section-5.4.1.1) contains: In order to include non-L4S packets in the L queue, a network node MUST NOT alter Not-ECT or ECT(0) in the IP-ECN field of an L4S identifier. (Also, a question was sent regarding the change from "to" to "of".) --> 13) <!--[rfced] Section 2.4. For clarity and consistency, may we remove the punctuation from equations (2), (3), and (4)? Original: p_C = (p')^2. (2) p_CL = k*p'. (3) p_L = max(p'_L, p_CL), (4) Perhaps: p_C = (p')^2 (2) p_CL = k*p' (3) p_L = max(p'_L, p_CL) (4) --> 14) <!--[rfced] RFC 2309 has been obsoleted by RFC 7576. May we indicate this relationship by adding the following note? Original: Curvy RED is derived from RED [RFC2309], except its configuration parameters are delay-based to make them insensitive to link rate and it requires fewer operations per packet than RED. Perhaps: Curvy RED is derived from RED [RFC2309] (which was obsoleted by [RFC7576]), except its configuration parameters are delay-based to make them insensitive to link rate, and it requires fewer operations per packet than RED. (and adding RFC 7576 as a informative reference) --> 15) <!--[rfced] A newer version of this document is available. We assume you would like to keep this as is (January 2019) in order to match the reference in RFC-to-be 9330 (rather than update to Version I24 (October 2022)). Original: [DOCSIS3.1] CableLabs, "MAC and Upper Layer Protocols Interface (MULPI) Specification, CM-SP-MULPIv3.1", Data-Over-Cable Service Interface Specifications DOCSIS® 3.1 Version i17 or later, 21 January 2019, <https://specification- search.cablelabs.com/CM-SP-MULPIv3.1>. --> 16) <!-- [rfced] FYI, we have updated these references (as in RFC-to-be 9331) as follows per the guidance in the "Web Portion of the Style Guide" at https://www.rfc-editor.org/styleguide/part2/#ref_repo (see section titled "Referencing Web-Based Public Code Repositories (e.g., GitHub)". We did not make any changes to the titles chosen, but please note that the "Web Portion of the Style Guide" states that "some judgement may be required on the part of the editors and authors to have a sensible title". Please review the titles as well as other changes in the updated reference entries to confirm accuracy. Original: [BBRv2] Cardwell, N., "BRTCP BBR v2 Alpha/Preview Release", GitHub repository; Linux congestion control module, <https://github.com/google/bbr/blob/v2alpha/README.md>. [SCReAM] Johansson, I., "SCReAM", GitHub repository; , <https://github.com/EricssonResearch/scream/blob/master/ README.md>. [Heist21] Heist, P. and J. Morton, "L4S Tests", GitHub README, August 2021, <https://github.com/heistp/l4s- tests/#underutilization-with-bursty-traffic>. Updated: [BBRv2] "TCP BBR v2 Alpha/Preview Release", commit 17700ca, June 2022, <https://github.com/google/bbr>. [SCReAM] "SCReAM", commit fda6c53, June 2022, <https://github.com/EricssonResearch/scream>. [Heist21] "L4S Tests", commit e21cd91, August 2021, <https://github.com/heistp/l4s-tests>. (with mention of "Underutilization with Bursty Links" added in text, similar to RFC-to-be 9331) --> 17) <!--[rfced] FYI: We have updated "<em>P</em>roportional” and "<em>I</em>ntegral" to be "<em>Proportional</em>” and "<em>Integral</em> for readability. Please let us know if you prefer otherwise. Text output: Original: _P_roportional and _I_ntegral Current: _Proportional_ and _Integral_ --> 18) <!--[rfced] May this be rephrased as follows or otherwise? This is to avoid "unrepresentatively", which reads oddly and does not appear in Merriam-Webster or a few other dictionaries. (The derivative "unrepresentativeness" appears, but not "unrepresentatively".) Original: Countries were ranked by number of Internet users, and once 90% of Internet users were covered, smaller countries were excluded to avoid unrepresentatively small sample sizes. Perhaps: Countries were ranked by number of Internet users, and once 90% of Internet users were covered, smaller countries were excluded to avoid small sample sizes that are not representative. Or simply: Countries were ranked by number of Internet users, and once 90% of Internet users were covered, smaller countries were excluded to avoid small sample sizes. --> 19) <!--[rfced] Would it be correct to insert space between "25" and "ms" for consistency, or should it be left as is? Also, should Tupdate=16 ms have spacing around the equals sign (2 instances)? Original: target = RTT_typ * g * f = 25ms * 0.38 * 2 = 19 ms Tupdate=16 ms Perhaps: target = RTT_typ * g * f = 25 ms * 0.38 * 2 = 19 ms Tupdate = 16 ms --> 20) <!--[rfced] To yield superscript, <sup> has been added. Please let us know if you prefer otherwise, as we note that you may want the lines to match the usage of "^" within artwork elements (in Figure 2, for example). This affects the HTML and PDF files. Also, please consider whether <sub> should be used anywhere for subscript. Example from Figure 2: alpha = 0.1 * Tupdate / RTT_max^2 vs. When using <sup>, it appears with a superscript "2" in the following paragraph in the HTML and PDF files. One way to proceed would be to use <sup> only when the base is a numeral (e.g., 2^32 and 2^2). --> 21) <!--[rfced] Should "five" be updated to "three" in the following lead-in text since there are only three main functions listed? Original: The pseudocode manipulates three main structures of variables: the packet (pkt), the L4S queue (lq) and the Classic queue (cq) and consists of the following five functions: * The initialization function cred_params_init(...) (Figure 2) that sets parameter defaults (the API for setting non-default values is omitted for brevity); * The dequeue function cred_dequeue(lq, cq, pkt) (Figure 4); * The scheduling function scheduler(), which selects between the head packets of the two queues. It also uses the following functions that are either shown elsewhere, or not shown in full here: * The enqueue function, which is identical to that used for DualPI2, dualpi2_enqueue(lq, cq, pkt) in Figure 3; * mark(pkt) and drop(pkt) for ECN-marking and dropping a packet; * cq.byt() or lq.byt() returns the current length (aka. backlog) of the relevant queue in bytes; * cq.time() or lq.time() returns the current queuing delay of the relevant queue in units of time (see Note a in Appendix A.1). --> 22) <!--[rfced] FYI, for the XML that creates the links to Notes 1-3 and Notes a-c (in the HTML and PDF files), rather than rely on the dynamic counter value, we decided to hard-code it so that "Note x" (rather than only the single character) is the linked text. If you prefer otherwise, please let us know. For example: Original XML: Note <xref format="counter" target="dualq_note_dequeue"/> Current XML: <xref format="none" target="dualq_note_dequeue">Note 1</xref> --> 23) <!--[rfced] Is "equation (1)" in Section 2.1, or should be readers be directed to "(1)" within Appendix C.2? Please let us know if A or B is correct. Original: The coupling factor, k, determines the balance between L4S and Classic flow rates (see Section 2.5.2.1 and equation (1)). Perhaps: A) The coupling factor, k, determines the balance between L4S and Classic flow rates (see Section 2.5.2.1 and equation (1) below). or B) The coupling factor, k, determines the balance between L4S and Classic flow rates (see Section 2.5.2.1 and equation (1) in Section 2.1). --> 24) <!-- [rfced] Throughout the text, the following terminology appears to be used inconsistently. Please review these occurrences and let us know if/how they may be made consistent. DualQ AQM vs. DualQ Coupled AQM [Note: Are these terms the same? If so, may we update these 2 instances to include "Coupled": - the goal of a DualQ AQM is to behave no worse than a single-queue AQM. - Both FQ and DualQ AQMs can be enablers... --> 25) <!-- [rfced] Please review the "Inclusive Language" portion of the online Style Guide <https://www.rfc-editor.org/styleguide/part2/#inclusive_language> and let us know if any changes are needed. For example, please consider whether "native" should be updated. In addition, please consider whether "traditionally" should be updated for clarity. While the NIST website <https://www.nist.gov/nist-research-library/nist-technical-series-publications-author-instructions#table1> indicates that this term is potentially biased, it is also ambiguous. As noted there, "traditional" is a subjective term. --> Thank you. RFC Editor/kc/ar On Nov 23, 2022, at 10:17 PM, rfc-editor@rfc-editor.org wrote: *****IMPORTANT***** Updated 2022/11/23 RFC Author(s): -------------- Instructions for Completing AUTH48 Your document has now entered AUTH48. Once it has been reviewed and approved by you and all coauthors, it will be published as an RFC. If an author is no longer available, there are several remedies available as listed in the FAQ (https://www.rfc-editor.org/faq/). You and you coauthors are responsible for engaging other parties (e.g., Contributors or Working Group) as necessary before providing your approval. Planning your review --------------------- Please review the following aspects of your document: * RFC Editor questions Please review and resolve any questions raised by the RFC Editor that have been included in the XML file as comments marked as follows: <!-- [rfced] ... --> These questions will also be sent in a subsequent email. * Changes submitted by coauthors Please ensure that you review any changes submitted by your coauthors. We assume that if you do not speak up that you agree to changes submitted by your coauthors. * Content Please review the full content of the document, as this cannot change once the RFC is published. Please pay particular attention to: - IANA considerations updates (if applicable) - contact information - references * Copyright notices and legends Please review the copyright notice and legends as defined in RFC 5378 and the Trust Legal Provisions (TLP – https://trustee.ietf.org/license-info/). * Semantic markup Please review the markup in the XML file to ensure that elements of content are correctly tagged. For example, ensure that <sourcecode> and <artwork> are set correctly. See details at <https://authors.ietf.org/rfcxml-vocabulary>. * Formatted output Please review the PDF, HTML, and TXT files to ensure that the formatted output, as generated from the markup in the XML file, is reasonable. Please note that the TXT will have formatting limitations compared to the PDF and HTML. Submitting changes ------------------ To submit changes, please reply to this email using ‘REPLY ALL’ as all the parties CCed on this message need to see your changes. The parties include: * your coauthors * rfc-editor@rfc-editor.org (the RPC team) * other document participants, depending on the stream (e.g., IETF Stream participants are your working group chairs, the responsible ADs, and the document shepherd). * auth48archive@rfc-editor.org, which is a new archival mailing list to preserve AUTH48 conversations; it is not an active discussion list: * More info: https://mailarchive.ietf.org/arch/msg/ietf-announce/yb6lpIGh-4Q9l2USxIAe6P8O4Zc * The archive itself: https://mailarchive.ietf.org/arch/browse/auth48archive/ * Note: If only absolutely necessary, you may temporarily opt out of the archiving of messages (e.g., to discuss a sensitive matter). If needed, please add a note at the top of the message that you have dropped the address. When the discussion is concluded, auth48archive@rfc-editor.org will be re-added to the CC list and its addition will be noted at the top of the message. You may submit your changes in one of two ways: An update to the provided XML file — OR — An explicit list of changes in this format Section # (or indicate Global) OLD: old text NEW: new text You do not need to reply with both an updated XML file and an explicit list of changes, as either form is sufficient. We will ask a stream manager to review and approve any changes that seem beyond editorial in nature, e.g., addition of new text, deletion of text, and technical changes. Information about stream managers can be found in the FAQ. Editorial changes do not require approval from a stream manager. Approving for publication -------------------------- To approve your RFC for publication, please reply to this email stating that you approve this RFC for publication. Please use ‘REPLY ALL’, as all the parties CCed on this message need to see your approval. Files ----- The files are available here: https://www.rfc-editor.org/authors/rfc9332.xml https://www.rfc-editor.org/authors/rfc9332.html https://www.rfc-editor.org/authors/rfc9332.pdf https://www.rfc-editor.org/authors/rfc9332.txt Diff file of the text: https://www.rfc-editor.org/authors/rfc9332-diff.html https://www.rfc-editor.org/authors/rfc9332-rfcdiff.html (side by side) Diff of the XML: https://www.rfc-editor.org/authors/rfc9332-xmldiff1.html Tracking progress ----------------- The details of the AUTH48 status of your document are here: https://www.rfc-editor.org/auth48/rfc9332 Please let us know if you have any questions. Thank you for your cooperation, RFC Editor -------------------------------------- RFC9332 (draft-ietf-tsvwg-aqm-dualq-coupled-25) Title : Dual Queue Coupled Active Queue Management (AQM) for Low Latency, Low Loss, and Scalable Throughput (L4S) Author(s) : K. De Schepper, B. Briscoe, Ed., G. White WG Chair(s) : Gorry Fairhurst, David L. Black, Marten Seemann Area Director(s) : Martin Duke, Zaheduzzaman Sarker
- [auth48] [C350] AUTH48: RFC-to-be 9332 <draft-iet… rfc-editor
- Re: [auth48] [C350] AUTH48: RFC-to-be 9332 <draft… rfc-editor
- Re: [auth48] [C350] AUTH48: RFC-to-be 9332 <draft… Bob Briscoe
- Re: [auth48] [C350] AUTH48: RFC-to-be 9332 <draft… Karen Moore
- Re: [auth48] [C350] AUTH48: RFC-to-be 9332 <draft… Bob Briscoe