IETF Logo Mail Archive

[MMUSIC] SDP Directorate: Review of draft-ietf-xrblock-rtcp-xr-loss-conceal-05

"Ali C. Begen (abegen)" <abegen@cisco.com> Mon, 03 June 2013 10:59 UTC

Return-Path: <abegen@cisco.com>
X-Original-To: mmusic@ietfa.amsl.com
Delivered-To: mmusic@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 8195321F9455 for <mmusic@ietfa.amsl.com>; Mon, 3 Jun 2013 03:59:25 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -10.599
X-Spam-Level:
X-Spam-Status: No, score=-10.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, RCVD_IN_DNSWL_HI=-8]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id HYaDnNFXdGfa for <mmusic@ietfa.amsl.com>; Mon, 3 Jun 2013 03:59:18 -0700 (PDT)
Received: from rcdn-iport-4.cisco.com (rcdn-iport-4.cisco.com [173.37.86.75]) by ietfa.amsl.com (Postfix) with ESMTP id 0409621F93BA for <mmusic@ietf.org>; Mon, 3 Jun 2013 03:59:16 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=1249; q=dns/txt; s=iport; t=1370257157; x=1371466757; h=from:to:cc:subject:date:message-id:content-id: content-transfer-encoding:mime-version; bh=myWBsqCEYmKMsQ7itZornkwpE/RsVng13bubOP7S72s=; b=ku1Vlxvpce3+la4UvvtJCkuDwSMjpJTf5tSRCs988KpjSEVD71sC0mBH I2d7KZmwZf8JjttG3fvVqjuxkY971opJfx8jx83CxgvFG9vaJ/UQOaC5s 033Gjt3U/7gWkDM3qf3UlsWN5EMUEaRp8lQZG/Sga7R8uMpU7dCuAChbo Y=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: AusFAJV2rFGtJXHB/2dsb2JhbABZgmghML8EfxZtB4IlAQQ6PxIBKhRCJwQODQGIBAELrT2NLo52MQKCfGEDqH6DD4In
X-IronPort-AV: E=Sophos;i="4.87,792,1363132800"; d="scan'208";a="218090379"
Received: from rcdn-core2-6.cisco.com ([173.37.113.193]) by rcdn-iport-4.cisco.com with ESMTP; 03 Jun 2013 10:58:59 +0000
Received: from xhc-rcd-x02.cisco.com (xhc-rcd-x02.cisco.com [173.37.183.76]) by rcdn-core2-6.cisco.com (8.14.5/8.14.5) with ESMTP id r53AwxmQ003819 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=FAIL); Mon, 3 Jun 2013 10:58:59 GMT
Received: from xmb-aln-x01.cisco.com ([fe80::747b:83e1:9755:d453]) by xhc-rcd-x02.cisco.com ([173.37.183.76]) with mapi id 14.02.0318.004; Mon, 3 Jun 2013 05:58:59 -0500
From: "Ali C. Begen (abegen)" <abegen@cisco.com>
To: "draft-ietf-xrblock-rtcp-xr-loss-conceal@tools.ietf.org" <draft-ietf-xrblock-rtcp-xr-loss-conceal@tools.ietf.org>
Thread-Topic: SDP Directorate: Review of draft-ietf-xrblock-rtcp-xr-loss-conceal-05
Thread-Index: AQHOYElZgyXNkt/U/0+WIwSPFpWz0g==
Date: Mon, 03 Jun 2013 10:58:58 +0000
Message-ID: <C15918F2FCDA0243A7C919DA7C4BE9940D1076B4@xmb-aln-x01.cisco.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.86.240.24]
Content-Type: text/plain; charset="us-ascii"
Content-ID: <56803ABD1204DF4AB58427CE3A1CB8D7@emea.cisco.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Cc: xrblock-chairs <xrblock-chairs@tools.ietf.org>, mmusic <mmusic@ietf.org>
Subject: [MMUSIC] SDP Directorate: Review of draft-ietf-xrblock-rtcp-xr-loss-conceal-05
X-BeenThere: mmusic@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Multiparty Multimedia Session Control Working Group <mmusic.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mmusic>, <mailto:mmusic-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/mmusic>
List-Post: <mailto:mmusic@ietf.org>
List-Help: <mailto:mmusic-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mmusic>, <mailto:mmusic-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 03 Jun 2013 10:59:25 -0000

I am the assigned SDP directorate reviewer for this draft. For background on the SDP directorate, please see the FAQ at
http://www.ietf.org/iesg/directorate/sdp.html

Please wait for direction from your document shepherd or AD before posting a new version of the draft.

Summary:
- The RTCP XR extension syntax seems to have an issue, see below
- There are a few editorial issues

Issues:
1) Section 1.4: Can I use these metrics for the audio component of A/V applications? If yes, then section 1.4 sounds overly restrictive.

2) Capitalization of the block/metric names is inconsistent.

3) Section 5.1, first parag.: The document defines two blocks not one.

4) Section 5.1:

Why are using 

   xr-format =/ xr-conceal-block
   xr-format =/ xr-conc-sec-block

instead of

   xr-format =/ xr-conceal-block / xr-conc-sec-block

5) Section 6.1: Where do "NCS" and "NLC" come from? Why not choose more appropriate acronyms? Also use the correct registry name: "RTP Control Protocol Extended Reports (RTCP XR) Block Type Registry"

6) Section 6.2: Please use the correct registry name: "RTP Control Protocol Extended Reports (RTCP XR) Session Description Protocol (SDP) Parameters Registry".

v2.23.0 | Report a Bug | By Email