Skip to main content

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 revision 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
Authors Alan Clark , Varun Singh , Qin Wu
I-D 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 M Emery (diff)
Assignment Reviewer Meral Shirazipour
State Completed
Request Last Call review on draft-ietf-xrblock-rtcp-xr-jb by General Area Review Team (Gen-ART) Assigned
Reviewed revision 11 (document currently at 14)
Result Ready w/nits
Completed 2013-06-06
review-ietf-xrblock-rtcp-xr-jb-11-genart-lc-shirazipour-2013-06-06-00

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