Re: [Detnet] Intdir last call review of draft-ietf-detnet-bounded-latency-08

Mohammadpour Ehsan <ehsan.mohammadpour@epfl.ch> Wed, 16 February 2022 09:39 UTC

Return-Path: <ehsan.mohammadpour@epfl.ch>
X-Original-To: detnet@ietfa.amsl.com
Delivered-To: detnet@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 5C1913A0D0C for <detnet@ietfa.amsl.com>; Wed, 16 Feb 2022 01:39:16 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.099
X-Spam-Level:
X-Spam-Status: No, score=-2.099 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, HTML_MESSAGE=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=epfl.ch
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id KtYVDZUTFzYs for <detnet@ietfa.amsl.com>; Wed, 16 Feb 2022 01:39:11 -0800 (PST)
Received: from smtp4.epfl.ch (smtp4.epfl.ch [IPv6:2001:620:618:1e0:1:80b2:e059:1]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 3DC063A0CF6 for <detnet@ietf.org>; Wed, 16 Feb 2022 01:39:10 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=epfl.ch; s=epfl; t=1645004346; h=From:To:CC:Subject:Date:Message-ID:Content-Type:MIME-Version; bh=YgKczFsg9TCKxSNtRMk8pSPd6zM+lwAEcRvuxLpZCFg=; b=wUhnZJXPvFPWdBC9lSr060yWKChgqX8L5JaPXogSBzg9vhdR+9zXX1h8A7j7G5iE8 jiQPEshjtdTsAxFY5ONSHZ+HxsW4q2sNk/61kUlWx7ct8Lt51TTK117eU8IFkj+Er rUM9kAAvYWfLvYkGcwSooPf5kLT0seknPElf84bD8=
Received: (qmail 35546 invoked by uid 107); 16 Feb 2022 09:39:06 -0000
Received: from ax-snat-224-169.epfl.ch (HELO ewa03.intranet.epfl.ch) (192.168.224.169) (TLS, ECDHE-RSA-AES256-GCM-SHA384 (X25519 curve) cipher) by mail.epfl.ch (AngelmatoPhylax SMTP proxy) with ESMTPS; Wed, 16 Feb 2022 10:39:06 +0100
X-EPFL-Auth: mUIdN2kAw+igo5RHmNu0C9fYNkrjBzxh6mPNRTphWNy8aP9BNZA=
Received: from ewa02.intranet.epfl.ch (128.178.224.159) by ewa03.intranet.epfl.ch (128.178.224.169) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2375.18; Wed, 16 Feb 2022 10:39:05 +0100
Received: from ewa02.intranet.epfl.ch ([fe80::ddaf:e0cc:a2d6:4aaf]) by ewa02.intranet.epfl.ch ([fe80::ddaf:e0cc:a2d6:4aaf%3]) with mapi id 15.01.2375.018; Wed, 16 Feb 2022 10:39:05 +0100
From: Mohammadpour Ehsan <ehsan.mohammadpour@epfl.ch>
To: Ralf Weber <rweber@akamai.com>
CC: "int-dir@ietf.org" <int-dir@ietf.org>, "detnet@ietf.org" <detnet@ietf.org>, "draft-ietf-detnet-bounded-latency.all@ietf.org" <draft-ietf-detnet-bounded-latency.all@ietf.org>, "last-call@ietf.org" <last-call@ietf.org>
Thread-Topic: Intdir last call review of draft-ietf-detnet-bounded-latency-08
Thread-Index: AQHYG2YSlws0rXOzCkS0RLE7mRHEXayV6oOA
Date: Wed, 16 Feb 2022 09:39:05 +0000
Message-ID: <E24E63D5-CE29-4A5C-ABB0-2D9F9D3E171E@epfl.ch>
References: <164415781002.3978.5883868989371563330@ietfa.amsl.com>
In-Reply-To: <164415781002.3978.5883868989371563330@ietfa.amsl.com>
Accept-Language: en-US, fr-CH
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [128.178.151.68]
Content-Type: multipart/alternative; boundary="_000_E24E63D5CE294A5CABB02D9F9D3E171Eepflch_"
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/detnet/y1_nPK0uk-xAJbKvuoonvSt_Mso>
Subject: Re: [Detnet] Intdir last call review of draft-ietf-detnet-bounded-latency-08
X-BeenThere: detnet@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Discussions on Deterministic Networking BoF and Proposed WG <detnet.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/detnet>, <mailto:detnet-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/detnet/>
List-Post: <mailto:detnet@ietf.org>
List-Help: <mailto:detnet-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/detnet>, <mailto:detnet-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 16 Feb 2022 09:39:17 -0000

Dear Ralf,

Thank you for your comments. We modified the formulas according to your proposal and used the term MaxPayloadSize in the T-SPEC. You can find the new version of the draft in:
https://www.ietf.org/archive/id/draft-ietf-detnet-bounded-latency-09.html

as well as the difference between the new version and the previous version in:
https://www.ietf.org/rfcdiff?url2=draft-ietf-detnet-bounded-latency-09


Best,
Ehsan




--
Ehsan Mohammadpour
PhD candidate at Swiss Federal Institute of Technology (EPFL)
IC IINFCOM, LCA2, INF 011, Station 14, 1015 Lausanne, Switzerland
https://people.epfl.ch/ehsan.mohammadpour

On 6 Feb 2022, at 15:30, Ralf Weber via Datatracker <noreply@ietf.org<mailto:noreply@ietf.org>> wrote:

Reviewer: Ralf Weber
Review result: Ready with Nits

Moin!

I am an assigned INT directorate reviewer for draft-ietf-detnet-bounded-latency.
These comments were written primarily for the benefit of the Internet Area
Directors. Document editors and shepherd(s) should treat these comments just
like they would treat comments from any other IETF contributors and resolve
them along with any other Last Call comments that have been received. For more
details on the INT Directorate, see
https://datatracker.ietf.org/group/intdir/about/

The draft is very detailed, well written and covers the subject exhaustively. I
think it is ready for publication.

The biggest problem I had was reading all the formulas, that because of the
limitations of being published as an RFC can't be printed nice as you would see
them in a math book. And while they are correct (with one exception) I think
they would be more readable if they had parentheses for groups especially when
going over multiple lines like e.g on page 13:

     backlog_bound = nb_input_ports * max_packet_length + total_in_rate
     * max_delay456

IMHO would be nicer as:

     backlog_bound = ( nb_input_ports * max_packet_length  ) +
                                 ( total_in_rate * max_delay456 )

there is on error on page 19 where the opening parentheses is missing:

     T_A = L_nA + b_h + r_h * L_n/c)/(c-r_h)

could be:

     T_A = ( L_nA + b_h + ( r_h * L_n/c )) / ( c-r_h )

or at least has to be:

     T_A = (L_nA + b_h + r_h * L_n/c)/(c-r_h)

we may have rules formulas in the IETF for this which might be different from
my examples and if we do we should of course follow them, but at least I don't
know of them.

Another nit was that on page 10 there is a description of a T-Spec (Traffice
Specification) of a DeNet flow that is defined in RFC9016 as:
       "Interval: tau, MaxPacketsPerInterval: K, MaxPacketSize: L"
however using the terms defined in RFC9016 section 5.5 it should be:
       "Interval: tau, MaxPacketsPerInterval: K, MaxPayloadSize: L"

Overall I'd like to thank the authors for a very well written document.

So long
-Ralf