Re: [payload] WGLC Feedback for draft-ietf-payload-rtp-h265-09

["Wang, Ye-Kui" <[e-mail]>]

Thanks Bo - the suggested change is indeed an improvement! Thanks Magnus as well for confirming the other change. 

I will now go ahead to make these two changes and submit -11. 

BR, YK

-----Original Message-----
From: Bo Burman [mailto:[e-mail]] 
Sent: Tuesday, June 02, 2015 5:12 AM
To: Magnus Westerlund; Wang, Ye-Kui; Stephan Wenger; Ben Campbell
Cc: [e-mail]; [e-mail]
Subject: RE: [payload] WGLC Feedback for draft-ietf-payload-rtp-h265-09

I checked -09 and -10 versions. Besides the comment made by Magnus and the solution suggested below, which I agree to, I  found only the following nit:

Section 4.1, below Figure 2:
OLD:
Marker bit (M): 1 bit
       Set for the last packet carried in the current RTP stream of
       the access unit.  This is in line with the normal use of the M
NEW:
Marker bit (M): 1 bit
       Set for the last packet of the access unit, carried in the current
       RTP stream.  This is in line with the normal use of the M

Cheers,
Bo

> -----Original Message-----
> From: payload [mailto:[e-mail]] On Behalf Of Magnus 
> Westerlund
> Sent: den 2 juni 2015 10:07
> To: Wang, Ye-Kui; Stephan Wenger; Ben Campbell
> Cc: [e-mail]; [e-mail]
> Subject: Re: [payload] WGLC Feedback for 
> draft-ietf-payload-rtp-h265-09
> 
> Wang, Ye-Kui skrev den 2015-06-01 20:02:
> > Ben noted earlier (in his initial reviewing comments) that we have 
> > the following use of this bit specified at the end of
> 4.4.3:
> >
> > "A receiver in an endpoint or in a MANE MAY aggregate the first n-1 
> > fragments of a NAL unit to an (incomplete) NAL
> unit, even if fragment n of that NAL unit is not received.  In this 
> case, the forbidden_zero_bit of the NAL unit MUST be set to one to indicate a syntax violation."
> >
> > This is also more-or-less aligned with what Magnus mentioned below.
> >
> > Thus, maybe it is better to remove the UDP-Lite example, and mention its actual use in this document, as follows:.
> >
> > "forbidden_zero_bit.  Required to be zero in [HEVC].  Note that the 
> > inclusion of this bit in the NAL unit header was to
> enable transport of HEVC video over MPEG-2 transport systems 
> (avoidance of start code emulations) [MPEG2S].  In the context of this 
> memo, the value 1 may be used to indicate a syntax violation, e.g. for a NAL unit resulted from aggregating a number of fragmented units of a NAL unit but missing the last fragment, as described in Section 4.4.3."
> >
> > If agreeable, I can quickly update the document and submit -v11.
> 
> Yes, that appears a good solution.
> 
> Cheers
> 
> Magnus Westerlund
> 
> ----------------------------------------------------------------------
> Services, Media and Network features, Ericsson Research EAB/TXM
> ----------------------------------------------------------------------
> Ericsson AB                 | Phone  [phone]
> Färögatan 6                 | Mobile [phone]
> SE-164 80 Stockholm, Sweden | mailto: [e-mail]
> ----------------------------------------------------------------------
> 
> _______________________________________________
> payload mailing list
> [e-mail]
> https://www.ietf.org/mailman/listinfo/payload