Re: [AVT] AD review: draft-ietf-avt-rtp-h264-rcdo-06

Tom Kristensen <2mkristensen@gmail.com> Wed, 06 October 2010 14:49 UTC

Return-Path: <2mkristensen@gmail.com>
X-Original-To: avt@core3.amsl.com
Delivered-To: avt@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 3F3523A70E8 for <avt@core3.amsl.com>; Wed, 6 Oct 2010 07:49:17 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.599
X-Spam-Level:
X-Spam-Status: No, score=-2.599 tagged_above=-999 required=5 tests=[AWL=0.000, BAYES_00=-2.599]
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 3zO1fbie0Q-S for <avt@core3.amsl.com>; Wed, 6 Oct 2010 07:49:15 -0700 (PDT)
Received: from mail-qw0-f44.google.com (mail-qw0-f44.google.com [209.85.216.44]) by core3.amsl.com (Postfix) with ESMTP id 8E7043A70E0 for <avt@ietf.org>; Wed, 6 Oct 2010 07:49:15 -0700 (PDT)
Received: by qwc9 with SMTP id 9so5045161qwc.31 for <avt@ietf.org>; Wed, 06 Oct 2010 07:50:15 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:received:in-reply-to :references:date:message-id:subject:from:to:cc:content-type :content-transfer-encoding; bh=YBQ5sNLfdCCDUJgJkEqiLeDH41MDYPEjW0rj2LAaNcc=; b=vZUWgo9eKYrzcsJlFZFZ7j0w7YBE60zGXe4dDjCvTcIBGkr0rCLy4rKjfvRJ7UzTwl 7+1t4nZ8H+1x1FX4Gid/NJkORN4ocdeBPH8SItzwEBDSoDrkgrtbL/To+TDB4x0GTX67 agbK3HdOCkStlKghj6WXZqAMZbvgILkBk4zyo=
DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; b=Fm8FR5Av80enEp4QEcpypWa5gbegi0sTciAH6P4GOhIzSEYu9RcZbfmoZYhO2CH+Gz IVgKGiQ5KPD1rtJ+UZ9z4qG6PgkPOPyFFHC/vwN5u/K/aKxh9FStv2Wws1FTsyFznZAH HxStpoEGJXUXcW1G5Gpcj7OFixKKM3sqlxXK8=
MIME-Version: 1.0
Received: by 10.224.36.132 with SMTP id t4mr1617724qad.72.1286376615267; Wed, 06 Oct 2010 07:50:15 -0700 (PDT)
Received: by 10.229.30.212 with HTTP; Wed, 6 Oct 2010 07:50:15 -0700 (PDT)
In-Reply-To: <4CA9AA1D.6030508@ericsson.com>
References: <4CA9AA1D.6030508@ericsson.com>
Date: Wed, 06 Oct 2010 16:50:15 +0200
Message-ID: <AANLkTi=mmyPZ-ao0U-EW8OvQKQoCJDWuSr+ZfNhQRL4M@mail.gmail.com>
From: Tom Kristensen <2mkristensen@gmail.com>
To: Gonzalo Camarillo <Gonzalo.Camarillo@ericsson.com>
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: quoted-printable
Cc: avt@ietf.org, Tom Kristensen <tomkrist@cisco.com>
Subject: Re: [AVT] AD review: draft-ietf-avt-rtp-h264-rcdo-06
X-BeenThere: avt@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: Audio/Video Transport Working Group <avt.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/avt>, <mailto:avt-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/avt>
List-Post: <mailto:avt@ietf.org>
List-Help: <mailto:avt-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/avt>, <mailto:avt-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 06 Oct 2010 14:49:17 -0000

On 4 October 2010 12:19, Gonzalo Camarillo
<Gonzalo.Camarillo@ericsson.com> wrote:
> Hi,
>
> as part of a load balancing exercise between Robert and I, I will be
> acting as the responsible AD for this draft.
>
> https://datatracker.ietf.org/doc/draft-ietf-avt-rtp-h264-rcdo/
>
> Below you can find my AD review. As soon as the authors address these
> comments (it should be trivial to address them), I will IETF Last Call
> this draft.

Great, thanks! I just submitted a new version addressing these
comments as detailed below.

[...]
> The acronyms RTP and RCDO need to be expanded in the title of the
> draft. In general, all acronyms need to be expanded on their first
> use.

RTP kept as is, cf. answer to SVC draft on this list. Other "not
well-known" acronyms are now expanded.

> The document refers to [3] as RFC3984bis. Given that [3] is a
> normative reference, this document will not be published until [3] is
> published as well. So, referring to it as, say, RFCYYYY and then get
> the RFC Editor to update the text seems like a better option than
> calling it RFC3984bis.

Done. We also have to remember syncing the identical parts of the
media subtype definition with draft-ietf-avt-rtp-rfc3984bis-11 when
it's 100% sure this will not change.

> Page 19: section X should be section 9.

Fixed.

> Section 9: "although if suitable the usage of SRTP [11] is
> recommended". Should that be a normative RECOMMENDED instead?

Not so sure, so in spirit of draft-ietf-avt-srtp-not-mandatory the
following sentence is used instead:

"Usage of data origin authentication and data integrity protection of
at least the RTP packet is RECOMMENDED, for example by use of the
Secure Real-time Transport Protocol (SRTP) <xref target="RFC3711"/>."

> The draft should have a reference to the SDP spec: RFC 4566.

Yes, now referenced in the Introduction section.

-- Tom

-- 
# Cisco
## tomkrist@cisco.com  |  http://www.tandberg.com
### http://folk.uio.no/tomkri/