[AVTCORE] AD Evaluation of draft-ietf-avtcore-rtp-multi-stream-09

"Ben Campbell" <ben@nostrum.com> Mon, 09 November 2015 21:31 UTC

Return-Path: <ben@nostrum.com>
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 A9BD41B34ED; Mon, 9 Nov 2015 13:31:21 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.91
X-Spam-Level:
X-Spam-Status: No, score=-1.91 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, T_RP_MATCHES_RCVD=-0.01] 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 fklDR4gtV0Ed; Mon, 9 Nov 2015 13:31:20 -0800 (PST)
Received: from nostrum.com (raven-v6.nostrum.com [IPv6:2001:470:d:1130::1]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 900051B3504; Mon, 9 Nov 2015 13:31:20 -0800 (PST)
Received: from [10.0.1.10] (cpe-70-119-203-4.tx.res.rr.com [70.119.203.4]) (authenticated bits=0) by nostrum.com (8.15.2/8.14.9) with ESMTPSA id tA9LTk3i080778 (version=TLSv1 cipher=DHE-RSA-AES128-SHA bits=128 verify=NO); Mon, 9 Nov 2015 15:29:46 -0600 (CST) (envelope-from ben@nostrum.com)
X-Authentication-Warning: raven.nostrum.com: Host cpe-70-119-203-4.tx.res.rr.com [70.119.203.4] claimed to be [10.0.1.10]
From: Ben Campbell <ben@nostrum.com>
To: draft-ietf-avtcore-rtp-multi-stream.all@ietf.org, "avt@ietf.org WG" <avt@ietf.org>
Date: Mon, 09 Nov 2015 15:29:46 -0600
Message-ID: <C22EBF77-96B4-4594-BBAE-0E7865B4E7C3@nostrum.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Transfer-Encoding: 8bit
X-Mailer: MailMate (1.9.3r5164)
Archived-At: <http://mailarchive.ietf.org/arch/msg/avt/J918XNLrpNXTtkrGAkQ9rRFR2cs>
Subject: [AVTCORE] AD Evaluation of draft-ietf-avtcore-rtp-multi-stream-09
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: Mon, 09 Nov 2015 21:31:21 -0000

Hi,

Here is my AD evaluation of draft-ietf-avtcore-rtp-multi-stream-09:

Thanks!

Ben.


Substantive Comments:
=====================

- 3.3, last paragraph: "The endpoint MUST keep its total media sending 
rate within
    this share."

Does that need to be a 2119 "MUST"? Does it create a new normative 
requirement, or restate an existing requirement?

- 5: " ... the specification MUST be interpreted as each SSRC counting 
as a separate participant ..."

Does that really need to be a 2119 MUST? If so, please consider avoiding 
a word so open to interpretation as "interpreted".

- 5.2, Note: "... based on an TCP initial window of 4 packets, not the 
larger TCP initial windows..."

I assume this means that you borrowed the window size from tcp, not that 
this mechanism actually uses or requires tcp, right? “Based on TCP” 
is a bit confusing.


Editorial Comments:
===================

-3.3, last paragraph:

s/"to be generating" / "to generate"

- 5.3.2, last paragraph:

singular/plural mismatch between "investigations" and "has".

- 6.1, 2nd paragraph:

First sentence seems redundant to the previous paragraph.

- 6.2, 1st paragraph:

s/"...SSRCs are removed ..."/"... SSRCs be removed ..."