Re: [auth48] AD - Re: AUTH48: RFC-to-be 9435 <draft-ietf-tsvwg-dscp-considerations-13> for your review (round 2)
Gorry Fairhurst <gorry@erg.abdn.ac.uk> Tue, 11 July 2023 15:51 UTC
Return-Path: <gorry@erg.abdn.ac.uk>
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 C26A4C151981; Tue, 11 Jul 2023 08:51:45 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.897
X-Spam-Level:
X-Spam-Status: No, score=-1.897 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, NICE_REPLY_A=-0.001, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=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 7cpc_S2Q55GS; Tue, 11 Jul 2023 08:51:41 -0700 (PDT)
Received: from pegasus.erg.abdn.ac.uk (pegasus.erg.abdn.ac.uk [137.50.19.135]) by ietfa.amsl.com (Postfix) with ESMTP id EA9FCC151068; Tue, 11 Jul 2023 08:51:38 -0700 (PDT)
Received: from [137.50.187.24] (oa-edu-187-24.wireless.abdn.ac.uk [137.50.187.24]) by pegasus.erg.abdn.ac.uk (Postfix) with ESMTPSA id 653C61B0010F; Tue, 11 Jul 2023 16:51:26 +0100 (BST)
Message-ID: <8a5fc880-06c0-96f9-89c7-28d584e90ab8@erg.abdn.ac.uk>
Date: Tue, 11 Jul 2023 16:51:26 +0100
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.13.0
Content-Language: en-US
To: Martin Duke <martin.h.duke@gmail.com>
Cc: Alice Russo <arusso@amsl.com>, tsvwg-ads@ietf.org, tsvwg-chairs@ietf.org, david.black@dell.com, "ana C. Custura" <ana@netstat.org.uk>, "Secchi, Raffaello" <r.secchi@abdn.ac.uk>, rfc-editor@rfc-editor.org, auth48archive@rfc-editor.org
References: <1fae0d26-2e9b-3ec7-d779-c52c895ed039@erg.abdn.ac.uk> <bed0297b-2c91-c58e-4f0b-eef4d28bafc4@erg.abdn.ac.uk> <09D2C1F5-BDB9-4543-B0B0-730EC5DACD50@amsl.com> <20b6e7d6-da48-0c01-229c-d472ea67dbf1@erg.abdn.ac.uk> <CAM4esxQtGtca9e+jYfoMjRvcxKBxf2XS58yFk_zyO4FdY9Ex3g@mail.gmail.com>
From: Gorry Fairhurst <gorry@erg.abdn.ac.uk>
In-Reply-To: <CAM4esxQtGtca9e+jYfoMjRvcxKBxf2XS58yFk_zyO4FdY9Ex3g@mail.gmail.com>
Content-Type: text/plain; charset="UTF-8"; format="flowed"
Content-Transfer-Encoding: 8bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/auth48archive/wEICbNxn8_HJErMMUErmSrXzGYA>
Subject: Re: [auth48] AD - Re: AUTH48: RFC-to-be 9435 <draft-ietf-tsvwg-dscp-considerations-13> for your review (round 2)
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: Tue, 11 Jul 2023 15:51:45 -0000
On 11/07/2023 16:16, Martin Duke wrote: > (S4.2.1) > You changed "greater than 48 decimal (0x30)" to "greater than 48 > decimals (0x30)" > > This means "48 in decimal notation" so 48 decimals is definitely > wrong. I understood the original perfectly well, but if there must be > a change here it would be better to say "48 in decimal notation > (0x30)" or simply "(0x30)" > > (S5.4) > OLD: Metro Ethernet Forum (MEF) > NEW: MEF Forum (MEF) > ????!!!???? > I agree to the above, please add this to the requested changes. Gorry > > > > On Tue, Jul 11, 2023 at 1:54 AM Gorry Fairhurst <gorry@erg.abdn.ac.uk> > wrote: > > Hello Alice! > > Please see below: > > On 07/07/2023 00:35, Alice Russo wrote: >> Gorry and Martin (as AD), >> >> Martin, >> Please review and let us know if you approve the changes listed >> below. They are also visible in this diff file: >> https://www.rfc-editor.org/authors/rfc9435-auth48diff.html >> >> - Section 3.1: text added re: Best Effort and CS0. >> - Sections 6.2, 6.2.1, and 6.3: one sentence added at start of each. >> - Figures 2, 3, and 5: new titles. >> >> The last 2 are per "cross-area INT review": >> - Section 4: added text defining unknown DSCP and unexpected DSCP. >> - Section 5.1.1: sentence deleted. ("This aligned with the now >> deprecated use of CS1 as the codepoint for the Lower Effort >> service, as previously specified in [RFC4594].") >> -- >> >> Gorry, >> Thank you for your reply. Please see the follow-ups below. >> The revised files are here (please refresh): >> https://www.rfc-editor.org/authors/rfc9435.html >> https://www.rfc-editor.org/authors/rfc9435.txt >> https://www.rfc-editor.org/authors/rfc9435.pdf >> https://www.rfc-editor.org/authors/rfc9435.xml >> >> This diff file shows all changes from the approved I-D: >> https://www.rfc-editor.org/authors/rfc9435-diff.html >> https://www.rfc-editor.org/authors/rfc9435-rfcdiff.html (side by >> side) >> >> This diff file shows the changes made during AUTH48 thus far: >> https://www.rfc-editor.org/authors/rfc9435-auth48diff.html >> >> Re: #6 ("being re-marked to 'x' & 0x07 and then forwarded") >>> GF: I'd prefer to NOT change. The ampersand here represents a >>> "bit-wise and". If this does need to be clarified please propose >>> something. >> >> Thanks for clarifying. In RFCs, we see it written out as "bitwise >> AND". Below are examples of how it has been defined in some >> recentish RFCs. Perhaps add ("&" represents the bitwise >> AND operator) after the sentence? >> >> *rfc6225.txt**:** "&" is a **bitwise AND* >> *rfc7188.txt**:** where "bitand" denotes a **bitwise AND* >> *rfc8439.txt**:** Where "&=" is the C language **bitwise >> AND**assignment operator.* >> *rfc8554.txt**:** AND : a AND b denotes the **bitwise >> AND**of the two nonnegative* >> *rfc8881.txt**:** ("&" represents the **bitwise AND**operator)* >> *rfc9189.txt**:** i & j **bitwise AND**of unsigned integers i >> and j.* >> > * Re: #6 ("being re-marked to 'x' & 0x07 and then forwarded") > > The proposed resolution is to add a definition to the end of > section 2 to define'&': > NEW: > In this document, the symbol "&" denotes a bitwise AND of two > unsigned values. > > --- >> >> Re: #11, we updated the figure titles as requested, but the >> question was actually about tables. > * We suggest the best way to address the question regarding the > table headings is to add text below the first use of the table, as > below: > OLD: > The DSCP space is shown in the following table. > NEW: > The DSCP space is shown in the following table. Each row > corresponds to one setting of the the first three bits of the DSCP > field and each column to one setting of the the last three bits of > the DSCP field. > > --- >> Specifically, should there be any column or row labels added to >> Table 2 or 3? >> >> Re: >>> We could add a title row above the bit pattern with the bit >>> number within the DS field: >>> >>> ADD NEW: >>> +-+-+-+-+-+-+ >>> |7|6|5|4|3|2|1|0| >>> +-+-+-+-+-+-+ >>> | ... >>> +-+-+-+-+-+-+ >> >> We added this bit ruler, but it's longer than the bits provided; >> please tell us how Figures 2, 3, and 5 should be updated. >> > * Sorry!!! The following were a result of requested changes, but > were incorrect, can you please update as shown below: > > Figure 2 (This should be a 6b field): > > OLD: > +-+-+-+-+-+-+ > |7|6|5|4|3|2|1|0| > +-+-+-+-+-+-+ > |0 0 0|x x x| > +-+-+-+-+-+-+ > NEW: > +-+-+-+-+-+-+ > |5|4|3|2|1|0| > +-+-+-+-+-+-+ > |0 0 0|x x x| > +-+-+-+-+-+-+ > > > Figure 3 (This should be a 6b field): > OLD: > +-+-+-+-+-+-+ > |7|6|5|4|3|2|1|0| > +-+-+-+-+-+-+ > |0 0 1|x x x| > +-+-+-+-+-+-+ > > NEW: > +-+-+-+-+-+-+ > |5|4|3|2|1|0| > +-+-+-+-+-+-+ > |0 0 0|x x x| > +-+-+-+-+-+-+ > > Figure 4 (This should be consistent with 2,3 as a 6b field): > > OLD: > +-+-+-+-+-+-+ > |x x x|. . .| > +-+-+-+-+-+-+ > NEW: > +-+-+-+-+-+-+ > |5|4|3|2|1|0| > +-+-+-+-+-+-+ > |x x x|. . .| > +-+-+-+-+-+-+ > > Figure 5: > OLD: > +-+-+-+-+-+-+ > |7|6|5|4|3|2|1|0| > +-+-+-+-+-+-+ > |x x x|0 0 0| > +-+-+-+-+-+-+ > NEW: > +-+-+-+-+-+-+ > |5|4|3|2|1|0| > +-+-+-+-+-+-+ > |x x x|0 0 0| > +-+-+-+-+-+-+ >> >> Re: >>> 1. An updated citation for [IEEE.802.11] is: [...] 802.11-2016 >> >> Updated as requested. FYI, the page says it was superseded >> by 802.11-2020. Please let us know if you want to update the >> reference to that. >> > * - Yes please update. >> We will wait to hear from you again and from your coauthors >> before continuing the publication process. This page shows >> the AUTH48 status of your document: >> https://www.rfc-editor.org/auth48/rfc9435 >> >> Thank you. >> RFC Editor/ar > > * After cross-checking your uypdated text with the RFCs and IEE > spec, we would like to consistently use the IEEE usage of the term > AC. We understand that the use of "Access Classes" in RFC8325 is > less correct, and was inconsistent in that RFC, so please update > as follows: > > OLD > The IEEE 802.11 standards [IEEE-802.11] provide Media Access > Control (MAC) functions to support QoS in WLANs using Access > Classes (AC). > NEW: > The IEEE 802.11 standards [IEEE-802.11] provide Media Access > Control (MAC) functions to support QoS in WLANs using Access > Categories (ACs). > --- > * The following requested change was actually ambiguous (sorry) as > to whether all bits were set to 1 or the combined field had a > value of 1. Please change: > OLD: > This considers a proposed DSCP where the least significant 3 bits > have a value of 1. > > NEW: > This considers a proposed DSCP where the least significant 3 bits > together represent a value of 1 (i.e., 0b001). > > --- > > Please could you also correct some typos: > > * Please insert /the/: > OLD: > defined by IETF, > NEW: > defined by the IETF, > --- > * Two use of the word /new/ in the same sentence seems more than > needed in the abstract, the new part is the DSCP assignment: > > OLD: > This document discusses considerations for assigning a new > recommended Differentiated Services Code Point (DSCP) for a new > standard Per-Hop Behavior (PHB). > NEW: > This document discusses considerations for assigning a new > recommended Differentiated Services Code Point (DSCP) for a > standard Per-Hop Behavior (PHB). > > --- > > Thanks for preparing this document. All the editors have now read > the text and we expect to approve the next version after we > confirm these updates. > > Best wishes, > > Gorry > >> >>> On Jul 4, 2023, at 5:31 AM, Gorry Fairhurst >>> <gorry@erg.abdn.ac.uk> wrote: >>> >>> Thank you we have now reviewed your changes, we were quite >>> impressed at the level of attention to detail. >>> >>> Please see our comments and replies marked "GF", and some >>> additional requests at the end. >>> >>> We plan to complete a full review and approve this, once these >>> changes have been implemented. >>> >>> Best wishes, >>> Gorry >>> >>> ==== >>> >>> Authors, >>> >>> While reviewing this document during AUTH48, please resolve (as >>> necessary) the following questions, which are also in the XML file. >>> >>> 1) <!-- [rfced] FYI, the title of the document has been updated >>> as follows. >>> >>> Original: Considerations for Assigning a new Recommended >>> DiffServ Codepoint >>> (DSCP) >>> >>> Current: Considerations for Assigning a New Recommended >>> Differentiated >>> Services Code Point (DSCP) >>> --> >>> >>> GF: Works for me. >>> >>> 2) <!-- [rfced] FYI: Repeated references >>> >>> a) Since "[RFC2474]" was cited twice in this sentence, and no other >>> citation was present, we placed it at the end of the sentence. >>> >>> Original: >>> It provides differentiated traffic forwarding >>> based on the DiffServ Code Point (DSCP) [RFC2474] carried in the >>> DiffServ field [RFC2474] of the IP packet header. >>> >>> Current: >>> It provides differentiated traffic forwarding >>> based on the DSCP carried in the Diffserv field of the IP packet >>> header [RFC2474]. >>> >>> GF: Works for me. >>> >>> b) Similarly, since "[RFC4594]" was cited twice in this >>> sentence, we placed this reference at the end of the sentence. >>> >>> Original: >>> DiffServ associates traffic with a service class [RFC4594] and >>> categorises it into Behavior Aggregates [RFC4594]. >>> Suggested: >>> Diffserv associates traffic with a service class and >>> categorizes it into Behavior Aggregates [RFC4594]. --> >>> >>> GF: Works for me. >>> >>> 3) <!-- [rfced] FYI, we have updated this text to: >>> - cite RFC 2474 because it's the source of the quote, and >>> - change "standards" to "standardized" to match the source, and >>> - include the phrase "if Pool 1 is ever exhausted" because it's >>> in the source. >>> Please let us know if you prefer otherwise. >>> >>> Original: >>> This was initially available for experimental (EXP) or Local Use >>> (LU), but was originally specified to be "preferentially utilized >>> for standards assignments" if Pool 1 is ever exhausted. >>> Current: >>> This was initially available for Experimental (EXP) Use or Local Use >>> (LU), but was originally specified to be "preferentially utilized >>> for standardized assignments if Pool 1 is ever exhausted" >>> [RFC2474]. --> >>> >>> GF: Works for me. >>> >>> 4) <!-- [rfced] Please note that the quoted text does not appear >>> in RFC 8436. Please review and let us know how this may be updated. >>> Perhaps you intended the "NEW" text in Section 3 of RFC 8436 >>> (https://www.rfc-editor.org/rfc/rfc8436#section-3)? >>> >>> Original: >>> Pool 3 >>> codepoints are now "utilized for standards assignments and are no >>> longer available for assignment to experimental or local use" >>> [RFC8436]. >>> Perhaps: >>> Pool 3 >>> codepoints are now "utilized for standardized assignments (replacing >>> the previous availability for experimental or local use)" >>> [RFC8436]. --> >>> >>> GF: Works for me. >>> >>> 5) <!-- [rfced] Regarding Table 2 references: >>> >>> a) Neither "Best Effort" nor "CS0" is mentioned in RFC 2474. How >>> should the second row be updated? >>> >>> GF: Correct - but Class Selector mapping is - this is confusing. >>> A sentence or two here would be better to explain these rather >>> complicated equivalent words. >>> >>> OLD: >>> *BE (Best Effort),* also known as*"CS0",* describes the default forwarding treatment. >>> NEW: >>> The Default PHB is defined in section 4.1 of [RFC2474]. This >>> provides Best Effort (BE) forwarding, and the recommended DSCP >>> of *'000000' *(section 4.2.2.1 of [RFC2474]). This >>> is the lowest value in the set of Class Selector (CS) DSCPs, and >>> hence is also known as "CS0" >>> [https://www.iana.org/assignments/dscp-registry/dscp-registry.xhtml]. >>> >>> GF: I realise this may also move the definition of CS earlier, >>> please check and correct. >>> >>> b) "Voice Admit" nor "VA" appears in RFC 5865. >>> How should it be updated? >>> >>> GF: RFC 5865 uses "VOICE-ADMIT" as the name >>> GF: It seems industry uses VA as an abbreivation, I'd like to >>> propose we keep this (see also below on a caption change). >>> >>> Perhaps it refers to Call Admission Control? >>> GF: No: This is not the same thing. >>> >>> Current: >>> + - + - - - - - - - - - - + - - - - - + >>> | CS | Class Selector | [RFC2474] | >>> + - + - - - - - - - - - - + - - - - - + >>> | BE | Best Effort (CS0) | [RFC2474] | >>> + - + - - - - - - - - - - + - - - - - + >>> | AF | Assured Forwarding | [RFC2597] | >>> + - + - - - - - - - - - - + - - - - - + >>> | EF | Expedited Forwarding | [RFC3246] | >>> + - + - - - - - - - - - - + - - - - - + >>> | VA | Voice Admit | [RFC5865] | >>> + - + - - - - - - - - - - + - - - - - + >>> | LE | Lower Effort | [RFC8622] | >>> + - + - - - - - - - - - - + - - - - - + --> >>> >>> GF: There is no reason to say "Used in Published RFCs", although >>> that was the goal, so I suggest: >>> >>> OLD: >>> Table 2: Summary of the DSCP Abbreviations Used in Published RFCs >>> >>> NEW: >>> >>> Table 2: Abbreviations for DSCPs and PHB Groups >>> >>> >>> >>> 6) <!--[rfced] Regarding use of ampersand in Section 4.2.1, may >>> it be replaced with "and" or is the symbol preferred? >>> >>> Original: >>> ... result in the DSCP being re-marked to 'x' & 0x07 and then >>> forwarded using the PHB specified... >>> >>> Perhaps: >>> ... result in the DSCP being re-marked to 'x' and 0x07 and then >>> forwarded using the PHB specified... >>> --> >>> GF: I'd prefer to NOT change. The ampersand here represents a >>> "bit-wise and". If this does need to be >>> clarified please propose something. >>> >>> 7) <!-- [rfced] What do "Section 5.1.2" and "Section 4.1" refer to - >>> sections within RFC 8325, IEEE 802.11 UP, or this RFC? >>> Original: >>> RFC8325 [RFC8325] notes inconsistencies that can result from >>> such re- >>> marking, and recommends a different mapping to perform this re- >>> marking, dependent on the direction in which a packet is forwarded. >>> It provides recommendations for mapping a DSCP to an IEEE 802.11 UP >>> for interconnection between wired and wireless networks. The >>> recommendation in Section 5.1.2 maps network control traffic, >>> CS6 and >>> CS7, as well as unassigned DSCPs, to UP 0 when forwarding in the >>> upstream direction (wireless-to-wired). It also recommends mapping >>> CS6 and CS7 traffic to UP 7, when forwarding in the downstream >>> direction (Section 4.1). >>> >>> Perhaps as follows? >>> - Section 5.1.2 of this document ("Mapping Specified for IEEE >>> 802.11") >>> - Section 4.1 of RFC 8325 ("Network Control Traffic") >>> --> >>> >>> GF: This works for me. >>> >>> 8) <!--[rfced] Regarding Table 5, should the header be updated >>> as follows or otherwise? The term "Aggregate Class" does not >>> appear in RFC 8100. Should it be defined within this document? >>> >>> Original: >>> | RFC8100 | DSCP | | Agg. Class | | >>> Perhaps: >>> | Treatment Aggregate [RFC8100] | DSCP | >>> >>> GF: This works for me. >>> >>> Similarly, regarding Table 6, should the header be updated as >>> follows or otherwise? We note that GSMA IR.34 lists the four >>> "UMTS QoS classes" in Section 6.2.1. The term "Aggregate Class" >>> does not appear in IR.34. >>> >>> Original: >>> | GSMA IR.34 | PHB | | Agg. Class | | >>> Perhaps: >>> | QoS Class in [GSMA-IR.34] | PHB | >>> --> >>> >>> GF: This works for me. >>> >>> 9) <!-- [rfced] To more clearly indicate the forwarding >>> direction, may we update >>> the hyphens to "to" in the following? >>> >>> Original: >>> A provider service path may consist of sections where multiple and >>> changing layers use their own code points to determine >>> differentiated >>> forwarding (e.g., IP - MPLS - IP - Ethernet - Wi-Fi). >>> >>> Suggested: >>> A provider service path may consist of sections where multiple and >>> changing layers use their own code points to determine >>> differentiated >>> forwarding (e.g., IP to MPLS to IP to Ethernet to Wi-Fi). >>> --> >>> >>> GF: This seems good. >>> >>> 10) <!--[rfced] Will the use of ampersand in the title of >>> Section 6.2.1 be clear to readers? Would it be more clear to >>> write the meaning in prose? >>> >>> Original: >>> 6.2. Where the proposed DSCP > 0x07 (7) >>> 6.2.1. Where the proposed DSCP&0x07=0x01 >>> 6.3. Where the proposed DSCP <= 0x07 (7) >>> --> >>> >>> GF: I'm unsure I want to changge. >>> GF: I do suggest we add one sentence after each subsection >>> heading clarify this: >>> >>> 6.2. Where the proposed DSCP > 0x07 (7) >>> ADD NEW: >>> This considers a proposed DSCP with a codepoint larger than 7. >>> >>> 6.2.1. Where the proposed DSCP&0x07=0x01 >>> ADD NEW: >>> This considers a proposed DSCP where the least significant 3 >>> bits have a value of 1. >>> >>> 6.3. Where the proposed DSCP <= 0x07 (7) >>> ADD NEW: >>> This considers a proposed DSCP where the DSCP is less than or >>> equal to 7. >>> >>> >>> 11) <!-- [rfced] Table Headers >>> >>> a) In Tables 1, 3, and 4, there's an identical line (Tables 1 >>> and 3, last >>> row; Table 4, top row). Should this row be in a consistent location >>> of each table (e.g., the top row)? >>> >>> Identical row: >>> + - - - + - - + - + - + - + - + - + - + >>> | 0/CS0 | 1/LE | 2 | 3 | 4 | 5 | 6 | 7 | >>> + - - - + - - + - + - + - + - + - + - + >>> >>> GF: No: This seems a nice idea, but the row ordering is >>> significant, and this corresponds >>> to the lowest set of numerical values and belongs at the bottom >>> of tables 1,3. >>> >>> b) For Tables 2, 5, and 6, please consider whether any labels >>> should be >>> added to the columns or rows so that it's clearer to the reader >>> what the table contains. >>> --> >>> GF: Please include 3 in this list, it's the same format >>> For Tables 2,3,5 >>> >>> We could add a title row above the bit pattern with the bit >>> number within the DS field: >>> >>> ADD NEW: >>> +-+-+-+-+-+-+ >>> |7|6|5|4|3|2|1|0| >>> +-+-+-+-+-+-+ >>> | ... >>> +-+-+-+-+-+-+ >>> >>> GF: In addition, I think it would be clearer if the legend >>> explained in each case that this was the D >>> >>> OLD: >>> Figure 2: Bleaching of the ToS Precedence >>> NEW: >>> Figure 2: Bits in the Diffserv field modified by bleaching of >>> the ToS Precedence >>> >>> OLD: >>> Figure 3: ToS Precedence Re-marking >>> NEW: >>> Figure 3: Bits in the Diffserv field modified by ToS Precedence >>> Re-marking >>> >>> OLD: >>> Figure 5: Observed Re-marking Behavior Resulting from Deriving >>> from UP-to-DSCP Mapping in Some 802.11 Networks >>> NEW: >>> Figure 5: Bits in the Diffserv field modified by Re-marking >>> Observed as a Result of UP-to-DSCP Mapping in Some 802.11 Networks >>> >>> 12) <!-- [rfced] Terminology >>> >>> a) Throughout the text, the following terminology appears to be >>> used inconsistently. May we update to the version on the right? >>> >>> ToS Precedence bleaching vs. ToS Precedence Bleaching >>> ToS precedence field vs. ToS Precedence field >>> >>> GF: Please do update as you suggest. >>> >>> b) We see that "Traffic Class" and "traffic class" along with >>> the acronym "TC" >>> are used throughout the document. Would you like to expand the >>> first instance >>> of these and then use the acronyms in the remainder of the >>> document? Or do >>> you prefer the current usage? >>> >>> GF: Ah, good question. These are various things with the same >>> name, and I think these are mostly correct: >>> >>> GF: No CHANGE: The term "traffic class", in section 5 is ok >>> GF: No CHANGE: The term "traffic class", lower case, in section >>> 5.1 is not related to MPLS and should remain in lower case. >>> GF: No CHANGE: The term "traffic class", lower case, in section >>> 5.2 does relate to MPLS. >>> >>> GF: No CHANGE: The term "traffic class", lower case, in section >>> 5.3 is not related to MPLS and should remain in lower case. >>> >>> >>> GF: PLEASE DO UPDATE: The term "traffic class", in section 6.2.1 >>> could potentially be confused, and would be better as "PHB": >>> OLD: >>> demoting the traffic to the Lower Effort traffic class. >>> NEW: >>> demoting the traffic to the Lower Effort PHB. >>> >>> c) To avoid confusion with "Access Classes (AC)", may we make >>> "Access Categories" lowercase? >>> >>> Original: >>> Then, in turn, this is mapped >>> to the four Wi-Fi Multimedia (WMM) Access Categories. >>> Perhaps: >>> Then, in turn, this is mapped >>> to the four Wi-Fi Multimedia (WMM) access categories. >>> >>> GF: Good note, please go ahead and do this. >>> >>> d) FYI, we have added expansions for abbreviations upon first >>> use. Please review each expansion in the document carefully to >>> ensure correctness. >>> >>> DSCP - Differentiated Services Code Point >>> EF - Expedited Forwarding >>> EXP - updated "experimental" to "Experimental" per RFC 8126 >>> IPX - IP Packet eXchange >>> LSR - updated "Label-Switched Router" to "Label Switching >>> Router" per published RFCs >>> LU - updated "local use" to "Local Use" per RFC 8126 >>> MAC - Media Access Control >>> MEF - updated "Metro Ethernet Forum" to "MEF Forum" >>> PDB - updated "Per Domain Behavior" to "Per-Domain Behavior" >>> like "Per-Hop" in PHB >>> P-GW - updated "Packet Gateway" to "Packet Data Network Gateway" >>> UMTS - Universal Mobile Telecommunications System >>> --> >>> >>> GF: These changes are all good. >>> >>> 13) <!-- [rfced] Please review whether any of the notes in this >>> document >>> should be in the <aside> element. It is defined as "a container >>> for content that is semantically less important or tangential to >>> the content that surrounds it" >>> (https://authors.ietf.org/rfcxml-vocabulary#aside). >>> --> >>> >>> GF: I am not aware of any content of this type. >>> >>> 14) <!-- [rfced] Please review the "Inclusive Language" portion >>> of the online Style Guide >>> <https://www.rfc-editor.org/styleguide/part2/#inclusive_language> >>> <https://www.rfc-editor.org/styleguide/part2/#inclusive_language> >>> and let us know if any changes are needed. >>> >>> Note that our script did not flag any words in particular, but >>> this should still be reviewed as a best practice. >>> --> >>> >>> GF: We plan to review the entire text bvefore we approve and we >>> will review this at this next stage >>> >>> Thank you. >>> >>> RFC Editor/st/ar >>> >>> GF: We have some very late cross-area INT review comments that >>> it was suggested we incorporate at this stage: >>> >>> 1. An updated citation for [IEEE.802.11] is: >>> >>> IEEE, "IEEE Standard for Information technology - >>> Telecommunications and information exchange between >>> systems - Local and metropolitan area networks - Specific >>> requirements - Part 11: Wireless LAN Medium Access Control >>> (MAC) and Physical Layer (PHY) Specifications", >>> IEEE 802.11, DOI 10.1109/IEEESTD.2016.7786995, December >>> 2016, https://standards.ieee.org/ieee/8802-11/10034/802.11/5536/. >>> >>> 2. s/exchange point/diffserv domain boundary/ in Section 4 >>> OLD: >>> and agreements between providers at an exchange point >>> NEW: >>> and agreements between providers at a diffserv domain boundary >>> >>> 3. Please remove the following sentence: This aligned with the >>> now deprecated use of >>> CS1 as the codepoint for the lower effort service, as previously >>> specified in [RFC4594]. >>> DELETE: >>> This aligned with the now deprecated use of CS1 as the codepoint >>> for the Lower Effort service, as previously specified in [RFC4594]. >>> >>> 4. Explain "unknown" using our agreed text: >>> OLD: >>> The treatment of packets that are marked with an unknown or an >>> unexpected DSCP at Diffserv domain boundaries is determined by >>> the policy for a Diffserv domain >>> NEW: >>> A packet that arrives with a DSCP that is not associated with a >>> PHB, results in an “unknown DSCP”. A node could receive a packet >>> with an "Unexpected DSCP" due to misconfiguration, or because >>> there is no consistent policy in place. The treatment of packets >>> that are marked with an unknown or an unexpected DSCP at >>> Diffserv domain boundaries is determined by the policy for a >>> Diffserv domain. >>> >>> === >>> >>> Thanks! >>> >>> On Jul 3, 2023, rfc-editor@rfc-editor.org wrote: >>> >>> *****IMPORTANT***** >>> >>> Updated 2023/07/03 >>> >>> 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> >>> <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/rfc9435.xml >>> https://www.rfc-editor.org/authors/rfc9435.html >>> https://www.rfc-editor.org/authors/rfc9435.pdf >>> https://www.rfc-editor.org/authors/rfc9435.txt >>> >>> Diff file of the text: >>> https://www.rfc-editor.org/authors/rfc9435-diff.html >>> https://www.rfc-editor.org/authors/rfc9435-rfcdiff.html (side by >>> side) >>> >>> Diff of the XML: >>> https://www.rfc-editor.org/authors/rfc9435-xmldiff1.html >>> >>> The following files are provided to facilitate creation of your >>> own diff files of the XML. >>> Initial XMLv3 created using XMLv2 as input: >>> https://www.rfc-editor.org/authors/rfc9435.original.v2v3.xml >>> XMLv3 file that is a best effort to capture v3-related format >>> updates only: https://www.rfc-editor.org/authors/rfc9435.form.xml >>> >>> >>> Tracking progress >>> ----------------- >>> >>> The details of the AUTH48 status of your document are here: >>> https://www.rfc-editor.org/auth48/rfc9435 >>> >>> Please let us know if you have any questions. >>> Thank you for your cooperation, >>> >>> RFC Editor >>> >>> -------------------------------------- >>> RFC9435 (draft-ietf-tsvwg-dscp-considerations-13) >>> >>> Title : Considerations for Assigning a New Recommended >>> Differentiated Services Codepoint (DSCP) >>> Author(s) : A. Custura, G. Fairhurst, R. Secchi >>> WG Chair(s) : Gorry Fairhurst, Marten Seemann >>> Area Director(s) : Martin Duke, Zaheduzzaman Sarker >>> >>> === >> >
- [auth48] Fwd: Fwd: AUTH48: RFC-to-be 9435 <draft-… Gorry Fairhurst
- [auth48] AD - Re: AUTH48: RFC-to-be 9435 <draft-i… Alice Russo
- Re: [auth48] AD - Re: AUTH48: RFC-to-be 9435 <dra… Gorry Fairhurst
- Re: [auth48] AUTH48: RFC-to-be 9435 <draft-ietf-t… Alice Russo
- Re: [auth48] AD - Re: AUTH48: RFC-to-be 9435 <dra… Martin Duke
- Re: [auth48] AD - Re: AUTH48: RFC-to-be 9435 <dra… Gorry Fairhurst
- Re: [auth48] AD - Re: AUTH48: RFC-to-be 9435 <dra… Alice Russo
- Re: [auth48] AD - Re: AUTH48: RFC-to-be 9435 <dra… Martin Duke
- Re: [auth48] AD - Re: AUTH48: RFC-to-be 9435 <dra… Ana C. Custura
- Re: [auth48] AUTH48: RFC-to-be 9435 <draft-ietf-t… Alice Russo
- Re: [auth48] AD - Re: AUTH48: RFC-to-be 9435 <dra… Alice Russo
- Re: [auth48] AD - Re: AUTH48: RFC-to-be 9435 <dra… Martin Duke
- Re: [auth48] AUTH48: RFC-to-be 9435 <draft-ietf-t… Gorry Fairhurst
- Re: [auth48] AUTH48: RFC-to-be 9435 <draft-ietf-t… Ana C. Custura
- Re: [auth48] AUTH48: RFC-to-be 9435 <draft-ietf-t… Secchi, Raffaello
- Re: [auth48] AUTH48: RFC-to-be 9435 <draft-ietf-t… Alice Russo