Re: [Cellar] Adam Roach's Discuss on draft-ietf-cellar-ebml-15: (with DISCUSS and COMMENT)

Dave Rice <dave@dericed.com> Tue, 17 December 2019 21:16 UTC

Return-Path: <dave@dericed.com>
X-Original-To: cellar@ietfa.amsl.com
Delivered-To: cellar@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 4C54512086A; Tue, 17 Dec 2019 13:16:09 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.118
X-Spam-Level:
X-Spam-Status: No, score=-1.118 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_NEUTRAL=0.779, URIBL_BLOCKED=0.001] autolearn=no 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 9egb3z5lee_D; Tue, 17 Dec 2019 13:16:07 -0800 (PST)
Received: from server172-3.web-hosting.com (server172-3.web-hosting.com [68.65.122.111]) (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 A52C3120077; Tue, 17 Dec 2019 13:16:07 -0800 (PST)
Received: from [146.96.19.240] (port=20097 helo=[10.10.201.20]) by server172.web-hosting.com with esmtpsa (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.92) (envelope-from <dave@dericed.com>) id 1ihKCI-002lA8-7P; Tue, 17 Dec 2019 16:16:07 -0500
From: Dave Rice <dave@dericed.com>
Message-Id: <4A5AD779-2C48-460B-A34C-672D98F39C98@dericed.com>
Content-Type: multipart/alternative; boundary="Apple-Mail=_062788C0-12C7-4AD1-935C-D83DD6B043BE"
Mime-Version: 1.0 (Mac OS X Mail 12.4 \(3445.104.8\))
Date: Tue, 17 Dec 2019 16:16:00 -0500
In-Reply-To: <69d0eb8f-f790-293c-0cd9-deadc915254f@nostrum.com>
Cc: Steven Villereal <villereal@gmail.com>, draft-ietf-cellar-ebml@ietf.org, The IESG <iesg@ietf.org>, cellar-chairs@ietf.org, cellar@ietf.org
To: Adam Roach <adam@nostrum.com>
References: <157656052355.24550.17056837047628625307.idtracker@ietfa.amsl.com> <2D4A95A1-B570-47C7-B040-923EF3F24C9C@dericed.com> <69d0eb8f-f790-293c-0cd9-deadc915254f@nostrum.com>
X-Mailer: Apple Mail (2.3445.104.8)
X-OutGoing-Spam-Status: No, score=-2.9
X-AntiAbuse: This header was added to track abuse, please include it with any abuse report
X-AntiAbuse: Primary Hostname - server172.web-hosting.com
X-AntiAbuse: Original Domain - ietf.org
X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12]
X-AntiAbuse: Sender Address Domain - dericed.com
X-Get-Message-Sender-Via: server172.web-hosting.com: authenticated_id: dave@dericed.com
X-Authenticated-Sender: server172.web-hosting.com: dave@dericed.com
X-Source:
X-Source-Args:
X-Source-Dir:
X-From-Rewrite: unmodified, already matched
Archived-At: <https://mailarchive.ietf.org/arch/msg/cellar/2so5tzOx8TGg2czJjU2kSVxZ8L0>
Subject: Re: [Cellar] Adam Roach's Discuss on draft-ietf-cellar-ebml-15: (with DISCUSS and COMMENT)
X-BeenThere: cellar@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Codec Encoding for LossLess Archiving and Realtime transmission <cellar.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/cellar>, <mailto:cellar-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/cellar/>
List-Post: <mailto:cellar@ietf.org>
List-Help: <mailto:cellar-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/cellar>, <mailto:cellar-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 17 Dec 2019 21:16:09 -0000


> On Dec 17, 2019, at 2:56 PM, Adam Roach <adam@nostrum.com> wrote:
> 
> Dave --
> 
> Thanks for the quick response! The issues that the PR addresses look good to me. Two comments inline below.
> 
> On 12/17/19 9:27 AM, Dave Rice wrote:
>> 
>>> 
>>> Abstract:
>>> 
>>> I see that the Introduction has been revised to address Ben Campbell's AD
>>> review comment regarding the document positioning itself as a general-purpose
>>> data format rather than being scoped to its use in Matroska. The Abstract
>>> still claims the much broader scope -- please update it to match the reduced
>>> scope in the Introduction.
>>> 
> 
> I didn't see a response to this discuss point. See also the GENART review, which raises this issue in some detail.

IIUC, this issue is discussed in https://github.com/cellar-wg/ebml-specification/pull/311/files <https://github.com/cellar-wg/ebml-specification/pull/311/files>.

>>> §17.1:
>>> 
>>>> The VINT Data value of one-octet Element IDs MUST be between 0x01 and
>>>> 0x7E.  These items are valuable because they are short, and need to
>>>> be used for commonly repeated elements.  Values from 1 to 126 are to
>>>> be allocated according to the "RFC Required" policy [RFC8126].
>>> 
>>> This, combined with the values that are being registered, is extremely
>>> confusing, and I don't know how IANA is supposed to understand what's going on
>>> without reading and understanding the VINT bit encoding scheme (which is way
>>> too much to ask of them). This is because of the document-wide practice
>>> of speaking of IDs in their VINT-encoded values (e.g., 0xBF) instead of their
>>> data values (e.g., 63 or 0x3F), including in the initial registry in this section.
>>> 
>>> Please either revise the prose to speak in terms of VINT-encoded values
>>> (e.g., "MUST be between 0x81 and 0xFE"), or revise the registration
>>> tables to indicate the VINT data values (e.g., "0x3F" for CRC-32).
>> 
>> I understand the request and the reason for it and am interested in assessing rough consensus amongst the other authors before making this change. These elements have been long-associated with these particular vint expressions of the Element IDs, so I have some concerns that a global change of Element ID reference from VINT to the Element Data of that VINT would confuse readers.
> 
> Given that information, I imagine that the better path would be revising the IANA instructions to operate on VINTs ("MUST be between 0x81 and 0xFE") instead of VINT element data. This should be very straightforward, as the encoding of VINTs results in contiguous ranges that should be easy for IANA to deal with.

Thanks. I’ve updated the section to use the VINT value rather than the decimal of the Element Data of that VINT. The commit is at https://github.com/cellar-wg/ebml-specification/pull/313/commits/948291dfddefa02f857c83a7155519b980d9b2f6 <https://github.com/cellar-wg/ebml-specification/pull/313/commits/948291dfddefa02f857c83a7155519b980d9b2f6>.

Dave Rice