Re: [tsvwg] Benjamin Kaduk's No Objection on draft-ietf-tsvwg-rlc-fec-scheme-14: (with COMMENT)

Roman Danyliw <> Tue, 18 June 2019 09:23 UTC

Return-Path: <>
Received: from localhost (localhost []) by (Postfix) with ESMTP id B8255120074; Tue, 18 Jun 2019 02:23:37 -0700 (PDT)
X-Virus-Scanned: amavisd-new at
X-Spam-Flag: NO
X-Spam-Score: -1.998
X-Spam-Status: No, score=-1.998 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: (amavisd-new); dkim=pass (1024-bit key)
Received: from ([]) by localhost ( []) (amavisd-new, port 10024) with ESMTP id N-VZAQfJHuVM; Tue, 18 Jun 2019 02:23:36 -0700 (PDT)
Received: from ( []) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by (Postfix) with ESMTPS id D194912000E; Tue, 18 Jun 2019 02:23:35 -0700 (PDT)
Received: from ( []) by (8.14.7/8.14.7) with ESMTP id x5I9NYuD005448; Tue, 18 Jun 2019 05:23:34 -0400
DKIM-Filter: OpenDKIM Filter v2.11.0 x5I9NYuD005448
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=yc2bmwvrj62m; t=1560849814; bh=Fk7Vgf5W6JDapiUWPNBExYKdWiIlboi7e06MoyERaBc=; h=From:To:CC:Subject:Date:References:In-Reply-To:From; b=gIDhQaMDnMyA23xnv9kzZzEp1OVgUEjKYmDB7oXiI0Uyjb1ntlY76/wEtGJCOu4hs Hn+vfPg40+E9gj6xrldRC1ZG9xzdlnMVA+ANLA2DarOeO6+NGsC71pH5XCK4b1xBG9 wrS1YOSwzR9lsUFBu5nqBarisAup6K+PQMQMiO4k=
Received: from ( []) by (8.14.7/8.14.7) with ESMTP id x5I9NXlJ021017; Tue, 18 Jun 2019 05:23:33 -0400
Received: from ([]) by ([]) with mapi id 14.03.0439.000; Tue, 18 Jun 2019 05:23:32 -0400
From: Roman Danyliw <>
To: Vincent Roca <>
CC: The IESG <>, "" <>, "" <>, "" <>, "Benjamin Kaduk" <kaduk@MIT.EDU>
Thread-Topic: [tsvwg] Benjamin Kaduk's No Objection on draft-ietf-tsvwg-rlc-fec-scheme-14: (with COMMENT)
Thread-Index: AQHVFde/aAoeGJm/QEO9wocH49MKcaaYi6GAgAFFd6CAB6q+AP//xwqg
Date: Tue, 18 Jun 2019 09:23:32 +0000
Message-ID: <359EC4B99E040048A7131E0F4E113AFC01B33996F9@marathon>
References: <> <> <359EC4B99E040048A7131E0F4E113AFC01B33926C6@marathon> <>
In-Reply-To: <>
Accept-Language: en-US
Content-Language: en-US
x-originating-ip: []
Content-Type: multipart/alternative; boundary="_000_359EC4B99E040048A7131E0F4E113AFC01B33996F9marathon_"
MIME-Version: 1.0
Archived-At: <>
Subject: Re: [tsvwg] Benjamin Kaduk's No Objection on draft-ietf-tsvwg-rlc-fec-scheme-14: (with COMMENT)
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Transport Area Working Group <>
List-Unsubscribe: <>, <>
List-Archive: <>
List-Post: <>
List-Help: <>
List-Subscribe: <>, <>
X-List-Received-Date: Tue, 18 Jun 2019 09:23:38 -0000


From: Vincent Roca []
Sent: Tuesday, June 18, 2019 4:46 AM
To: Roman Danyliw <>;; Benjamin Kaduk <kaduk@MIT.EDU>;
Cc: Vincent Roca <>;; The IESG <>;;;;
Subject: Re: [tsvwg] Benjamin Kaduk's No Objection on draft-ietf-tsvwg-rlc-fec-scheme-14: (with COMMENT)

Hi Roman, Benjamin, all,

Back to the RLC I-D.
I have copied below the Discuss/Comments made for the -14 version of the document.

Many thanks, the document quality has been further improved.

   Vincent and Belkacem


> Roman Danyliw Discuss
> Discuss (2019-05-28)
> A few code nits for Section 3.6 so that the code compiles:
> ** To make the combination of this source code and that in draft-ietf-tsvwg-tinymt32 compile requires that the directive “#include <string.h>” be added (for the memset).
> ** The final return in Section 3.6 is missing a semicolon:
> s/return 0/return 0;/

[VR] Done. I usually don't add system headers to focus on key aspects, but it does not hurt. And thanks for the ";".


> Comment (2019-05-28)
> (1) Section 3.5  Is there any guidance that needs to be provided on the value of the seed to tinymt32_init?

[VR] Good point. Anything is accepted, which is now clearly stated. Since we already discuss Repair_Key management (this repair_key is used as seed), we refer to section 6.1.

   With the FEC Schemes defined in this document, the seed is in
   practice restricted to a value between 0 and 0xFFFF inclusive (note
   that this PRNG accepts a seed value equal to 0), since this is the
   Repair_Key 16-bit field value of the Repair FEC Payload ID
   (Section 4.1.3).  In practice, how to manage the seed and Repair_Key
   values (both are equal) is left to the implementer, using a
   monotonically increasing counter being one possibility (Section 6.1).

> (2) Section 3.6.  In the code comments for cc_nb, I recommend explicitly stating:
> s/number of entries in the table/
> number of entries in the cc_tab[] table/

[VR] Done.

> (3) Section 3.6.  Multiple C code fragments are used in the text.  Somewhere the text should state that the examples are C code and made a reference to what version -- consider C99 (ISO/IEC 9899:1999), C11 (ISO/IEC 9899:2011), C18 (ISO/IEC 9899:2018).

[VR] Yes. I added the same C99 normative reference as in the TinyMT32 I-D
and included some text to clearly state it. I did it for both Sections, 3.5
and 3.6.

NEW (3.6):
   Figure 3 shows the reference generate_coding_coefficients()
   implementation.  This is a C language implementation, written for C99

> (4) Other References Nits
> Section 1.  Please include a reference for FECFRAME on first introduction.

[VR] Done.

> Section 1.1.  What is the reference for Raptor/RaptorQ?

[VR] Added RFC6681.

> (6) Editorial Nits
> Abstract.  No references are permitted in the abstract

[VR] Removed.

> Section 1.1.  Multiple Typos.  s/either a end-/either an end-/g
> Section 1.2.  Extra space. s/constraints) ./constraints)./
> Section 1.4.  Multiple Typos.  s/signalling/signaling/g
> Section 6.2.  Typo.  s/Section Section 3.6/Section 3.6/

[VR] Done. Thanks.

[Roman] Thank you for making these changes.  They address all of my feedback.