Re: [xrblock] Comments on draft-ietf-xrblock-rtcp-xr-synchronization-00

Kevin Gross <kevin.gross@avanw.com> Tue, 30 October 2012 18:50 UTC

Return-Path: <kevin.gross@avanw.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 C264C21F85C0 for <xrblock@ietfa.amsl.com>; Tue, 30 Oct 2012 11:50:15 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.642
X-Spam-Level:
X-Spam-Status: No, score=-1.642 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, FM_FORGED_GMAIL=0.622, HTML_MESSAGE=0.001, IP_NOT_FRIENDLY=0.334]
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 fvhxKVA5+r2u for <xrblock@ietfa.amsl.com>; Tue, 30 Oct 2012 11:50:14 -0700 (PDT)
Received: from oproxy6-pub.bluehost.com (oproxy6-pub.bluehost.com [67.222.54.6]) by ietfa.amsl.com (Postfix) with SMTP id 6524E21F85A6 for <xrblock@ietf.org>; Tue, 30 Oct 2012 11:49:56 -0700 (PDT)
Received: (qmail 8028 invoked by uid 0); 30 Oct 2012 18:49:32 -0000
Received: from unknown (HELO host291.hostmonster.com) (74.220.215.91) by cpoproxy3.bluehost.com with SMTP; 30 Oct 2012 18:49:32 -0000
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=avanw.com; s=default; h=Content-Type:Cc:To:From:Subject:Message-ID:Date:References:In-Reply-To:MIME-Version; bh=S9cxbSc6Er8228PSN0HcTlgERLvjrlk6Df+9/oWeb9s=; b=SZfAaSxvg/fAem284yAUlZdGyLC6iy3WaLrmBGLZWhJsD8NSVOpguAL89rzGIyoKAvl4fRq70IWfh9FkfTTWca8BNwGJ2nM9dYsxGtEbex/iFli7MS4+K7B1o+lzdT3i;
Received: from [74.125.83.44] (port=51954 helo=mail-ee0-f44.google.com) by host291.hostmonster.com with esmtpsa (TLSv1:RC4-SHA:128) (Exim 4.76) (envelope-from <kevin.gross@avanw.com>) id 1TTGsU-000704-DD for xrblock@ietf.org; Tue, 30 Oct 2012 12:49:30 -0600
Received: by mail-ee0-f44.google.com with SMTP id d4so391380eek.31 for <xrblock@ietf.org>; Tue, 30 Oct 2012 11:49:28 -0700 (PDT)
MIME-Version: 1.0
Received: by 10.14.0.198 with SMTP id 46mr76865043eeb.21.1351622968784; Tue, 30 Oct 2012 11:49:28 -0700 (PDT)
Received: by 10.223.152.201 with HTTP; Tue, 30 Oct 2012 11:49:28 -0700 (PDT)
In-Reply-To: <20121030.133804.10211661.asaeda@sfc.wide.ad.jp>
References: <51E6A56BD6A85142B9D172C87FC3ABBB443AB6A8@szxeml539-mbx.china.huawei.com> <12D5C620-EEE0-4FBC-B77C-165748978B97@csperkins.org> <51E6A56BD6A85142B9D172C87FC3ABBB443ACC48@szxeml539-mbx.china.huawei.com> <20121030.133804.10211661.asaeda@sfc.wide.ad.jp>
Date: Tue, 30 Oct 2012 12:49:28 -0600
Message-ID: <CALw1_Q3YkvxYCqNwuoL8se=a01j0x9tr2JLAagVDj=z1vxPkhA@mail.gmail.com>
From: Kevin Gross <kevin.gross@avanw.com>
To: Hitoshi Asaeda <asaeda@sfc.wide.ad.jp>
Content-Type: multipart/alternative; boundary="047d7b62275eecb63304cd4b3ec0"
X-Identified-User: {1416:host291.hostmonster.com:avanwcom:avanw.com} {sentby:smtp auth 74.125.83.44 authed with kevin.gross@avanw.com}
Cc: xrblock@ietf.org, csp@csperkins.org
Subject: Re: [xrblock] Comments on draft-ietf-xrblock-rtcp-xr-synchronization-00
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: Tue, 30 Oct 2012 18:50:15 -0000

The accuracy requirement we've discussed for presentation times is 10
microseconds or better. I apologize that I have not yet drafted an I-D I
promised summarizing these requirements. Required range is on the order of
several seconds, maybe up to a minute. We need most of the bits to the
right of the decimal in the NTP timestamp but we don't need many to the
left. The reason we included all 64 is because the 64-bit NTP timestamp is
the smallest standard timestamp format that gives us the resolution
required. A non-standard format as small as 40 bits would suffice but RTCP
is not a particularly bandwidth constrained protocol and the standard
format and 32-bit word alignment as proposed in the draft should simplify
implementation.

It looks like I've overlooked issues with Synchronization Delay and
Synchronization Offset in my previous review.

The Synchronization Delay definition needs to provide a more specific
description of what's being referred to in RFC 6051 for defining the
reference point for this measurement. Specifically the draft needs to
define "beginning of session". I don't find such a concept in RFC 6051. RFC
6051 section 2.1.1 defines when sessions are joined but probably not in
enough detail to know how to do a measurement referenced to this point.

The text in 4.2 of the delay draft, "from the RTCP packets received on all
of the components RTP sessions to the beginning of session" might lead some
implementers to compute a negative Synchronization Delay. Revers the from
and to clauses in the text or add a formula to clarify.

There seems to be a dimensional problem in the Synchronization Offset
definition. Everything in the definition is in units of RTP clock but since
we're using an NTP timestamp for the report, I assume this is supposed to
define a quantity in units of NTP time.

Kevin Gross
+1-303-447-0517
Media Network Consultant
AVA Networks - www.AVAnw.com <http://www.avanw.com/>, www.X192.org



On Mon, Oct 29, 2012 at 10:38 PM, Hitoshi Asaeda <asaeda@sfc.wide.ad.jp>wrote:

> Hi Colin and Rachel,
>
> >>>> Hi folks,
> >>>>
> >>>> I have 2 comments for draft-ietf-xrblock-rtcp-xr-synchronization:
> >>>>
> >>>> 1.       In RTP flows synchronization offset metric block, only one
> SSRC set to the SSRC of the reference RTP stream has been specified. IMO,
>  the SSRC of the reporting stream should be also required.
> >>>> 2.       Synchronization offset is a 64-bit unsigned fixed-point
> number. No indication shows which stream, the reporting stream or the
> reference stream,  is lag behind. So I propose to split one bit from
> "Reserved" field to indicate the offset direction.
> >>>
> >>> It would probably be easier to implement if you made the
> Synchronisation Offset a signed 64 bit value, rather than an unsigned 64
> bit value and a separate sign bit.
> >>>
> >> [Rachel]: The reason I proposed that is because the synchronization
> offset is represented using the timestamp format of NTP (a 64-bit unsigned
> fixed-point number), which I think it's better to keep it unchanged.  Are
> you suggesting to use a nonstandard format instead of the standard NTP
> format? But you're right that it may be easier for implementations to use a
> signed value to tell fast or slow from 2 streams.
> >
> >
> > Thinking about this more, I'm confused why you need a 64 bit NTP format
> timestamp here. With the addition of a sign bit, this choice allows you to
> represent synchronisation offsets up to 136 years in either direction with
> picosecond accuracy. That range seems unnecessary. What is the actual
> requirement?
> >
> > [Rachel]: The reason of using 64 bit NTP format timestamp is for
> accuracy, which has been discussed in the list before (
> http://www.ietf.org/mail-archive/web/xrblock/current/msg00504.html ). But
> I agree with you that an additional sign bit is unnecessary. Your proposal
> makes sense here.
>
> Well, the reason we use a 64 bit NTP format timestamp is just its
> simplicity. While we don't need such long format, it'd be easier to
> reuse it for the implementation, I thought.
> But I also don't have no objection to make it signed.
>
> Regards,
> --
> Hitoshi Asaeda
> _______________________________________________
> xrblock mailing list
> xrblock@ietf.org
> https://www.ietf.org/mailman/listinfo/xrblock
>