Re: [auth48] AUTH48: RFC-to-be 9550 <draft-ietf-detnet-pof-11> for your review
Balázs Varga A <balazs.a.varga@ericsson.com> Tue, 05 March 2024 09:36 UTC
Return-Path: <balazs.a.varga@ericsson.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 087D2C14F615; Tue, 5 Mar 2024 01:36:53 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.107
X-Spam-Level:
X-Spam-Status: No, score=-2.107 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_BLOCKED=0.001, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01, URIBL_BLOCKED=0.001, URIBL_DBL_BLOCKED_OPENDNS=0.001, URIBL_ZEN_BLOCKED_OPENDNS=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=ericsson.com
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 wuRGbETJ97fA; Tue, 5 Mar 2024 01:36:48 -0800 (PST)
Received: from EUR02-AM0-obe.outbound.protection.outlook.com (mail-am0eur02on20601.outbound.protection.outlook.com [IPv6:2a01:111:f403:2606::601]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 3FD9CC14F6A1; Tue, 5 Mar 2024 01:36:46 -0800 (PST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=dGsvhizBM90G2hjPoRMMAODbn90sAPFGDSvblHPeuvV0HySbwS0J8alRa+k5g0+pKyaVpKgLa9m4za3rT+Wepmm456xMqdyDpshlhP/YpLklfPSJ1IUAtBcqoRBytIIhJlFJ4TiarjcBAQiWh6amjc7FbyCGYusTmNiTEnMCc1LHdcA94T5cIwyBilgtXQEd3lwMZgOgaSheSInnsBN/rK75hmPCdF3lhOnTAjJ8KvHhheRdnSabzaEW1+b762wZlYqeBbfo6RYPWY/5pwCTr7Le1QaJ8KpT5GrBqPN2LSY8sFdyKn8ULvto5fgbkM3AJDTUPs2fbLl1mqx6GxOYdg==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=+Pi8iYN0DbMAcYskwWPJAYO2/AvA4hme/bfsXD3mAGw=; b=Fw4HvpRBXQd8fyjyO9iSeF4NE8erUfnMhdozjmo4y6TLOSCxPgO61QKlR38M6BKDTeD9jDFquUE/mT7CyjZMvhcS+uASFYGIKRfqnTkR7O8Fds5WhoYRQPk/OWXy14MhDJvbg6nBvMzHq2w0OlvOFykWq69PvCo04/Vi+07TI4G2ryYC1xklb3n227/LYSadWfA9EqNfqNGKupcnBX30cSi+ghEzH8tr9aj60iodcae1YmlT6aSgRz/O6fM4Yx/ZAXaD1TT8hmts97RXXcbwIpkVLR0e9NiSNnhF9EQoTxGdlDB/Ss79LnlvgvL0z5LvEHEAwx6c2HIpdQwAgJbhGg==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=ericsson.com; dmarc=pass action=none header.from=ericsson.com; dkim=pass header.d=ericsson.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ericsson.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=+Pi8iYN0DbMAcYskwWPJAYO2/AvA4hme/bfsXD3mAGw=; b=H9z7X7luIV84EdWYv66HB8pUTwJT4eiA5dJ4evrQ3BnimYsHA1knYhrmIRIpcCOaqml+gbJyLkW7d12UIbXtC6VVkhOE7Cm2D/1J7V5KiICdW6v1vy6Jxybx6IG5hDUz568LNXX+7/YGND/cpQGnV5nqLIW4MxLZUmAKm8XSE478UE0GP9WVdo/336nnUNLRcainIT5SPl49VEdeAusbQa+szBSE6hacjfCGHC/UoGpAKDLdpes3C4VcaSEGdDC+X2ozyKbFlPh0EAApR4ip05ImhmqAwQarnr572jY6r6AKT/yh6zfqXPtKlzPcHPk4lshj8OsfGAnrHO5effnUrA==
Received: from PA4PR07MB7214.eurprd07.prod.outlook.com (2603:10a6:102:fa::10) by DU2PR07MB8077.eurprd07.prod.outlook.com (2603:10a6:10:2b6::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7339.39; Tue, 5 Mar 2024 09:36:41 +0000
Received: from PA4PR07MB7214.eurprd07.prod.outlook.com ([fe80::2d73:e65:2300:5937]) by PA4PR07MB7214.eurprd07.prod.outlook.com ([fe80::2d73:e65:2300:5937%3]) with mapi id 15.20.7339.035; Tue, 5 Mar 2024 09:36:41 +0000
From: Balázs Varga A <balazs.a.varga@ericsson.com>
To: "rfc-editor@rfc-editor.org" <rfc-editor@rfc-editor.org>, Janos Farkas <Janos.Farkas@ericsson.com>, "stephan.kehrer@belden.com" <stephan.kehrer@belden.com>, "tobias.heer@belden.com" <tobias.heer@belden.com>
CC: "detnet-ads@ietf.org" <detnet-ads@ietf.org>, "detnet-chairs@ietf.org" <detnet-chairs@ietf.org>, "lberger@labn.net" <lberger@labn.net>, "rdd@cert.org" <rdd@cert.org>, "auth48archive@rfc-editor.org" <auth48archive@rfc-editor.org>
Thread-Topic: AUTH48: RFC-to-be 9550 <draft-ietf-detnet-pof-11> for your review
Thread-Index: AQHabEKkzu/y0g2RaEOu6Qfd0d4LAbEnnbOg
Date: Tue, 05 Mar 2024 09:36:41 +0000
Message-ID: <PA4PR07MB721432B59F57D35140DBEE89AC222@PA4PR07MB7214.eurprd07.prod.outlook.com>
References: <20240302014004.27D8419759CC@rfcpa.amsl.com>
In-Reply-To: <20240302014004.27D8419759CC@rfcpa.amsl.com>
Accept-Language: hu-HU, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=ericsson.com;
x-ms-publictraffictype: Email
x-ms-traffictypediagnostic: PA4PR07MB7214:EE_|DU2PR07MB8077:EE_
x-ms-office365-filtering-correlation-id: edb9c7fa-b11d-40a3-e039-08dc3cf7c373
x-ms-exchange-senderadcheck: 1
x-ms-exchange-antispam-relay: 0
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: LpS3dXyB3SDfv/iIpDyYk/vrxckLvwMSyl4yYvlbu9GD0i+JAr0DmWhEhoMZ2V3/ReQSm04qHHeXttt39vw2U39G073AGGgDdWGmuilui4Qbz1coet/JmM21/hs8Okzlm2grhuXO3qmbu7eH/5xmeLJQ/rpUUrAGi/kTreyDL50WYl0K5ch1UK0Nl790rPnTTldFGzZYFyML7+E5VbwFGd/UaGl2TQg/toKkXYvgZCb2H6DsdF46+qAZKLZnz5mb6dpzfbYKfrqtz2bIVxfl1FwSHVoQqR/T0HqFSlBDPKMjBgFoSLXqI/w8t+8qoYqZbKECnsniW/BWuYTZjS4JyeWqgRcdBZCE7n2HUBo6boQgGsHchW1tf2wr/KArjP9qtYeK0SEWt6Iqe4n5w2ErpTe5SUVdzNsf3kq/TECYpeuuKXgRo5iqrmr8sYFIoJqyU5pNl3DGvCu6vepVRsF6mYJ5zpF2mjGbX4nl2p0YX8yxNTbnjHD6XTqB1OMOv6TAHS3bZh0fc0I/Wvh0YWtEb4pjdfXMe/L56huXSRMgzeY139zfXtkBKJtuQKwepKady0I1OlA3nirxA3fuSPKugKsbKm45WFqBNff44uJIhhk5ISHPZCLjJvhOrgKR8Z6s8sNYGvhfE7vVVcUxA4dR/GoKrv0Kf1x/S/0j6tTogWA=
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PA4PR07MB7214.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230031)(376005)(38070700009); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: b1RkklaFohUU4GkdvqQyg7xhi3H3l5rItzoOwRt58uLGNXsXQ3r3lmh2ZsYJvq+5QqlQGbyJSDMEkT4HF34pWCTjUSZQj/HrirR7eiGfJXme7eBJw5amlBsRC9RpVksqvMdUA2OY6aCE+qfZxG4/ndSKogGzdL+hHCM9HpXunOxSjE4gsI8+3cJfuQzpWGIIH0AKqCfdJNurNSNhcCdS/96dLMX2dPA7T0TG/rfoaBpQFG+8mjSpOuDVFK0wdJ3l7dQVuIF/aL6WMeDQPX3DDovYTfab+rrdf3eiwRo94EmYTUvcewzyNw6kcsOgqpv5I6cca0toeoWhXVq9k0WapqSvE1v1z8t1axQMvFX38sJfhdhXGBm3XaRZ/MRJNP/mqIpyggc1SIeUm/sKlYBI7PtOleFzhdqRCR0whHVjAmspE7wYKdsRYODECcIEoNSH06OhSrDY47ZFHWj/aRAvHFqNV6PdwtsR5RTjwMniVl4zrwMiN342Rjx3sY9+sv/HsgDRis6qa9lZ+OLiAtKvjyzRiEYTW3o/UN7Y9Qi8juAmedcYl5QIPb5D+7luzZKl+OzLfHQ3s0tqQ9sx+iAnXoZQOyFEO+go5C69/HA0FUkRsarc2zQZrfzuPiW9ZyeqM+NHpnVZ1Rt0nRNfPajGs95CnJUS9X1pJM2EydCWjJK+mxCaywwz86LC3NYRX8EtdZeM0OSnTdQwnUGRNnyfjFzdJjSN3Rl5+3M7KXMdsPZaC7Lrg4R9cVr1slscNLJ969KvtJsx493MwhgTwzPBs+dveWUwRRYm47XENXBdq0fvJkaPZQNZTaRE1aXlz8swzrP/6ahv4HHtPBHJ+UOfOQ2Wt3aQZJvIT8IgmnOduq4QIQ/yFStYtD0gQW7BQGDdybm2PI0f3bRQ2dcn4eN7kaBxbiecCMoI7HxZ3w5nJdoCQcB2jvqGDIm3UhlijgcOQnl2gDbadVAQs2vjO0xcbxZhOfL6HZ3VrsZL2Id/ipH+RMjO81ebuoOTjPZNuruFiw/6s1Saq7X4UC5iMwDGyRm/DxVmAVJzMVG8zSw8L86OtpRMB1avr+M/PYK9O45JirSAG6bEGYGiWQ/apAFvhBt0tASX+UOm4auXqLR5JTYkSyP9DHQlQooBj22PobKNRdLGOs6v5FD81QFrYg6vppgLxSq3xDqG7YGAjyQQW0oftWJW3pt83m7MgLjdMrEHmlNo2s5t3YXYvKoYzsftD4V88CXmFYam6c/kKbqnhdHzxakXCc1xfqza912tEposUb5FNQOgNYYTdYuvVTssgsp/6RaTc+eiTi+DhLbCmspchKhzlM9hMZlxbUr+thNkDNBWwqHCRDgM9VUkTU7O9M1x3+9r8eK8crQRRA+sjU44pTSksfVyJSM+8t+TOMeZ0uaBk1bxX5UL1ltbhREByo90IeLdHCpJBdOBVJ5xqtCbHWXNg0otehFjP2wl116pOKqAT3WTgU3UqEpcfZcUy/M9z49pQknqGzrJR2ieJ140Vicj50NARRmA7ddtj1yfJCT923j8TE+fBBYZx3an7lAxEg8+ZFHRr2TZDsPjg6gQC8TIfAq3u7+hGI/jhTWS
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: ericsson.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: PA4PR07MB7214.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: edb9c7fa-b11d-40a3-e039-08dc3cf7c373
X-MS-Exchange-CrossTenant-originalarrivaltime: 05 Mar 2024 09:36:41.2958 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 92e84ceb-fbfd-47ab-be52-080c6b87953f
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: I/zTYehRlYVwiBHi/8Sya8QsBQwLGT2Biq1wnfL7QCgGLgnNCGH32qXctTVushQOFea0emWYB/0v4jznWvWpRxWVA2hhhXLLkOlWSoSGlT0=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU2PR07MB8077
Archived-At: <https://mailarchive.ietf.org/arch/msg/auth48archive/t8tdpP8ZlOe-o9OP_T-ICG81QzQ>
Subject: Re: [auth48] AUTH48: RFC-to-be 9550 <draft-ietf-detnet-pof-11> 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: Tue, 05 Mar 2024 09:36:53 -0000
Hi RFC Editor/rv, Many thanks for your detailed review and proposed changes. My replies inline marked with <BalazsV>. Please, let us know if further clarification needed. I will reach out to co-authors regarding your first comment. Again, many thanks for your efforts Bala'zs -----Original Message----- From: rfc-editor@rfc-editor.org <rfc-editor@rfc-editor.org> Sent: Saturday, March 2, 2024 2:40 AM To: Balázs Varga A <balazs.a.varga@ericsson.com>; Janos Farkas <Janos.Farkas@ericsson.com>; stephan.kehrer@belden.com; tobias.heer@belden.com Cc: rfc-editor@rfc-editor.org; detnet-ads@ietf.org; detnet-chairs@ietf.org; lberger@labn.net; rdd@cert.org; auth48archive@rfc-editor.org Subject: Re: AUTH48: RFC-to-be 9550 <draft-ietf-detnet-pof-11> for your review Authors, While reviewing this document during AUTH48, please resolve (as necessary) the following questions, which are also in the XML file. 1) <!-- [rfced] This is a question for Stephan and Tobias. Would you like to use a shortened form of your affiliation in the first-page header and then the full name in the Authors' Addresses section? Please review the first-page header in each of the output formats (txt, html, and pdf), and let us know your thoughts. --> 2) <!-- [rfced] Please insert any keywords (beyond those that appear in the title) for use on https://www.rfc-editor.org/search. --> <BalazsV> I think no further keywords needed. 3) <!-- [rfced] Would it be helpful to revise this sentence to state that "[RFC8655] defines" rather than "The DetNet Working Group has defined"? In addition, please clarify "but without any implementation details" in the second sentence below. Original: The DetNet Working Group has defined packet replication (PRF) and packet elimination (PEF) functions for achieving extremely low packet loss. PRF and PEF are described in [RFC8655] and provide service protection for DetNet flows. ... The IETF DetNet WG has defined in [RFC8655] the external observable result of a POF function, i.e., that packets are reordered, but without any implementation details. Perhaps: [RFC8655] defines the Packet Replication Function (PRF) and Packet Elimination Function (PEF) in DetNet for achieving extremely low packet loss. PRF and PEF provide service protection for DetNet flows. ... [RFC8655] defines the external observable result of a POF function (i.e., that packets are reordered) but does not specify any implementation details. --> <BalazsV> Your proposal is perfect. OLD TEXT: The DetNet Working Group has defined packet replication (PRF) and packet elimination (PEF) functions for achieving extremely low packet loss. PRF and PEF are described in [RFC8655] and provide service protection for DetNet flows. ... The IETF DetNet WG has defined in [RFC8655] the external observable result of a POF function, i.e., that packets are reordered, but without any implementation details. NEW TEXT: [RFC8655] defines the Packet Replication Function (PRF) and Packet Elimination Function (PEF) in DetNet for achieving extremely low packet loss. PRF and PEF provide service protection for DetNet flows. ... [RFC8655] defines the external observable result of a POF function (i.e., that packets are reordered) but does not specify any implementation details. END 4) <!-- [rfced] Please confirm that [RFC8655] is correct here. We believe it is correct, but we want to make sure. We see "sequence number" and "encapsulation" in RFC 8655 but not used together. Original: This can be done by adding a sequence number as part of DetNet encapsulation [RFC8655]. --> <BalazsV> Yes confirmed. [RFC8655] is correct here. 5) <!-- [rfced] Are the parentheses needed here with "DetNet"? Original: POF ensures in-order delivery for packets being within the latency bound of the (DetNet) flow. Perhaps: POF ensures in-order delivery for packets within the latency bound of the DetNet flow. --> <BalazsV> No parentheses needed here. Your proposed change is perfect. OLD TEXT: POF ensures in-order delivery for packets being within the latency bound of the (DetNet) flow. NEW TEXT: POF ensures in-order delivery for packets within the latency bound of the DetNet flow. END 6) <!-- [rfced] The title of Section 3 uses "POF Implementations", but the first sentence in the section uses "POF function". Should these be consistent? Original: 3. Requirements on POF Implementations The requirements on a POF function are: Perhaps: 3. Requirements for POF Implementations The requirements for POF implementations are: --> <BalazsV> Consistency is important. Your proposed change is perfect. OLD TEXT: 3. Requirements on POF Implementations The requirements on a POF function are: NEW TEXT: 3. Requirements for POF Implementations The requirements for POF implementations are: END 7) <!-- [rfced] Would moving "in network nodes" to the end of the sentence improve readability? Also, please review "states/configuration parameters and resources". Specifically, is "states" separate from "configuration parameters", or does "states/configuration" describe the type of parameters? Original: * to be simple and to require in network nodes only a minimum set of states/configuration parameters and resources per DetNet Flow. Perhaps: * To be simple and to require only a minimum set of states, configuration parameters, and resources per DetNet flow in network nodes. Or ("states" to "state"): * To be simple and to require only a minimum set of state/configuration parameters and resources per DetNet flow in network nodes. --> <BalazsV> Your proposed change reads better. Yes, "states" are separate from "configuration parameters". OLD TEXT: * to be simple and to require in network nodes only a minimum set of states/configuration parameters and resources per DetNet Flow. NEW TEXT: * To be simple and to require only a minimum set of states, configuration parameters, and resources per DetNet flow in network nodes. END 8) <!-- [rfced] The text above Figure 2 uses "Conditional buffer", but Figure 2 uses "Conditional Delay Buffer". Should the figure be updated for consistency with the text or vice versa? --> <BalazsV> Right. Please, change text to use "Conditional Delay Buffer". 9) <!-- [rfced] Please review "the difference of sequence number" in these sentences. Is the intended meaning "the difference between sequence numbers"? Original: Note: the difference of sequence number in consecutive packets is bounded due to the history window of the Elimination function before the POF. ... For example, under normal circumstances the difference of sequence number in consecutive packets is bounded due to the history window of PEF. Perhaps: The difference between sequence numbers in consecutive packets is bounded due to the history window of the elimination function before the POF. ... For example, under normal circumstances, the difference between sequence numbers in consecutive packets is bounded due to the history window of PEF. --> <BalazsV> Right. Confirmed, that it means "the difference between sequence numbers". Your proposal is OK. OLD TEXT: Note: the difference of sequence number in consecutive packets is bounded due to the history window of the Elimination function before the POF. ... For example, under normal circumstances the difference of sequence number in consecutive packets is bounded due to the history window of PEF. NEW TEXT: The difference between sequence numbers in consecutive packets is bounded due to the history window of the elimination function before the POF. ... For example, under normal circumstances, the difference between sequence numbers in consecutive packets is bounded due to the history window of PEF. END 10) <!-- [rfced] We updated "Packets being in order" as shown below. Please review. Original: Packets being in order are not delayed. Updated: In-order packets are not delayed. --> <BalazsV> Right. Your proposal is OK. OLD TEXT: Packets being in order are not delayed. NEW TEXT: In-order packets are not delayed. END 11) <!-- [rfced] Please confirm that "relations" is the correct word choice here. Original: Figure 3 shows the delay budget relations at the POF point. ... Figure 3: Delay Budget Relations at the POF Point Perhaps: Figure 3 shows the relationship between delay budgets at the POF point. ... Figure 3: Relationship between Delay Budgets at the POF Point --> <BalazsV> Right. "relations" is not the correct word here. Using "situation" would be better. OLD TEXT: Figure 3 shows the delay budget relations at the POF point. ... Figure 3: Delay Budget Relations at the POF Point NEW TEXT: Figure 3 shows the delay budget situation at the POF point. ... Figure 3: Delay Budget Situation at the POF Point END 12) <!-- [rfced] In the text introducing the list, may we update "needs two extensions of" in one of the following ways to improve clarity? Original: The advanced POF algorithm needs two extensions of the basic POF algorithm: * to identify the received packet's path at the POF location and * to make the value of "POFMaxDelay" for buffered packets path dependent ("POFMaxDelay_i", where "i" notes the path the packet has used). Perhaps: The advanced POF algorithm extends the basic POF algorithm: * to identify the received packet's path at the POF location and * to make the value of "POFMaxDelay" for buffered packets path dependent ("POFMaxDelay_i", where "i" notes the path the packet has used). Or: The advanced POF algorithm requires extensions of the basic POF algorithm: * to identify the received packet's path at the POF location and * to make the value of "POFMaxDelay" for buffered packets path dependent ("POFMaxDelay_i", where "i" notes the path the packet has used). --> <BalazsV> Right. Your second proposal is OK. OLD TEXT: The advanced POF algorithm needs two extensions of the basic POF algorithm: * to identify the received packet's path at the POF location and * to make the value of "POFMaxDelay" for buffered packets path dependent ("POFMaxDelay_i", where "i" notes the path the packet has used). NEW TEXT: The advanced POF algorithm requires extensions of the basic POF algorithm: * to identify the received packet's path at the POF location and * to make the value of "POFMaxDelay" for buffered packets path dependent ("POFMaxDelay_i", where "i" notes the path the packet has used). END 13) <!-- [rfced] How may we clarify the beginning of this sentence (i.e., "By....information")? Also, should "POFMaxDelay_i" appear in parentheses? Original: By identifying the path of a given packet, the POF algorithm can use this information to select what predefined time "POFMaxDelay_i" to apply for the buffered packet. Perhaps: The POF algorithm identifies the path of a given packet and uses this information to select the predefined time ("POFMaxDelay_i") to apply for the buffered packet. --> <BalazsV> Right. Your proposal is OK. OLD TEXT: By identifying the path of a given packet, the POF algorithm can use this information to select what predefined time "POFMaxDelay_i" to apply for the buffered packet. NEW TEXT: The POF algorithm identifies the path of a given packet and uses this information to select the predefined time ("POFMaxDelay_i") to apply for the buffered packet. END 14) <!-- [rfced] We are having trouble understanding the text in the parenthetical (i.e., "e.g., replication...PathID"). Please clarify. Original: It can be implemented via various techniques, for example using ingress interface information, encoding the path in the packet itself (e.g., replication functions can set different FlowID per egress what can be used as a PathID), or in other means. Perhaps: It can be implemented via various techniques, for example, using ingress interface information, encoding the path in the packet itself (e.g., replication functions can set a different FlowID per egress, which can be used as a PathID), or other means. --> <BalazsV> Right. Your proposal is somewhat finetuned. Please, let us know if further clarification needed. OLD TEXT: It can be implemented via various techniques, for example using ingress interface information, encoding the path in the packet itself (e.g., replication functions can set different FlowID per egress what can be used as a PathID), or in other means. NEW TEXT: It can be implemented via various techniques, for example, using ingress interface information, encoding the path in the packet itself (e.g., replication functions set a different FlowID per member flows at their egress and such a FlowID is used to identify the path of a packet at the POF), or other means. END 15) <!-- [rfced] Please review "for example" here. Is it needed, or would it be more clear if placed elsewhere in the sentence? Original: The challenge for POF initialization is that for example after a reset it is not known whether the first received packet is in-order or out-of-order. Perhaps: The challenge for POF initialization is that it is not known whether the first received packet is in order or out of order (for example, after a reset). Or: The challenge for POF initialization is that after a reset, it is not known whether the first received packet is in-order or out-of-order. --> <BalazsV> Right. Your first proposal is OK. OLD TEXT: The challenge for POF initialization is that for example after a reset it is not known whether the first received packet is in-order or out-of-order. NEW TEXT: The challenge for POF initialization is that it is not known whether the first received packet is in order or out of order (for example, after a reset). END 16) <!-- [rfced] Should "(see before)" be updated to a specific section? If not, we will update to "(see above)". Original: The original initialization (see before) considers the first packet as in-order, so out-of-order packet(s) during "POFMaxTime"/"POFMaxTime_path_i" time - after the first packet was received - cannot be corrected. --> <BalazsV> Right. Better to add an explicit reference to Section 4.3. Please change as follow: OLD TEXT: The original initialization (see before) considers the first packet as in-order, so out-of-order packet(s) during "POFMaxTime"/"POFMaxTime_path_i" time - after the first packet was received - cannot be corrected. NEW TEXT: The original initialization (see Section 4.3) considers the first packet as in-order, so out-of-order packet(s) during "POFMaxTime"/"POFMaxTime_path_i" time - after the first packet was received - cannot be corrected. END 17) <!-- [rfced] Will "basic/advanced" be clear to readers? Original: * No basic/advanced POF rules are applied until the first timer expires. ... * The basic/advanced POF rules are applied for the packet(s) in the buffer and the subsequently received packets. Perhaps: * No basic or advanced POF rules are applied until the first timer expires. ... * The basic or advanced POF rules are applied for the packet(s) in the buffer and the subsequently received packets. --> <BalazsV> Right. No strong preference here. Your change is OK. OLD TEXT: * No basic/advanced POF rules are applied until the first timer expires. ... * The basic/advanced POF rules are applied for the packet(s) in the buffer and the subsequently received packets. NEW TEXT: * No basic or advanced POF rules are applied until the first timer expires. ... * The basic or advanced POF rules are applied for the packet(s) in the buffer and the subsequently received packets. END 18) <!-- [rfced] How may we update "needs setting of" in this sentence? Original: POF algorithms needs setting of the following parameters: Perhaps: The following parameters should be set for POF algorithms: Or: POF algorithms require the following parameters to be set: --> <BalazsV> Right. No strong preference here. Slightly prefer Your second version. OLD TEXT: POF algorithms needs setting of the following parameters: NEW TEXT: POF algorithms require the following parameters to be set: END 19) <!-- [rfced] We tried to access the URL in this reference entry, but it is behind a sign-in wall. Also, it looks like this reference entry points to a draft that has now been published. May use one of the following URLs and update the rest of the entry accordingly (e.g., title, DOI, and date)? Possible URLs: https://standards.ieee.org/ieee/802.1CBcv/7285/ https://ieeexplore.ieee.org/document/9715061 Original: [IEEEP8021CBcv] Kehrer, S., "FRER YANG Data Model and Management Information Base Module", IEEE P802.1CBcv /D1.2 P802.1CBcv, March 2021, <https://protect2.fireeye.com/v1/url?k=31323334-501cfaf3-313273af-454445554331-3e5e7c6dcc50f3ea&q=1&e=f2d48fdf-37cd-4595-a8fa-4cf3226814ef&u=https%3A%2F%2Fwww.ieee802.org%2F1%2Ffiles%2Fprivate%2Fcv-drafts%2Fd1%2F802- 1CBcv-d1-2.pdf>. Perhaps: [IEEEP8021CBcv] IEEE, "IEEE Standard for Local and metropolitan area networks - Frame Replication and Elimination for Reliability - Amendment 1: Information Model, YANG Data Model, and Management Information Base Module, IEEE Std 802.1CBcv-2001, DOI 10.1109/IEEESTD.2022.9715061, February 2022, <https://standards.ieee.org/ieee/802.1CBcv/7285/>. --> <BalazsV> Thanks. Your change is correct. OLD TEXT: [IEEEP8021CBcv] Kehrer, S., "FRER YANG Data Model and Management Information Base Module", IEEE P802.1CBcv /D1.2 P802.1CBcv, March 2021, <https://protect2.fireeye.com/v1/url?k=31323334-501cfaf3-313273af-454445554331-3e5e7c6dcc50f3ea&q=1&e=f2d48fdf-37cd-4595-a8fa-4cf3226814ef&u=https%3A%2F%2Fwww.ieee802.org%2F1%2Ffiles%2Fprivate%2Fcv-drafts%2Fd1%2F802- 1CBcv-d1-2.pdf>. NEW TEXT: [IEEEP8021CBcv] IEEE, "IEEE Standard for Local and metropolitan area networks - Frame Replication and Elimination for Reliability - Amendment 1: Information Model, YANG Data Model, and Management Information Base Module, IEEE Std 802.1CBcv-2001, DOI 10.1109/IEEESTD.2022.9715061, February 2022, <https://standards.ieee.org/ieee/802.1CBcv/7285/>. END 20) <!-- [rfced] We updated these names in the Acknowledgements section. Please review and let us know any concerns. a) Per https://datatracker.ietf.org/person/ehsan.mohammadpour@swisscom.com: Original: Mohammadpour Ehsan Updated: Ehsan Mohammadpour b) Per https://datatracker.ietf.org/person/shirley.yangfan@huawei.com: Original: Shirley Yangfan Updated: Fan Yang --> <BalazsV> Thanks. Your change is correct. 21) <!-- [rfced] Should "algorithm" be updated to "algorithms" (plural) in the following sentences as both a basic and advanced POF algorithm are defined in Section 4? Or in some of these instances (perhaps the last two below), is the intent to say "the basic POF algorithm" or "the advanced POF algorithm"? Original: The Packet Ordering Function (POF) algorithm described herein enables to restore the correct packet order when replication and elimination functions are used in DetNet networks. ... 4.6. Selecting and using the POF algorithm ... The POF Algorithm discussed in this document makes some assumptions and tradeoffs regarding the characteristics of the sequence of received packets. ... In particular, the algorithm assumes that a Packet Elimination Function (PEF) is performed on the incoming packets before they are handed to the POF function. ... The algorithm further requires that the delay difference between two replicated packets that arrive at the PEF before the POF is bounded and known. ... The algorithm also makes some tradeoffs. ... Note2: The algorithm can be extended to cope with multiple failure scenarios (i.e., simultaneous packet loss and out-of-order packets), ... ... By identifying the path of a given packet, the POF algorithm can use this information to select what predefined time "POFMaxDelay_i" to apply for the buffered packet. --> <BalazsV> Thanks. Using plural is better in this text, except the last two places, where explicit reference is OK. OLD TEXT: The Packet Ordering Function (POF) algorithm described herein enable to restore the correct packet order when replication and elimination functions are used in DetNet networks. ... 4.6. Selecting and using the POF algorithm ... The POF Algorithms discussed in this document makes some assumptions and tradeoffs regarding the characteristics of the sequence of received packets. ... In particular, the algorithm assumes that a Packet Elimination Function (PEF) is performed on the incoming packets before they are handed to the POF function. ... The algorithm further requires that the delay difference between two replicated packets that arrive at the PEF before the POF is bounded and known. ... The algorithm also makes some tradeoffs. ... Note2: The algorithm can be extended to cope with multiple failure scenarios (i.e., simultaneous packet loss and out-of-order packets), ... ... By identifying the path of a given packet, the POF algorithm can use this information to select what predefined time "POFMaxDelay_i" to apply for the buffered packet. NEW TEXT: The Packet Ordering Function (POF) algorithms described herein enables to restore the correct packet order when replication and elimination functions are used in DetNet networks. ... 4.1. Prerequisites and Assumptions ... The POF Algorithms discussed in this document makes some assumptions and tradeoffs regarding the characteristics of the sequence of received packets. ... In particular, the algorithms assume that a Packet Elimination Function (PEF) is performed on the incoming packets before they are handed to the POF function. ... The algorithms further require that the delay difference between two replicated packets that arrive at the PEF before the POF is bounded and known. ... The algorithms also make some tradeoffs. ... 4.3. The Basic POF Algorithm ... Note2: The basic POF algorithm can be extended to cope with multiple failure scenarios (i.e., simultaneous packet loss and out-of-order packets), ... ... 4.4. The Advanced POF Algorithm ... By identifying the path of a given packet, the advanced POF algorithm can use this information to select what predefined time "POFMaxDelay_i" to apply for the buffered packet. END 22) <!-- [rfced] Terminology a) We see inconsistency in the capitalization of the following. We updated to use the lowercase form. Please let us know any objections. Replication and Elimination function replication and elimination functions replication function Elimination function <BalazsV> OK to use lowercase, except in Section "2.2. Abbreviations". b) Should "function" be removed in these instances because the expansion of the acronym already includes the word "Function"? For example, if expanded, "POF function" would read "Packet Ordering Function function". PEF function POF function PREOF functions <BalazsV> Yes, please remove "function". c) Use of quotation marks for the following terms is inconsistent. Please review and let us know how to update for consistency. "POFLastSent" vs. POFLastSent "POFMaxDelay" vs. POFMaxDelay "POFMaxDelay_i" "POFMaxTime"/"POFMaxTime_path_i" "POFTakeAnyTime" <BalazsV> Hhmmm. Right. The intention was to use "POFLastSent" in descriptive text and use POFLastSent in mathematical formulas. Would that make sense for You? One good example is in Section 4.3. The Basic POF Algorithm, where "" used as per our intention: ... * If (seq_num <= POFLastSent + 1) - Then the packet is forwarded without buffering and "POFLastSent" is updated (POFLastSent = seq_num). Unfortunately bad examples also in the same section, which needs correction: OLD TEXT * The sequence number of the last forwarded packet (POFLastSent) is stored for each DetNet Flow. * The sequence number (seq_num) of a received packet is compared to that of the last forwarded one (POFLastSent). NEW TEXT * The sequence number of the last forwarded packet ("POFLastSent") is stored for each DetNet Flow. * The sequence number ("seq_num") of a received packet is compared to that of the last forwarded one ("POFLastSent"). END Could You make the changes accordingly? d) Please review the usage of the articles "a" or "the" with PEF, PRF, and POF when used as a noun. An article is used in some instances but not others. Are any updates needed, or is the current usage acceptable? This may be on a case-by-case basis. A few examples with and without the article are listed below. Some examples with article (the POF, the POF, the PEF): Note: the difference of sequence number in consecutive packets is bounded due to the history window of the Elimination function before the POF. Error cases in which duplicate packets are presented to the POF can lead to out of order delivery of duplicate packets as well as to increased delays. The algorithm further requires that the delay difference between two replicated packets that arrive at the PEF before the POF is bounded and known. Some examples with no article (PEF, POF, POF): For example, under normal circumstances the difference of sequence number in consecutive packets is bounded due to the history window of PEF. POF only provides ordering within the latency bound of a DetNet flow, and does not provide any additional reliability. POF ensures in-order delivery for packets being within the latency bound of the (DetNet) flow. POF does not correct errors in the packet flow e.g., duplicate packets, too late packets. --> <BalazsV> Right. As a non-native speaker, I have to admit having issues with articles. Sorry for that. I would trust your suggestions. 23) <!-- [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/en/rfcxml-vocabulary#aside). --> <BalazsV> Notes are important and related to the content that surrounds it. No need to use <aside> in our understanding. 24) <!-- [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. Note that our script did not flag any words in particular, but this should still be reviewed as a best practice. --> <BalazsV> Review done. No changes needed. Thank you. RFC Editor/rv On Mar 1, 2024, at 5:32 PM, rfc-editor@rfc-editor.org wrote: *****IMPORTANT***** Updated 2024/03/01 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/rfc9550.xml https://www.rfc-editor.org/authors/rfc9550.html https://www.rfc-editor.org/authors/rfc9550.pdf https://www.rfc-editor.org/authors/rfc9550.txt Diff file of the text: https://www.rfc-editor.org/authors/rfc9550-diff.html https://www.rfc-editor.org/authors/rfc9550-rfcdiff.html (side by side) Diff of the XML: https://www.rfc-editor.org/authors/rfc9550-xmldiff1.html Alt-diff of the text (allows you to more easily view changes where text has been deleted or moved): https://www.rfc-editor.org/authors/rfc9550-alt-diff.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/rfc9550.original.v2v3.xml XMLv3 file that is a best effort to capture v3-related format updates only: https://www.rfc-editor.org/authors/rfc9550.form.xml Tracking progress ----------------- The details of the AUTH48 status of your document are here: https://www.rfc-editor.org/auth48/rfc9550 Please let us know if you have any questions. Thank you for your cooperation, RFC Editor -------------------------------------- RFC9550 (draft-ietf-detnet-pof-11) Title : Deterministic Networking (DetNet): Packet Ordering Function Author(s) : B. Varga, J. Farkas, S. Kehrer, T. Heer WG Chair(s) : Lou Berger, János Farkas Area Director(s) : Alvaro Retana, John Scudder, Andrew Alston
- [auth48] AUTH48: RFC-to-be 9550 <draft-ietf-detne… rfc-editor
- Re: [auth48] AUTH48: RFC-to-be 9550 <draft-ietf-d… rfc-editor
- Re: [auth48] AUTH48: RFC-to-be 9550 <draft-ietf-d… Balázs Varga A
- Re: [auth48] AUTH48: RFC-to-be 9550 <draft-ietf-d… Rebecca VanRheenen
- Re: [auth48] AUTH48: RFC-to-be 9550 <draft-ietf-d… Balázs Varga A
- Re: [auth48] AUTH48: RFC-to-be 9550 <draft-ietf-d… Rebecca VanRheenen
- Re: [auth48] AUTH48: RFC-to-be 9550 <draft-ietf-d… Janos Farkas
- Re: [auth48] AUTH48: RFC-to-be 9550 <draft-ietf-d… Rebecca VanRheenen
- Re: [auth48] AUTH48: RFC-to-be 9550 <draft-ietf-d… Tobias Heer
- Re: [auth48] AUTH48: RFC-to-be 9550 <draft-ietf-d… Stephan Kehrer
- Re: [auth48] AUTH48: RFC-to-be 9550 <draft-ietf-d… Rebecca VanRheenen
- Re: [auth48] AUTH48: RFC-to-be 9550 <draft-ietf-d… Balázs Varga A