[AVTCORE] AD Review of draft-ietf-payload-vp9-12

"Murray S. Kucherawy" <superuser@gmail.com> Mon, 19 April 2021 22:32 UTC

Return-Path: <superuser@gmail.com>
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 CA33B3A471E for <avt@ietfa.amsl.com>; Mon, 19 Apr 2021 15:32:55 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -0.198
X-Spam-Level:
X-Spam-Status: No, score=-0.198 tagged_above=-999 required=5 tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com
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 MDBs75U-QgMf for <avt@ietfa.amsl.com>; Mon, 19 Apr 2021 15:32:51 -0700 (PDT)
Received: from mail-ua1-x936.google.com (mail-ua1-x936.google.com [IPv6:2607:f8b0:4864:20::936]) (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 2195F3A471C for <avt@ietf.org>; Mon, 19 Apr 2021 15:32:50 -0700 (PDT)
Received: by mail-ua1-x936.google.com with SMTP id j8so9000078uak.2 for <avt@ietf.org>; Mon, 19 Apr 2021 15:32:50 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:from:date:message-id:subject:to; bh=5TjRiYLaPBLml/SnT4P9tzMKKmYcqyU55EbtgIg6dfE=; b=nfF48+PRyf49gthR+AjA/9G5z8u56L9Xn1L86VeO60asjEMQ6BBEItsddE9M9p5JE6 +P5usPrXfzKJFT8Z/Uq/3n6k0GCT9QKctogISyP2Dvn7wCKqgMj7LgiFbFg7En+4cj1u ZyWgXYnWnryJY01hojnDiyVSdOHxm1Q9LF8hO0vwHOIre8KR6HJWalP/oWXhGn4y/V2h xeh8pweVO1rdsvREQNmuhmG2yqOeK2It1AS92rybVktIt9DvMLUcwg7QebpUDe4ZOc0g rZWMGN2d/YmwUNBxJdSayUVg6eYX/yoN38h6D2CL8LhEgetsbJ8wQN45e8dHthHzRqWl W/Eg==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:from:date:message-id:subject:to; bh=5TjRiYLaPBLml/SnT4P9tzMKKmYcqyU55EbtgIg6dfE=; b=SjAhdT61qffYJEIuhUUEl14x6P4lo9zubCJkHjdBTFJRJ14oBTbGg27z7ko/oK1fQ7 l2k54aPBP1MSNV0zSbZBt/0iLROkw4gGY9TGuCw91DE5lh5UcCEZtziUyE5Z5qquF2kE cRoIAZZjmhr7rQ/1wy/+FXsyRofink5T6UJ8E0i+HyYupBp9zlMz4v6fBfMXOwe1JPeq e0wnD0X4BDhHQEsDja1JGMxxZ7LYyGH06JgrLlZ3yCLZ8kaJbk/zbKsTXIdy6SJjFXUN QjMOiSv3Z3zv8gZm32Lf5ctFjIOTr20owRFJ99vgKFZUEdGGfGCC9M/urAEgDCwxGnjY 5cjQ==
X-Gm-Message-State: AOAM531tSMJXmRwF/i3L2yK1Vzdxkdjgjhz/s20Y6rGCRusv6MQDynMc MqyjYOkSZ66l55lCheUoYEjz+l61SqWx5mRQ4q8GNBJs+W0KHg==
X-Google-Smtp-Source: ABdhPJxPQIn6WxTFL0waf+Sow3uuCLVFXgq2q17nfXExmGD/tZsmhAMWdKCBE1HX39fYQ7jGbuYWitHy3/u8bwbVX/s=
X-Received: by 2002:ab0:63c5:: with SMTP id i5mr9230954uap.67.1618871568265; Mon, 19 Apr 2021 15:32:48 -0700 (PDT)
MIME-Version: 1.0
From: "Murray S. Kucherawy" <superuser@gmail.com>
Date: Mon, 19 Apr 2021 15:32:36 -0700
Message-ID: <CAL0qLwZhFSG=zciLMt=USudw-YGPWXmNka84oeS=rWagbC5WFA@mail.gmail.com>
To: IETF AVTCore WG <avt@ietf.org>
Content-Type: multipart/alternative; boundary="000000000000c3766205c05ae899"
Archived-At: <https://mailarchive.ietf.org/arch/msg/avt/6cXcylMMSVswYRwOxrznoGgoJqE>
Subject: [AVTCORE] AD Review of draft-ietf-payload-vp9-12
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: Mon, 19 Apr 2021 22:32:56 -0000

This is my review of draft-ietf-payload-vp9-12.

Video codecs are not exactly my strong suit, so finding technical errors
wasn't an expected part of my mission here.  I'm going on faith that this
is all technically correct, and just looking for editorial or procedural
things.

-MSK

--

Shepherd writeup:

* We should probably explain why a document coming out of AVTCORE bears a
name that doesn't start with "draft-ietf-avtcore-...".  The shepherd
writeup should mention this.  Or if you don't want to do that, please let
me know what the history is there so I can include it in a note to the IESG.

* The writeup identifies draft-ietf-avtext-lrr as a downward reference, but
that document is in the queue already for Proposed Standard status, and
this one is going for the same, so I don't believe this is a downward
reference (once both are published).

General nits:

* Referring to an I-D as a "memo" is a little outdated.  Suggest "document"
or "specification" instead.

Section 1:

* Should this document obsolete VP8 (RFC 6386)?

Section 3:

* The Editor's Note seems like it should've been resolved by now.  If no
suggestion for improved terminology arrives by the end of IESG Evaluation,
should that paragraph be removed?

Section 4.1:

* For "Timestamp", did you mean to say "alternately" (i.e., every other
one; alternating) or "alternatively" (i.e., presenting a different choice)?

Section 4.2:

* First sentence, "The" shouldn't be capitalized.

* The use of CONDITIONALLY RECOMMENDED or CONDITIONALLY REQUIRED is
puzzling because one of them is a BCP 14 word and one isn't, but both look
like they're supposed to be.  This might be better described in the prose
for each of these fields, and just use the BCP 14 words here (or omit them
entirely).

* The description of PID refers to a "maximum ID", but that's not specified
anywhere.  Is it simply the largest PID that will fit in the available
bits, or is there some other maximum to be observed?

* For the "D:" bullet, I think "MUST be set to one if current spatial layer
SID frame depends on spatial layer SID-1 frame of the same picture. MUST
only be set to zero if current spatial layer SID frame does not depend on
spatial layer SID-1 frame of the same picture." can be reduced to "MUST be
set to one if and only if the current spatial layer SID frame depends on
spatial layer SID-1 frame of the same picture."

Section 6.1:

* Does the media type's subtype name have to be "VP9", in specifically that
case?

* Per RFC 6838, please change Required Parameters to "N/A" instead of
"None".

* I think it's unusual for the BCP 14 text to appear in a media type
registration (specifically, the Optional Parameters).  You might consider
moving that discussion to some other prose section, and in the media type
registration just list the supported parameters and their syntaxes.