Re: [AVTCORE] AD review of draft-ietf-avtcore-cc-feedback-message-07
Colin Perkins <csp@csperkins.org> Thu, 23 July 2020 21:52 UTC
Return-Path: <csp@csperkins.org>
X-Original-To: avt@ietfa.amsl.com
Delivered-To: avt@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id CF0AE3A0407; Thu, 23 Jul 2020 14:52:28 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.1
X-Spam-Level:
X-Spam-Status: No, score=-2.1 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=csperkins.org
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 qekoT0GvyMqK; Thu, 23 Jul 2020 14:52:26 -0700 (PDT)
Received: from haggis.mythic-beasts.com (haggis.mythic-beasts.com [IPv6:2a00:1098:0:86:1000:0:2:1]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 9ED453A0403; Thu, 23 Jul 2020 14:52:26 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=csperkins.org; s=mythic-beasts-k1; h=To:Date:From:Subject; bh=B5FleaymmVlRjSAaCHaW16dbBKgyIMD7MSONajLih5Q=; b=AuzmwN1mgPSE2unt25MWIy6ovc SsqbxjC6fWB2YDKukyr+EZgKsWYsXvezVkUr420Xio0jMw9PEiawsSeqRyPpTXk2bB5A+NPEOC9BD 0G6bXXQ8gfo8qjg7ef7747SVwcpqoJW2iGfd+vYMmomvaG2LU2l5btgKtyYlUE7WJXgj0hy+Kjqm9 OWOPmVHVZqRdfp815lD2RV1vps6vjWwRSKrotQFY8oBdcLU/ogNRDUf74d1iE5NnzRO45fd1vmzJh zrT+6VSBzI8eTw1fA2yCPTdVL5veazCmhMqZNpE72q9+8HJuZL+5ziN69Zmhn2coi0udqbTwNdVhw HmMh9PpQ==;
Received: from [81.187.2.149] (port=49030 helo=[192.168.0.80]) by haggis.mythic-beasts.com with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92.3) (envelope-from <csp@csperkins.org>) id 1jyj8a-00079Z-T3; Thu, 23 Jul 2020 22:52:25 +0100
Content-Type: text/plain; charset="utf-8"
Mime-Version: 1.0 (Mac OS X Mail 12.4 \(3445.104.15\))
From: Colin Perkins <csp@csperkins.org>
In-Reply-To: <CALaySJKbFKzBU7AJDqZSTmBCJKh6QJU8pHi38KumuJ878TPY0Q@mail.gmail.com>
Date: Thu, 23 Jul 2020 22:52:21 +0100
Cc: draft-ietf-avtcore-cc-feedback-message.all@ietf.org, avt@ietf.org
Content-Transfer-Encoding: quoted-printable
Message-Id: <891B1BF4-3E27-460C-A701-2B357FEECE0F@csperkins.org>
References: <CALaySJKbFKzBU7AJDqZSTmBCJKh6QJU8pHi38KumuJ878TPY0Q@mail.gmail.com>
To: Barry Leiba <barryleiba@computer.org>
X-Mailer: Apple Mail (2.3445.104.15)
X-BlackCat-Spam-Score: 4
Archived-At: <https://mailarchive.ietf.org/arch/msg/avt/h2blZmjDNpNJvslmr9U03u3Wi5A>
Subject: Re: [AVTCORE] AD review of draft-ietf-avtcore-cc-feedback-message-07
X-BeenThere: avt@ietf.org
X-Mailman-Version: 2.1.29
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: Thu, 23 Jul 2020 21:52:29 -0000
Thanks – this all looks reasonable and minor. I’ll try to spin an update prior to the meeting. Colin > On 23 Jul 2020, at 22:38, Barry Leiba <barryleiba@computer.org> wrote: > > Thanks for the work on this clear, easy-to-read document, and I'm > sorry it took me a while to get through the workload to review it. > Here's my AD review, and I'll set the state to "revised I-D needed" > and start last call after -08 is posted. > > Everything here is minor; please let me know if you disagree with > anything below. > > — Section 1 — > Please include reference citations for RTP and RTCP. > > As a direct result, > the different congestion control (i.e., rate adaptation) designs are > not interoperable. > > Is “i.e.,” the right term here, or do you mean “such as”? I would > also move the parenthetical one word later, rather than having it > split the phrase “congestion control designs”. > > — Section 2 — > Please use the current BCP 10 boilerplate, found in RFC 8174, and add > a normative reference to that RFC. > > — Section 3 — > > o RTP sequence number: The receiver of an RTP flow needs to feedback > the sequence numbers of the received RTP packets to the sender > > Nit: “feed back”, as a verb, should be two words. Or perhaps “needs > to feed the sequence numbers … back to the sender”. (Check other uses > of “feedback” as a verb also.) > > — Section 3.1 — > > The contents of each 16-bit packet metric block comprises the L, ECN, > and ATO fields are as follows: > > Nit: It looks like “are” is an extra word here, yes? > > 1 represent the > packet was received and the subsequent bits in the block need to > be parsed. > > Nit: “1 represents that the” > > — Section 5 — > > However, if > multiple consecutive congestion control feedback packets are lost, > the sender SHOULD rapidly reduce its sending rate towards zero, as > this likely indicates a path failure. > > Doesn’t “reduce” already mean “towards zero”? Is something further > meant by saying “towards zero” that should be made clearer (I suspect > so)? > > — Section 6 — > > When used with "ccfb" feedback, the wildcard payload type ("*") MUST > be used. > > This doesn’t read well as written, and sorta doesn’t make much sense. > I think you just want to say this: > > NEW > The payload type used with “ccfb” feedback MUST be the wildcard type (“*”). > END > > — Section 8 — > > An efficient congestion control algorithm requires more fine grained > information on per packet reception quality > > Nit: hyphenate “fine-grained” and “per-packet”. Also hyphenate the > former in the following sentence. > > This is the opposite of the sender based congestion > control approach suggested in this memo, so TMMBR cannot be used > to convey the information needed for a sender based congestion > control. > > Nit: hyphenate “sender-based” in both places, and also at the end of > the section. > > — Section 11 — > > at a greater rate than the path can support, thereby congesting the > path. > > Is “congesting” really used as a verb generally? Better as “thereby > causing congestion on the path.” > > Barry -- Colin Perkins https://csperkins.org/
- [AVTCORE] AD review of draft-ietf-avtcore-cc-feed… Barry Leiba
- Re: [AVTCORE] AD review of draft-ietf-avtcore-cc-… Colin Perkins
- Re: [AVTCORE] AD review of draft-ietf-avtcore-cc-… Colin Perkins
- Re: [AVTCORE] AD review of draft-ietf-avtcore-cc-… Barry Leiba