Re: [Gen-art] Genart last call review of draft-ietf-payload-rtp-ttml-02

James Sandford <james.sandford@bbc.co.uk> Tue, 08 October 2019 10:24 UTC

Return-Path: <james.sandford@bbc.co.uk>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 8C7FC1200B8; Tue, 8 Oct 2019 03:24:32 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.2
X-Spam-Level:
X-Spam-Status: No, score=-4.2 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
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 7506s0vods_U; Tue, 8 Oct 2019 03:24:30 -0700 (PDT)
Received: from mailout1.cwwtf.bbc.co.uk (mailout1.cwwtf.bbc.co.uk [132.185.160.180]) (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 6A24A120169; Tue, 8 Oct 2019 03:24:30 -0700 (PDT)
Received: from BGB01XI1008.national.core.bbc.co.uk (bgb01xi1008.national.core.bbc.co.uk [10.161.14.22]) by mailout1.cwwtf.bbc.co.uk (8.15.2/8.15.2) with ESMTP id x98AOSoL007599; Tue, 8 Oct 2019 11:24:28 +0100 (BST)
Received: from BGB01XUD1001.national.core.bbc.co.uk ([10.184.52.80]) by BGB01XI1008.national.core.bbc.co.uk ([10.161.14.22]) with mapi id 14.03.0408.000; Tue, 8 Oct 2019 11:24:28 +0100
From: James Sandford <james.sandford@bbc.co.uk>
To: Russ Housley <housley@vigilsec.com>, "gen-art@ietf.org" <gen-art@ietf.org>
CC: "ietf@ietf.org" <ietf@ietf.org>, "avt@ietf.org" <avt@ietf.org>, "draft-ietf-payload-rtp-ttml.all@ietf.org" <draft-ietf-payload-rtp-ttml.all@ietf.org>
Thread-Topic: Genart last call review of draft-ietf-payload-rtp-ttml-02
Thread-Index: AQHVdXHh4NeQyOOEuUO8IHChUjOhfadQmbI3
Date: Tue, 08 Oct 2019 10:24:28 +0000
Message-ID: <734752AF0E88364D983373FE5CEFED5770DE00B7@bgb01xud1001>
References: <156961600000.25061.6985668960752306671@ietfa.amsl.com>
In-Reply-To: <156961600000.25061.6985668960752306671@ietfa.amsl.com>
Accept-Language: en-GB, en-US
Content-Language: en-GB
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [132.185.132.13]
x-exclaimer-md-config: c91d45b2-6e10-4209-9543-d9970fac71b7
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/t-Ra5XAh9c_7TCzl0mFsJvSkOJs>
Subject: Re: [Gen-art] Genart last call review of draft-ietf-payload-rtp-ttml-02
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 08 Oct 2019 10:24:32 -0000

Hello,
I've uploaded version 03 which addresses these comments and minor comments from the AD (Barry) and Document Shepherd (Roni). 

https://datatracker.ietf.org/doc/draft-ietf-payload-rtp-ttml/

Regards,
James

==========
James Sandford
R&D Project Engineer

BBC Research and Development
5th Floor
Dock House
MediaCityUK
Salford
M50 2LH

Tel: 030304 (09549)
Web: http://www.bbc.co.uk/rd

________________________________________
From: Russ Housley via Datatracker [noreply@ietf.org]
Sent: 27 September 2019 21:26
To: gen-art@ietf.org
Cc: ietf@ietf.org; avt@ietf.org; draft-ietf-payload-rtp-ttml.all@ietf.org
Subject: Genart last call review of draft-ietf-payload-rtp-ttml-02

Reviewer: Russ Housley
Review result: Ready with Issues

I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at
<http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Document: draft-ietf-payload-rtp-ttml-02
Reviewer: Russ Housley
Review Date: 2019-09-27
IETF LC End Date: 2019-10-10
IESG Telechat date: Unknown

Summary: Ready with Issues

Major Concerns:

Section 6 says:

   ...  An additional requirement if best-effort service is being
   used is users of this payload format MUST monitor packet loss to
   ensure that the packet loss rate is within acceptable parameters.

This MUST statement is very vague.  What does an implementer do?  Is
RFC 8083 (which is referenced in the following paragraph) the only way
to meet this MUST statement?  If so, please be very specific.

Please review Section 7.1; I suspect that RFC 2119 language is intended.


Minor Concerns:

Section 2 says:

   Unless otherwise stated, the term "document" is used in this draft to
   refer to the TTML document being transmitted in the payload of the
   RTP packet(s).

Please consider what this will say when it becomes an RFC.  The use of
"draft" will no longer be appropriate, and the use of "document" would
result in a very difficult sentence.  I propose:

   Unless otherwise stated, the term "document" refers to the TTML
   document being transmitted in the RTP payload.

Section 2 also says:

   Where the term "word" is used in this draft, it is to refer to byte
   aligned or 32-bit aligned words of data in a computing sense and not
   to refer to linguistic words that might appear in the transported
   text.

Again, "draft" is not appropriate once this becomes an RFC.  I suggest:

   The term "word" refers to byte aligned or 32-bit aligned words of
   data; it does not refer to linguistic words that might appear in the
   TTML document.

Section 2: Your reference to BCP 14 does not include "NOT RECOMMENDED".
Please use:

   The key words "MUST", "MUST NOT", "REQUIRED", "SHALL", "SHALL NOT",
   "SHOULD", "SHOULD NOT", "RECOMMENDED", "NOT RECOMMENDED", "MAY", and
   "OPTIONAL" in this document are to be interpreted as described in
   BCP 14 [RFC2119] [RFC8174] when, and only when, they appear in all
   capitals, as shown here.

Section 4.1 says:

   User Data Words: integer number of data words

I suspect that negative integers and zero are not allowed.

Section 4.2.1.2.1.3 says:

   A processor profile (X) is compatible with the processor profile in
   this document (P) if X includes all the features and extensions in P,
   identified by their character content, and the "value" attribute of
   each is at least as restrictive as the "value" attribute of the
   feature or extension in P that has the same character content.  The
   term "restrictive" here is as defined in [TTML2] Section 6.

The use of "document" does not follow the discussion in Section 2.
I suggest:

   A given processor profile is compatible with the processor profile
   specified here if the given profile includes all the features and
   extensions, identified by their character content, and the "value"
   attribute of each is at least as restrictive as the "value" attribute
   of the feature or extension specified here using the same character
   content.  The term "restrictive" here is as defined in Section 6
   of [TTML2].

Nits:

Section 6 says:

   Congestion control for RTP SHALL be used in accordance with RFC 3550
   [RFC3550], and with any applicable RTP profile: e.g., RFC 3551
   [RFC3551].

I suggest the elimination of some redundancy:

   Congestion control for RTP SHALL be used in accordance with [RFC3550],
   and with any applicable RTP profile, such as [RFC3551].

Section 7.2: s/Section 3 of RFC 4855 [RFC4855]/Section 3 of [RFC4855]/