Last Call Review of draft-ietf-xrblock-rtcp-xr-jb-11
review-ietf-xrblock-rtcp-xr-jb-11-genart-lc-shirazipour-2013-06-06-00

Request Review of draft-ietf-xrblock-rtcp-xr-jb
Requested rev. no specific revision (document currently at 14)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2013-05-31
Requested 2013-05-23
Draft last updated 2013-06-06
Completed reviews Genart Last Call review of -11 by Meral Shirazipour (diff)
Genart Telechat review of -12 by Meral Shirazipour (diff)
Secdir Last Call review of -11 by Shawn Emery (diff)
Assignment Reviewer Meral Shirazipour
State Completed
Review review-ietf-xrblock-rtcp-xr-jb-11-genart-lc-shirazipour-2013-06-06
Reviewed rev. 11 (document currently at 14)
Review result Ready with Nits
Review completed: 2013-06-06

Review
review-ietf-xrblock-rtcp-xr-jb-11-genart-lc-shirazipour-2013-06-06






I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at


http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq

  .




 




Please resolve these comments along with any other Last Call comments you may receive.




 




Document: draft-ietf-xrblock-rtcp-xr-jb-11




Reviewer: Meral Shirazipour




Review Date: 2013-05-31




IETF LC End Date: 2013-05-31




IESG Telechat date: NA




 




 




Summary:




This draft is almost ready to be published as Standard RFC but I do have some comments.




 




Nits/editorial comments:




-[Page 3], Section 1.4, Suggestion for clarity:




Old:"These metrics are used to report how the jitter buffer at the receiving end of RTP stream behaves





as a result of jitter in the network and are applicable to a range of RTP applications."




 




New:"These metrics are used to report how the jitter buffer at the receiving end of RTP stream behaves





as a result of jitter in the network; and they are applicable to a range of RTP applications."




 




-[Page 5], Section 3, paragraph 5, Suggestion for consistency:




"The Jitter Buffer"---->"The jitter buffer"




 




-[Page 5], last sentence, Suggestion for clarity:




"Assume that both are normalized in ticks."----->




"Assume that both are normalized in ticks (there are 10 000 ticks in a millisecond).




 




-[Page 6], point (iii), Suggestion for clarity:




"If r = t" --suggestion--> "If r == t"




 




-[Page 6], point (iv), Possible inconsistance:




"Delay playout of packet by D + (r-t)"




Comment: D is not normalized in ticks. r and t are. 




 




-[Page 7], Section 4.1, Suggestion for consistency:




"JB Metrics Block"---->"Jitter Block (JB) Metrics Block"




 




-[Page 8], 3 lines before last, "the time spend"--typo-->"the time spent"




 




-[Page 9], line 9, missing "." before "In simple":




"It is calculated based on the time spent in the jitter buffer for the earliest arriving packet In simple..."




 




-[Page 16], Section 10.2 Reference [QMB] does not point to latest version.




 




Best Regards,




Meral




 




---




Meral Shirazipour




Ericsson




Research




www.ericsson.com