Re: [xrblock] IESG evaluation comments for draft-ietf-xrblock-rtcp-xr-meas-identity-07

Keith,

I'd argue that using a familiar time format (32 or 64 bits NTP-format value as used in other bits of RTCP) is more important than saving 32 bits here.

Colin

On 4 Jul 2012, at 11:18, DRAGE, Keith (Keith) wrote:
> One other option which I will just throw into the ring is that one could divide the existing 32 bits into 2 fields, one of which is a value and the other a multiplier. This has certainly been done in other protocols.
>  
> Depends how much one feels the need to save bits.
>  
> Keith
>  
> From: xrblock-bounces@ietf.org [mailto:xrblock-bounces@ietf.org] On Behalf Of Qin Wu
> Sent: 04 July 2012 06:55
> To: Glen Zorn; xrblock@ietf.org
> Subject: Re: [xrblock] IESG evaluation comments for draft-ietf-xrblock-rtcp-xr-meas-identity-07
>  
> ----- Original Message -----
> From: Glen Zorn
> To: xrblock@ietf.org
> Sent: Wednesday, July 04, 2012 12:47 PM
> Subject: Re: [xrblock] IESG evaluation comments for draft-ietf-xrblock-rtcp-xr-meas-identity-07
>  
> On Wed, 2012-07-04 at 10:19 +0800, Qin Wu wrote:
> 
> ...
> 
> 
>  
> > Second, the cumulative duration is defined in section 4.1 as follows:
> > 
> >   Measurement Duration (Cumulative) : 32 bits
> >      The duration, expressed in units of 1/65536 seconds, of the
> >      reporting interval applicable to Cumulative reports which use this
> >      Measurement Information block.  The value of this field can be
> >      calculated by the receiver of the RTP media stream, for example,
> >      based on received RTP media packets or using RTCP method described
> >      in [RFC3550]. 
> > 
> > The units for the measurement was changed from millisecond to 1/65536 based on working group input. A side effect of this however, is that the maximum cumulative duration that can be expressed is 18.2 hours. This does not seem sufficient, as the cumulative duration may be up to the length of the RTP session.The proposal is to increase the field from 32 bits to 64 bits.
>  
> [Qin]: Agree, we should not limit the cumulative duration to less than 18.2 hours. The proposal looks good to me.
> Here is my input to fix this issue:
> OLD TEXT:
> "
>       Measurement Duration (Cumulative) : 32 bits
>       The duration, expressed in units of 1/65536 seconds, of the
>       reporting interval applicable to Cumulative reports which use this
>       Measurement Information block.  The value of this field can be
>       calculated by the receiver of the RTP media stream, for example,
>       based on received RTP media packets or using RTCP method described
>       in [RFC3550]. 
>  
> "
> NEW TEXT:
> "
>       Measurement Duration (Cumulative) : 64 bits
>       The duration of the reporting interval applicable to Cumulative reports which use this
>       Measurement Information block.  The value of this field is represented using a 64-bit 
>       NTP-format timestamp as defined in [RFC5905], which is 64-bit unsigned fixed-point
>       number with the integer part in the first 32 bits and the fractional part  in the last 32 bits. 
>       It can be calculated by the receiver of the RTP media stream, for example, based on 
>       received RTP media packets or using RTCP method described in [RFC3550]. 
>  
> "
>  
> A fine idea, but the question in my mind is why we don't simply standardize on NTP format for all time values...
>  
> 
> [Qin]: If my intepretation is correct, your question is why not apply NTP format to Measurement Duration (Interval) as well?
> My understanding is:
> the maximum measurement duration (Interval) 18.2 hours is more than enough,it is not necessary to get in line with 64 bit measurement duration (cumulative).
> Another benefit for not applying NTP format to Measurement Duration (Interval) is to make good use of 32bit space allocated and reduce overhead waste.
> ...
> _______________________________________________
> xrblock mailing list
> xrblock@ietf.org
> https://www.ietf.org/mailman/listinfo/xrblock
> _______________________________________________
> xrblock mailing list
> xrblock@ietf.org
> https://www.ietf.org/mailman/listinfo/xrblock

-- 
Colin Perkins
http://csperkins.org/