Re: [xrblock] WGLC - draft-ietf-xrblock-rtcp-xr-jb-02.txt

Glen Zorn <glenzorn@gmail.com> Fri, 14 December 2012 10:04 UTC

Return-Path: <glenzorn@gmail.com>
X-Original-To: xrblock@ietfa.amsl.com
Delivered-To: xrblock@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 1CFBE21F8732 for <xrblock@ietfa.amsl.com>; Fri, 14 Dec 2012 02:04:08 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.999
X-Spam-Level:
X-Spam-Status: No, score=-2.999 tagged_above=-999 required=5 tests=[AWL=-0.600, BAYES_00=-2.599, J_CHICKENPOX_25=0.6, J_CHICKENPOX_66=0.6, RCVD_IN_DNSWL_LOW=-1]
Received: from mail.ietf.org ([64.170.98.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 0bxPVcUP1kZb for <xrblock@ietfa.amsl.com>; Fri, 14 Dec 2012 02:04:07 -0800 (PST)
Received: from mail-da0-f44.google.com (mail-da0-f44.google.com [209.85.210.44]) by ietfa.amsl.com (Postfix) with ESMTP id EBD6621F84ED for <xrblock@ietf.org>; Fri, 14 Dec 2012 02:04:04 -0800 (PST)
Received: by mail-da0-f44.google.com with SMTP id z20so1288441dae.31 for <xrblock@ietf.org>; Fri, 14 Dec 2012 02:04:04 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=message-id:date:from:user-agent:mime-version:to:cc:subject :content-type:content-transfer-encoding; bh=ApRGGXkT1KiR1xOhMdGSFzm2hCaF1FPGSPqoT+dNE3Y=; b=OxwlMV8MI6Bx7BDqQty1UpheWcBC8IJWufurYIco63g7KZqW7IELJBQAVCcCkdv5Yq UdvDC59D94l9u9JdaWo5HWoKb6lsjMEhJ1SjY8yFwt99rH6FuxYfcmr0ofkAQPeuljI1 5+oZcUaXK3VclhEXw8orV775lDLs/auf43wC6u5ZUyejhuY4Ffi394ktbA2edtUtVvKL hJ3PpgyDz6TWVjCdR+VpZX5aYR38ylBAfU9ttDJlRJwh0edD/fmPDW9tvdjbaMVFSix3 PElJTyqF2ms0CpRmSTspatXdYQe+z1RB3HofRydJVsUAV5ZUUkaJKeYnsuIcgn+dW5iG rJjA==
Received: by 10.68.190.227 with SMTP id gt3mr14508428pbc.5.1355479444793; Fri, 14 Dec 2012 02:04:04 -0800 (PST)
Received: from [192.168.0.102] (ppp-110-169-250-159.revip5.asianet.co.th. [110.169.250.159]) by mx.google.com with ESMTPS id ni8sm2679841pbc.70.2012.12.14.02.04.02 (version=SSLv3 cipher=OTHER); Fri, 14 Dec 2012 02:04:03 -0800 (PST)
Message-ID: <50CAF991.8050508@gmail.com>
Date: Fri, 14 Dec 2012 17:04:01 +0700
From: Glen Zorn <glenzorn@gmail.com>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/17.0 Thunderbird/17.0
MIME-Version: 1.0
To: xrblock <xrblock@ietf.org>
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 7bit
Subject: Re: [xrblock] WGLC - draft-ietf-xrblock-rtcp-xr-jb-02.txt
X-BeenThere: xrblock@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Metric Blocks for use with RTCP's Extended Report Framework working group discussion list <xrblock.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/xrblock>, <mailto:xrblock-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/xrblock>
List-Post: <mailto:xrblock@ietf.org>
List-Help: <mailto:xrblock-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/xrblock>, <mailto:xrblock-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 14 Dec 2012 10:04:08 -0000

I have reviewed this draft and have some comments.

*Section 1.1*
"SSRC" should be expanded on first usage.

*Section 1.4*
I'm pretty sure that jitter cannot be "encountered", because it is a 
timing abberation, not a physical object.  Suggest using "introduced" 
instead.


The second sentence says:

    These metrics are used to measure how the jitter buffer behave as a 
result of the
    jitter and applicable to a range of RTP applications.

The English language usage needs to be cleaned up a bit, but more 
importantly the statement sees to contradict the first sentence of 
Section 3 which says

    This block describes the configuration and operating parameters of
    the jitter buffer in the receiver of the RTP end system or RTP mixer
    which sends the report.

How can both be correct?


Sorry, but I have no idea what the third sentence means.

*Section 3*
It would be a very good idea, I think, to make capitalization and 
terminology consistent throughout the document.  An example from this 
section: "Instances of this Metrics Block" in sentence one but "this 
metric block" elsewhere.

Sentence two says "Instances of this Metrics Block refer by SSRC to the 
separate auxiliary Measurement Information block  [RFC6776] which 
contains measurement intervals." but AFAICT the Measurement Information 
lock describes only one interval.  Suggest changing to "Instances of 
this metric block refer by SSRC to the separate auxiliary Measurement 
Information block [RFC6776] which describes the measurement interval in 
use."

*Section 3.2*
The description of the "jitter buffer nominal delay" says that "It is 
calculated based on the time for packets spent in the jitter buffer."  
How is it calculated?

*Section 8*
s/Hideaki Yamada,Claire Bi,Colin Perkin/Hideaki Yamada, Claire Bi, Colin 
Perkins, Glen Zorn

:-)