[payload] Review of draft-ietf-payload-rtp-klv-00

"Ali C. Begen (abegen)" <abegen@cisco.com> Sun, 05 June 2011 00:04 UTC

Return-Path: <abegen@cisco.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 5055D11E8072 for <payload@ietfa.amsl.com>; Sat, 4 Jun 2011 17:04:46 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8
X-Spam-Level:
X-Spam-Status: No, score=-8 tagged_above=-999 required=5 tests=[RCVD_IN_DNSWL_HI=-8]
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 g7kxyZOiSNo7 for <payload@ietfa.amsl.com>; Sat, 4 Jun 2011 17:04:45 -0700 (PDT)
Received: from sj-iport-2.cisco.com (sj-iport-2.cisco.com [171.71.176.71]) by ietfa.amsl.com (Postfix) with ESMTP id B265711E8071 for <payload@ietf.org>; Sat, 4 Jun 2011 17:04:45 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=abegen@cisco.com; l=2108; q=dns/txt; s=iport; t=1307232285; x=1308441885; h=mime-version:content-transfer-encoding:subject:date: message-id:from:to:cc; bh=j72zvSIQwI08s+GJ1XgH1Od6b3L4whjl5wbf811MB0U=; b=OAQ/n7z2vXfhHzCpgOtEDJ5TtnETcxOLw/CS76eSiU/Udf6Zg1YGkMvQ htTX39otWVO7W2WRaMSKOUGy9qDbjPq6XX0L/AlS51PHXikX4coAzzyXu XXo2QN9UCpsNtrGEtTFMOB/moYuZXqHuRtTpWpRKlYO5q1FTQqIXxzIXm Q=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: At8GAIzH6k2rRDoG/2dsb2JhbABTmCiOH3epP5xphiEEhnSOVosJ
X-IronPort-AV: E=Sophos;i="4.65,321,1304294400"; d="scan'208";a="370385812"
Received: from mtv-core-1.cisco.com ([171.68.58.6]) by sj-iport-2.cisco.com with ESMTP; 05 Jun 2011 00:04:45 +0000
Received: from xbh-sjc-211.amer.cisco.com (xbh-sjc-211.cisco.com [171.70.151.144]) by mtv-core-1.cisco.com (8.14.3/8.14.3) with ESMTP id p5504jxq002719; Sun, 5 Jun 2011 00:04:45 GMT
Received: from xmb-sjc-215.amer.cisco.com ([171.70.151.169]) by xbh-sjc-211.amer.cisco.com with Microsoft SMTPSVC(6.0.3790.4675); Sat, 4 Jun 2011 17:04:44 -0700
X-MimeOLE: Produced By Microsoft Exchange V6.5
Content-class: urn:content-classes:message
MIME-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Date: Sat, 04 Jun 2011 17:04:42 -0700
Message-ID: <04CAD96D4C5A3D48B1919248A8FE0D540F2E059C@xmb-sjc-215.amer.cisco.com>
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
Thread-Topic: Review of draft-ietf-payload-rtp-klv-00
Thread-Index: AcwjC+MjCxKm81W+S3unfbUVm5PvCA==
From: "Ali C. Begen (abegen)" <abegen@cisco.com>
To: payload@ietf.org
X-OriginalArrivalTime: 05 Jun 2011 00:04:44.0913 (UTC) FILETIME=[2D0E4610:01CC2314]
Cc: draft-ietf-payload-rtp-klv@tools.ietf.org
Subject: [payload] Review of draft-ietf-payload-rtp-klv-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: Sun, 05 Jun 2011 00:04:46 -0000

Section 3: 
- I suppose UL stands for Universal Label. Maybe define the acronym when "Universal Label" first appears.
- s/ "K L V"/"KLV"

Section 4.3.1.1:
- In the figure, "for time 30" was repeated twice. 
- The first rule seems to be underspecified. What if seq=3 had a marker set to 1, and seq=4 and 5 had a marker set to 0? Both 4 and 5 would be considered damaged, right? So, it is not just the prior packet. And in this case, SHOULD must be a MUST, right? The SHOULD in the 2nd rule seems correct, though.
- Consider replacing "may" with an alternative like "can" or "might". (You might wanna consider this for any lower-case version of 2119 keywords)

Section 6.1:
- Put a note for RFC editor to replace RFCXXXX with this doc's RFC number.
- Put a "None" in optional parameters.
- Are the target applications really streaming and conferencing tools? Any broadcast, video transport application targeted?
- Add "IETF Payload Working Group" for further contact. Also change the control from AVT to PAYLOAD.
- I believe you ought to repeat the registration for video, audio and text as well, although you do not need them.

Section 6.2: 
- This payload format does not have optional parameters. Actually it does not have anything besides the rate. Maybe, you should get the required text from 4855 and put it here (You don't need all the parts). 
- You need to have a section here called "Offer-Answer Model and Declarative Considerations". You might not have any such considerations, but you need to say it explicitly. Probably something like below would work:

   There no configuration parameters or optional format parameters for
   the this payload format.  Thus, when offering SMPTE 336M Encoded Data
   over RTP using SDP in an Offer/Answer model [RFC3264] or in
   a declarative manner (e.g., SDP in the Real-time Streaming Protocol
   (RTSP) [RFC2326] or the Session Announcement Protocol (SAP)
   [RFC2974]), there are no specific considerations.

(Add the related RFCs to the informative part)

-acbegen