[Gen-art] Gen-ART review of draft-ietf-avt-rtp-jpeg2000-beam-09.txt

Black_David@emc.com Sat, 15 March 2008 04:38 UTC

Return-Path: <gen-art-bounces@ietf.org>
X-Original-To: ietfarch-gen-art-archive@core3.amsl.com
Delivered-To: ietfarch-gen-art-archive@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 8184A3A6D7D; Fri, 14 Mar 2008 21:38:08 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.108
X-Spam-Level:
X-Spam-Status: No, score=-102.108 tagged_above=-999 required=5 tests=[AWL=-1.671, BAYES_00=-2.599, FH_RELAY_NODNS=1.451, HELO_MISMATCH_ORG=0.611, RDNS_NONE=0.1, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 0AIUvc5iJe0L; Fri, 14 Mar 2008 21:38:07 -0700 (PDT)
Received: from core3.amsl.com (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id A36EF3A6885; Fri, 14 Mar 2008 21:38:07 -0700 (PDT)
X-Original-To: gen-art@core3.amsl.com
Delivered-To: gen-art@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 436DA3A6ABF; Fri, 14 Mar 2008 21:38:07 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id TxIEW08OWjKK; Fri, 14 Mar 2008 21:38:06 -0700 (PDT)
Received: from mexforward.lss.emc.com (mexforward.lss.emc.com [128.222.32.20]) by core3.amsl.com (Postfix) with ESMTP id CD9033A6885; Fri, 14 Mar 2008 21:38:05 -0700 (PDT)
Received: from hop04-l1d11-si01.isus.emc.com (HOP04-L1D11-SI01.isus.emc.com [10.254.111.54]) by mexforward.lss.emc.com (Switch-3.2.5/Switch-3.1.7) with ESMTP id m2F4ZiMJ023993; Sat, 15 Mar 2008 00:35:44 -0400 (EDT)
Received: from mailhub.lss.emc.com (uraeus.lss.emc.com [10.254.144.14]) by hop04-l1d11-si01.isus.emc.com (Tablus Interceptor); Sat, 15 Mar 2008 00:35:43 -0400
Received: from corpussmtp3.corp.emc.com (corpussmtp3.corp.emc.com [10.254.64.53]) by mailhub.lss.emc.com (Switch-3.2.5/Switch-3.1.7) with ESMTP id m2F4Za2B012639; Sat, 15 Mar 2008 00:35:36 -0400 (EDT)
From: Black_David@emc.com
Received: from CORPUSMX20A.corp.emc.com ([128.221.62.11]) by corpussmtp3.corp.emc.com with Microsoft SMTPSVC(6.0.3790.1830); Sat, 15 Mar 2008 00:35:36 -0400
X-MimeOLE: Produced By Microsoft Exchange V6.5
Content-class: urn:content-classes:message
MIME-Version: 1.0
Date: Sat, 15 Mar 2008 00:35:35 -0400
Message-ID: <8CC6CEAB44F131478D3A7B429ECACD91923940@CORPUSMX20A.corp.emc.com>
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
Thread-Topic: Gen-ART review of draft-ietf-avt-rtp-jpeg2000-beam-09.txt
Thread-Index: AciGVgLfCCF5+tgpQa6jUPcq6v/yrw==
To: gen-art@ietf.org, andrew@ualberta.net, satosi-f@sm.sony.co.jp, itakura@sm.sony.co.jp, ietf@ietf.org
X-OriginalArrivalTime: 15 Mar 2008 04:35:36.0104 (UTC) FILETIME=[03517A80:01C88656]
X-PMX-Version: 4.7.1.128075, Antispam-Engine: 2.5.1.298604, Antispam-Data: 2007.8.30.51425
X-PerlMx-Spam: Gauge=, SPAM=0%, Reason='EMC_BODY_1+ -3, EMC_FROM_0+ -3, NO_REAL_NAME 0, __C230066_P5 0, __CP_MEDIA_BODY 0, __CP_URI_IN_BODY 0, __CT 0, __CTE 0, __CT_TEXT_PLAIN 0, __HAS_MSGID 0, __IMS_MSGID 0, __MIME_TEXT_ONLY 0, __MIME_VERSION 0, __SANE_MSGID 0'
X-Tablus-Inspected: yes
X-Tablus-Classifications: public
X-Tablus-Action: allow
Cc: fluffy@cisco.com, tom.taylor@rogers.com, roni.even@polycom.co.il, Black_David@emc.com, csp@csperkins.org
Subject: [Gen-art] Gen-ART review of draft-ietf-avt-rtp-jpeg2000-beam-09.txt
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/pipermail/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>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Sender: gen-art-bounces@ietf.org
Errors-To: gen-art-bounces@ietf.org

I have been selected as the General Area Review Team (Gen-ART) 
reviewer for this draft (for background on Gen-ART, please see 
http://www.alvestrand.no/ietf/gen/art/gen-art-FAQ.html). 
Please resolve these comments along with any other Last Call comments 
you may receive. 

Document: draft-ietf-avt-rtp-jpeg2000-beam-09.txt
Reviewer: David Black
Review Date: 15 March 2008
IETF LC End Date: 15 March 2008

Summary:

This draft is on the right track, but has open issues,
described in the review.

Comments:

There are two open issues, both of which should be fairly
easy to address:

[1] Section 3.2 doesn't provide enough information to calculate
a packet priority value from layer, resolution and component
values.  In fact the example it gives appears to be simple
enough to also be an example of the component based ordering
defined in Section 3.5.  Section 3.2 needs to explain
how the priority value is calculated and use a more complex
example to illustrate the results of the calculation.

[2] Section 4.1 contains this problematic text:

   An initial value of mh_id MUST be selected randomly between 1 and 7
   for security reasons.

This needs to be explained, and "security" is probably the wrong
word.  Any security that depends on this random choice is likely
to be easily subverted by the adversary trying all 7 possible values.

Nits:

As noted by IANA, Section 7.2 doesn't contain any IANA actions
and hence probably should not be a subsection of the IANA
Considerations section (7)

Reference [1] should reference the Internet Draft by name.
   [1]  Futemma, "RTP Payload Format for JPEG 2000 Video Streams",
        RFC XXXY, April 2007.
I believe this is draft-ietf-avt-rtp-jpeg2000-18.txt.  That
should be in the reference instead of RFC XXXY.  Then add
an RFC Editor note asking the RFC Editor to replace all instances
of RFC XXXY with the RFC number assigned when reference [1]
is published as an RFC.

idnits 2.08.04 found this problem with reference [1] and was
confused by reference [3].  Reference [3] is fine as-is; no
change is needed.

Thanks,
--David
----------------------------------------------------
David L. Black, Distinguished Engineer
EMC Corporation, 176 South St., Hopkinton, MA  01748
+1 (508) 293-7953             FAX: +1 (508) 293-7786
black_david@emc.com        Mobile: +1 (978) 394-7754
----------------------------------------------------

_______________________________________________
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art