Re: [AVTCORE] AD Evaluation of draft-ietf-avtcore-rtp-multi-stream-optimisation-08

Colin Perkins <csp@csperkins.org> Wed, 11 November 2015 15:18 UTC

Return-Path: <csp@csperkins.org>
X-Original-To: avt@ietfa.amsl.com
Delivered-To: avt@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 75BA01B29BA; Wed, 11 Nov 2015 07:18:28 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.9
X-Spam-Level:
X-Spam-Status: No, score=-1.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9] autolearn=ham
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id Lhs3QTrpIUcC; Wed, 11 Nov 2015 07:18:27 -0800 (PST)
Received: from balrog.mythic-beasts.com (balrog.mythic-beasts.com [IPv6:2a00:1098:0:82:1000:0:2:1]) (using TLSv1.2 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id E9C5A1AD49D; Wed, 11 Nov 2015 07:18:26 -0800 (PST)
Received: from [130.209.247.112] (port=64417 helo=mangole.dcs.gla.ac.uk) by balrog.mythic-beasts.com with esmtpsa (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.80) (envelope-from <csp@csperkins.org>) id 1ZwXAG-0002gv-7i; Wed, 11 Nov 2015 15:18:25 +0000
Content-Type: text/plain; charset="utf-8"
Mime-Version: 1.0 (Mac OS X Mail 8.2 \(2104\))
From: Colin Perkins <csp@csperkins.org>
In-Reply-To: <0FBEEF14-CA50-4C3A-815F-303F19A060B5@nostrum.com>
Date: Wed, 11 Nov 2015 15:18:22 +0000
Content-Transfer-Encoding: quoted-printable
Message-Id: <FC3911E7-66E5-414C-A460-9C7F0AEAA2B1@csperkins.org>
References: <0FBEEF14-CA50-4C3A-815F-303F19A060B5@nostrum.com>
To: Ben Campbell <ben@nostrum.com>
X-Mailer: Apple Mail (2.2104)
X-BlackCat-Spam-Score: -28
X-Mythic-Debug: Threshold = On =
Archived-At: <http://mailarchive.ietf.org/arch/msg/avt/5vivqUGo134cqp_XqZQyPor9kx0>
Cc: draft-ietf-avtcore-rtp-multi-stream-optimisation.all@ietf.org, "avt@ietf.org WG" <avt@ietf.org>
Subject: Re: [AVTCORE] AD Evaluation of draft-ietf-avtcore-rtp-multi-stream-optimisation-08
X-BeenThere: avt@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: Audio/Video Transport Core Maintenance <avt.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/avt>, <mailto:avt-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/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, 11 Nov 2015 15:18:28 -0000

> On 9 Nov 2015, at 21:48, Ben Campbell <ben@nostrum.com> wrote:
> 
> Hi,
> 
> Here is my AD Evaluation of draft-ietf-avtcore-rtp-multi-stream-optimisation-08:
> 
> Ben.
> 
> -----------
> 
> Substantive Comments:
> =====================
> 
> -3.1, 4th paragraph:
> 
> Is “SHOULD NOT…unless” the actual intent? That means you shouldn’t do it unless that condition occurs, or you have some other good reason and understand the consequences. (as compared to “MUST NOT … unless")

It’s trying to say don’t create a reporting group with only a single local SSRC, unless you expect to add more SSRCs to it later. Suggestions for better phrasing welcome.

> -4.2, 1st paragraph:
> 
> Does this refer to third-party observers? (That is, observers not involved in the signaling of support for rtcp-rgrp?

That’s the most likely example, yes.

> -7.2, reference to RFC3264:
> 
> Should this be a normative reference? I’m not sure 3.6 will make much sense without it.

It’s normative if you use SDP offer/answer, but not for declarative SDP. Happy to move it to a normative reference.

> Editorial Comments:
> ===================
> 
> -3.1, 5th paragraph:
> 
> Option "A" is hard to parse. I suggest:
> 
> “... if another reporting source exists,  have it report on the remote SSRCs that the departing source reported ..."

Okay, although I’m not sure it scans so clearly.

> - 3.2.1, 2nd paragraph:
> 
> Consider active voice for the first sentence, i.e. "This document defines..."
> 
> - 3.2.2, 1st paragraph:
> 
> Consider active voice for the first sentence, i.e. “This document defines..."

Fine.

> - 3.6, 2nd to last paragraph: "... neither agents SHALL use..."
> 
> Please restate as SHALL NOT/MUST NOT.

Is it sufficient to change this to “If the answerer does not wish to use RTCP Reporting Groups, it MUST NOT include the “a=rtcp-rgrp” in the answer.”?

> -4.1, last paragraph: "approximately 8.9"
> 
> That's a rather precise approximation :-)

We like precision :-)

-- 
Colin Perkins
https://csperkins.org/