TSV review of draft-ietf-sipping-rtcp-summary-08
Colin Perkins <csp@csperkins.org> Thu, 18 February 2010 00:08 UTC
Return-Path: <csp@csperkins.org>
X-Original-To: ietf@core3.amsl.com
Delivered-To: ietf@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 09F2C3A7F40; Wed, 17 Feb 2010 16:08:47 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.599
X-Spam-Level:
X-Spam-Status: No, score=-3.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, RCVD_IN_DNSWL_LOW=-1]
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 ieINWnlt8wWr; Wed, 17 Feb 2010 16:08:45 -0800 (PST)
Received: from lon1-msapost-3.mail.demon.net (lon1-msapost-3.mail.demon.net [195.173.77.182]) by core3.amsl.com (Postfix) with ESMTP id A45DB3A7DE3; Wed, 17 Feb 2010 16:08:45 -0800 (PST)
Received: from starkperkins.demon.co.uk ([80.176.158.71] helo=[192.168.0.14]) by lon1-post-3.mail.demon.net with esmtpsa (AUTH csperkins-dwh) (TLSv1:AES128-SHA:128) (Exim 4.69) id 1NhtyL-00018L-dY; Thu, 18 Feb 2010 00:10:25 +0000
Message-Id: <E684BC5F-CCC6-4813-A2D9-54BC08CF91D2@csperkins.org>
From: Colin Perkins <csp@csperkins.org>
To: IETF Discussion Mailing List <ietf@ietf.org>
Content-Type: text/plain; charset="US-ASCII"; format="flowed"; delsp="yes"
Content-Transfer-Encoding: 7bit
Mime-Version: 1.0 (Apple Message framework v936)
Subject: TSV review of draft-ietf-sipping-rtcp-summary-08
Date: Thu, 18 Feb 2010 00:10:23 +0000
X-Mailer: Apple Mail (2.936)
Cc: aspen@telchemy.com, Magnus Westerlund <magnus.westerlund@ericsson.com>, IETF AVT WG <avt@ietf.org>, Alan Clark <alan.d.clark@telchemy.com>, TSV Dir <tsv-dir@ietf.org>, henrys@adobe.com
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ietf>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 18 Feb 2010 00:08:47 -0000
I've reviewed this document as part of the transport area directorate's ongoing effort to review key IETF documents. These comments were written primarily for the transport area directors, but are copied to the document's authors for their information and to allow them to address any issues raised. The authors should consider this review together with any other last-call comments they receive. Please always CC tsv-dir@ietf.org if you reply to or forward this review. This document defines a SIP event package to enable reporting some RTCP XR metrics, in particular those deemed relevant to some classes of VoIP application. The metrics reported are a subset of those defined in RFC 3611, plus some extensions. This is a potentially useful specification, but I believe there are a number of problems with the way it interacts with RTP which I recommend be addressed before publication. The applicability statement (section 1.1) references the list of RTP topologies in RFC 5117, and states that only the first (point-to- point) sessions is relevant to this document. The Usage Scenarios (section 1.2) and the rest of the document contradict this, discussing point-to-point sessions, conference calls using a central conferencing server, and multicast VoIP calls using SSM. RTP is inherently a group communication protocol, and this document needs to properly consider the full range of topologies defined in RFC 5117. A large number of parameters are defined in Section 4.6: The PayloadType parameter states that "IANA registered values SHOULD be used where possible", but the IANA registry only applies to the RTP/ AVP profile, and has been closed to new registrations for several years now. The implication of this document is that the payload type is sufficient to identify the codec, but this is not the case, and for this reason RFC 3551 recommends the use of media subtype names instead. The PayloadDesc parameter is defined to provide a text description for the codec. It is stated that "The abbreviated standard name for the codec SHOULD be used (for example "G.729A")". The IETF has a standard namespace for codecs, in the form of media subtypes, registered according to RFC 4855. I believe it would be a mistake to introduce another, ill-defined, namespace in this document, rather than using the standard media type names as used for all other RTP-related documents. Several parameters are defined relating to frame-based codecs (e.g. FrameDuration, FrameOctets, and FramesPerPacket). However, not all RTP audio payload types are frame based (see RFC 3551, section 4.3). It is not clear how sample based codecs are handled. The FmtpOptions parameter is defined to convey "FMTP options from SDP". These options are parameters of media subtypes, and don't make sense unless the PayloadDesc is specified as a media subtype (and don't make any sense if just PayloadType is specified, since they're related to a media subtype name, not a payload type). The MetricType parameter is ill-defined. What do the values specified refer to, and how are new values to be registered? The LocalAddr and RemoteAddr parameters assume that STUN is used as a NAT traversal mechanism, if one is needed. While I agree with the principle of reporting the address on the "outside" of the NAT, STUN is not always available. Section 4.6.2.9.5 expresses interarrival jitter as defined in RFC 3550 in milliseconds, but RFC 3550 specifies it in RTP timestamp units. Nit: In the last sentence of the introduction, "Real-Time Control Protocol" should be "RTP Control Protocol". -- Colin Perkins http://csperkins.org/
- TSV review of draft-ietf-sipping-rtcp-summary-08 Colin Perkins