[avtext] Eric Rescorla's No Objection on draft-ietf-avtext-lrr-06: (with COMMENT)

Eric Rescorla <ekr@rtfm.com> Mon, 19 June 2017 23:32 UTC

Return-Path: <ekr@rtfm.com>
X-Original-To: avtext@ietf.org
Delivered-To: avtext@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id F22E912956D; Mon, 19 Jun 2017 16:32:27 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Eric Rescorla <ekr@rtfm.com>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-avtext-lrr@ietf.org, Rachel Huang <rachel.huang@huawei.com>, avtext-chairs@ietf.org, rachel.huang@huawei.com, avtext@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.55.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <149791514794.23784.4255587274124751697.idtracker@ietfa.amsl.com>
Date: Mon, 19 Jun 2017 16:32:27 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/avtext/8ROOoTXuRBc9nZP14So_OdkCDr4>
Subject: [avtext] Eric Rescorla's No Objection on draft-ietf-avtext-lrr-06: (with COMMENT)
X-BeenThere: avtext@ietf.org
X-Mailman-Version: 2.1.22
List-Id: Audio/Video Transport Extensions working group discussion list <avtext.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/avtext>, <mailto:avtext-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/avtext/>
List-Post: <mailto:avtext@ietf.org>
List-Help: <mailto:avtext-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/avtext>, <mailto:avtext-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 19 Jun 2017 23:32:28 -0000

Eric Rescorla has entered the following ballot position for
draft-ietf-avtext-lrr-06: No Objection

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-avtext-lrr/



----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

 S 2.1.
   In a layer refresh, however, other layers than the ones requested for
   refresh may still maintain dependency on earlier content of the
   stream.  This is the difference between a layer refresh and a Full

This "however" is hard to read because the entire previous graf
talks about layer refreshes.

All the diagrams in this section would be a lot easier to read
if they said that <- means "refers to"

 S 3.1
   The Feedback Control Information (FCI) for the Layer Refresh Request
   consists of one or more FCI entries, the content of which is depicted
   in Figure 5.  The length of the LRR feedback message MUST be set to
   2+3*N, where N is the number of FCI entries.

You should state that the length is in 32-bit words.


      +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
      | RES     | TTID| TLID          | RES     | CTID| CLID (opt)    |
      +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

How is CLID optional? It seems like it still has to be there,
unless I am misreading the text.


   Reserved (RES) (16 bits / 5 bits / 5 bits)  All bits SHALL be set to
      0 by the sender and SHALL be ignored on reception.

I would mention that this is three fields.