Re: [tsvwg] Ben Campbell's Yes on draft-ietf-tsvwg-sctp-ndata-12: (with COMMENT)

Ben Campbell <ben@nostrum.com> Wed, 30 August 2017 15:13 UTC

Return-Path: <ben@nostrum.com>
X-Original-To: tsvwg@ietfa.amsl.com
Delivered-To: tsvwg@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 7DF1E1329C0; Wed, 30 Aug 2017 08:13:35 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.88
X-Spam-Level:
X-Spam-Status: No, score=-1.88 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, T_SPF_HELO_PERMERROR=0.01, T_SPF_PERMERROR=0.01] autolearn=ham autolearn_force=no
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 mO7jQaX2wVoO; Wed, 30 Aug 2017 08:13:33 -0700 (PDT)
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 8D64F126B71; Wed, 30 Aug 2017 08:13:33 -0700 (PDT)
Received: from [10.0.1.63] (cpe-66-25-7-22.tx.res.rr.com [66.25.7.22]) (authenticated bits=0) by nostrum.com (8.15.2/8.15.2) with ESMTPSA id v7UFCtKG020673 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NO); Wed, 30 Aug 2017 10:12:57 -0500 (CDT) (envelope-from ben@nostrum.com)
X-Authentication-Warning: raven.nostrum.com: Host cpe-66-25-7-22.tx.res.rr.com [66.25.7.22] claimed to be [10.0.1.63]
Content-Type: text/plain; charset="us-ascii"
Mime-Version: 1.0 (Mac OS X Mail 10.3 \(3273\))
From: Ben Campbell <ben@nostrum.com>
In-Reply-To: <F924AD52-A907-49CD-B448-AA6BC34DF737@fh-muenster.de>
Date: Wed, 30 Aug 2017 10:12:55 -0500
Cc: Gorry Fairhurst <gorry@erg.abdn.ac.uk>, draft-ietf-tsvwg-sctp-ndata@ietf.org, The IESG <iesg@ietf.org>, tsvwg@ietf.org, tsvwg-chairs@ietf.org
Content-Transfer-Encoding: quoted-printable
Message-Id: <7D1CD491-0991-4950-95CF-69BDEA3D33F9@nostrum.com>
References: <150404598289.21566.8138949859492184191.idtracker@ietfa.amsl.com> <F924AD52-A907-49CD-B448-AA6BC34DF737@fh-muenster.de>
To: Michael Tuexen <tuexen@fh-muenster.de>
X-Mailer: Apple Mail (2.3273)
Archived-At: <https://mailarchive.ietf.org/arch/msg/tsvwg/yjpUIuU_JVqipVRgzQK6cRbqF10>
Subject: Re: [tsvwg] Ben Campbell's Yes on draft-ietf-tsvwg-sctp-ndata-12: (with COMMENT)
X-BeenThere: tsvwg@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: Transport Area Working Group <tsvwg.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/tsvwg>, <mailto:tsvwg-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/tsvwg/>
List-Post: <mailto:tsvwg@ietf.org>
List-Help: <mailto:tsvwg-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/tsvwg>, <mailto:tsvwg-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 30 Aug 2017 15:13:35 -0000

> On Aug 30, 2017, at 4:02 AM, Michael Tuexen <tuexen@fh-muenster.de> wrote:
> 
>> On 30. Aug 2017, at 00:33, Ben Campbell <ben@nostrum.com> wrote:
>> 
>> Ben Campbell has entered the following ballot position for
>> draft-ietf-tsvwg-sctp-ndata-12: Yes
>> 
>> When responding, please keep the subject line intact and reply to all
>> email addresses included in the To and CC lines. (Feel free to cut this
>> introductory paragraph, however.)
>> 
>> 
>> Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
>> for more information about IESG DISCUSS and COMMENT positions.
>> 
>> 
>> The document, along with other ballot positions, can be found here:
>> https://datatracker.ietf.org/doc/draft-ietf-tsvwg-sctp-ndata/
>> 
>> 
>> 
>> ----------------------------------------------------------------------
>> COMMENT:
>> ----------------------------------------------------------------------
>> 
> Hi Ben,
> 
> thanks for your review. Comments in-line.
> 
> Best regards
> Michael
>> I wish this had been around back when we were figuring out how to share
>> connections in MSRP.
>> 
>> Otherwise, just a couple of editorial comments:
>> 
>> - Abstract: "Multiple ways for performing
>>  this selection, called stream schedulers, are defined. "
>> I assume you mean that they are defined in this document rather than in
>> previously existing documents, but it seems ambiguous.
> Changed to
> 
> Multiple ways for performing this selection, called stream schedulers, are
> defined in this document.
> 

Works for me.

>> 
>> - 2.1: "The sender uses two counters for
>>     each outgoing stream, one for ordered messages, one for unordered
>>     messages.  All counters are independent and initially 0."
>> 
>> s/ all / both  ; unless this rule is intended to apply to more counters than
>> the ones mentioned in the previous sentence.
>> 
>> 
> With "all counters" I'm referring to two counters for each outgoing stream. But
> there might be more than one outgoing stream. So if there are n outgoing streams,
> I'm referring to 2 * n counters. And these 2 * n counters are mentioned in the
> previous sentence.

I read it in the context of the previous sentence, that is, both counters in a given stream. But this is not a blocking comment; do what you think is best.

Thanks!

Ben.