[payload] AD review: draft-ietf-payload-rfc3016bis-00

Robert Sparks <rjsparks@nostrum.com> Wed, 18 May 2011 16:08 UTC

Return-Path: <rjsparks@nostrum.com>
X-Original-To: payload@ietfa.amsl.com
Delivered-To: payload@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 13267E067E for <payload@ietfa.amsl.com>; Wed, 18 May 2011 09:08:42 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.562
X-Spam-Level:
X-Spam-Status: No, score=-102.562 tagged_above=-999 required=5 tests=[AWL=0.038, BAYES_00=-2.599, SPF_PASS=-0.001, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([64.170.98.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id rzt2nUsp+0de for <payload@ietfa.amsl.com>; Wed, 18 May 2011 09:08:41 -0700 (PDT)
Received: from nostrum.com (nostrum-pt.tunnel.tserv2.fmt.ipv6.he.net [IPv6:2001:470:1f03:267::2]) by ietfa.amsl.com (Postfix) with ESMTP id 043D3E0665 for <payload@ietf.org>; Wed, 18 May 2011 09:08:40 -0700 (PDT)
Received: from dn3-177.estacado.net (vicuna-alt.estacado.net [75.53.54.121]) (authenticated bits=0) by nostrum.com (8.14.3/8.14.3) with ESMTP id p4IG8a4S009549 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NO); Wed, 18 May 2011 11:08:36 -0500 (CDT) (envelope-from rjsparks@nostrum.com)
From: Robert Sparks <rjsparks@nostrum.com>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Date: Wed, 18 May 2011 11:08:37 -0500
Message-Id: <E6F93BBE-FA6D-4FD6-A187-A935222D9AB9@nostrum.com>
To: payload@ietf.org, draft-ietf-payload-rfc3016bis-00@tools.ietf.org
Mime-Version: 1.0 (Apple Message framework v1084)
X-Mailer: Apple Mail (2.1084)
Received-SPF: pass (nostrum.com: 75.53.54.121 is authenticated by a trusted mechanism)
Subject: [payload] AD review: draft-ietf-payload-rfc3016bis-00
X-BeenThere: payload@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Audio/Video Transport Payloads working group discussion list <payload.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/payload>, <mailto:payload-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/payload>
List-Post: <mailto:payload@ietf.org>
List-Help: <mailto:payload-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/payload>, <mailto:payload-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 18 May 2011 16:08:42 -0000

There are some issues to address with a draft revision before progressing to IETF LC:

Much of this text was from 3016, shouldn't the document be using the
pre-5378 boilerplate (see 6.c.iii of <http://trustee.ietf.org/docs/IETF-Trust-License-Policy.pdf>)?

It is hard to tell what the actual changes from 3016 are (even when using
diff against 3016). The changes section should summarize the technical changes
with a little more detail. For instance, what exactly is the change to the binary
format of the LATM payload? It would also help to summarize where requirements levels
have changed. For instance, noting the change of the MAY to MUST (actually SHALL) in
section 5.3.

The abstract needs to mention that the document is obsoleting 3016.
Can the document explain the limitations the abstract calls out and 
better motivate the preferrence for RFC3640? Why doesn't the abstract
mention the reason for revising 3016?

The document removed the word "new" from the description of some MPEG-4
features, but not all of them. Should it have removed all of them?

Section 1.3 should more explicitly state that the working group believes
there are NO implementations of 3016 that do not already behave as the 
3GPP document (and this document) specify, so there is no need to consider 
backwards compatability. (If that's not true, there are several places where 
backwards compatability needs more consideration.)

The three new definitions in section 2 don't define the terms. Instead
they try to clarify how values are chosen in various circumstances. This
would be much clearer in a short section titled "Clarifications on specifying
sampling rates" or similar.

The change control for the two media types should say PAYLOAD instead of AVT.

The second paragraph of the security considerations section starts by exploring
the existence of covert channels, then shifts to template text on providing
security to RTP. It's not clear how those existing mechanisms help with respect
to the possibility of a covert channel. Was the intent only to note that the
covert channels existed? If so, the paragraph should be split so pointing to
the security mechanisms for RTP doesn't appear to be a response to that issue.

Nits:

Section 1.1, first paragraph: Why did "bitrates" get changed to "bitrate"?
Section 5.2, first paragrah: "enhance layer" should be "enhanced layer"