IETF Logo Mail Archive

Re: [Gen-art] Gen-ART Last Call review of draft-ietf-avtcore-cc-feedback-message-08

Colin Perkins <csp@csperkins.org> Thu, 17 September 2020 22:43 UTC

Return-Path: <csp@csperkins.org>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id B415F3A0D9C; Thu, 17 Sep 2020 15:43:27 -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 Hov7Djb7xFUP; Thu, 17 Sep 2020 15:43:25 -0700 (PDT)
Received: from balrog.mythic-beasts.com (balrog.mythic-beasts.com [IPv6:2a00:1098:0:82: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 251333A0D81; Thu, 17 Sep 2020 15:43:20 -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=SoMCmtjP7+KEjtXrUIHlg6EuuF5HU+buHUCeZBAQie4=; b=FLHjwZL9+26JyXuPDsvz/nJyfC eqYabGTI86SSDhCG+OEpCCbm1GWl+w1BP4fq5PKGsv67CAz727T0Di36NW6Tan95lMSqEirxc01yY 9QqjS+BZ0LHHgiRUFZs0YcTHGxWqdhPYZ3yj7RmkPuIgFBCCnvu9DmkreCMm5JWMpCg3fU43hpkUm qgP+eC06kOoKa+rDNyDjc9sOeS7Xsf9Ea3GiFo2hi6g256ONuup6J1w9Uo5+ufdY4eN59GGHQ0PzP Zbdd+Hjed+dxi8MDX2Qp9jk4TxmK50ZicJO/NOPwTWUkxLEusyBeQTy3sMgLGE6pcSBUiQOMtQd8m 1WMRpx5A==;
Received: from [81.187.2.149] (port=44640 helo=[192.168.0.67]) by balrog.mythic-beasts.com with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92.3) (envelope-from <csp@csperkins.org>) id 1kJ2cU-0002yc-F0; Thu, 17 Sep 2020 23:43:18 +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: <b261567d-5b2b-298c-692b-136e87f5bcb8@alum.mit.edu>
Date: Thu, 17 Sep 2020 23:43:10 +0100
Cc: draft-ietf-avtcore-cc-feedback-message.all@ietf.org, General Area Review Team <gen-art@ietf.org>
Content-Transfer-Encoding: quoted-printable
Message-Id: <43689733-BECF-4A84-96A6-6738448ED960@csperkins.org>
References: <b261567d-5b2b-298c-692b-136e87f5bcb8@alum.mit.edu>
To: Paul Kyzivat <pkyzivat@alum.mit.edu>
X-Mailer: Apple Mail (2.3445.104.15)
X-BlackCat-Spam-Score: 4
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/IBtR0fCroAtXriAE0coyWvPwJgY>
Subject: Re: [Gen-art] Gen-ART Last Call review of draft-ietf-avtcore-cc-feedback-message-08
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 17 Sep 2020 22:43:28 -0000

> On 7 Sep 2020, at 16:46, Paul Kyzivat <pkyzivat@alum.mit.edu> wrote:
> 
> I am the assigned Gen-ART reviewer for this draft. The General Area
> Review Team (Gen-ART) reviews all IETF documents being processed
> by the IESG for the IETF Chair.  Please treat these comments just
> like any other last call comments.
> 
> For more information, please see the FAQ at
> 
> <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.
> 
> Document: draft-ietf-avtcore-cc-feedback-message-08
> Reviewer: Paul Kyzivat
> Review Date: 2020-09-16
> IETF LC End Date: 2020-09-16
> IESG Telechat date: ?
> 
> This draft is basically ready for publication, but has nits that should be fixed before publication.
> 
> I noticed one nit (typo) in section 7:
> 
>   s/ecn-capaable-rtp/ecn-capable-rtp/

Will fix.

> Beyond that I have one question/suggestion that I'm happy to have you consider or ignore as you see fit:
> 
> In the case of missing packets it seems that you have the opportunity to save some space by encoding a run of contiguous missing packets as a single packet metric block with L=0, using the remaining bits in the metric block to encode the number of missing packets. Whether that is an optimization worth the trouble to do is a question I can't answer.


It’s certainly possible to do that, and the WG did discuss optimisations and more complex formats. At present, I think simple and published outweigh optimised.

Cheers,
Colin



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

v2.23.0 | Report a Bug | By Email