Last Call Review of draft-ietf-ippm-checksum-trailer-03
review-ietf-ippm-checksum-trailer-03-genart-lc-romascanu-2015-10-22-00

Request Review of draft-ietf-ippm-checksum-trailer
Requested rev. no specific revision (document currently at 06)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2015-10-28
Requested 2015-10-14
Authors Tal Mizrahi
Draft last updated 2015-10-22
Completed reviews Genart Last Call review of -03 by Dan Romascanu (diff)
Genart Telechat review of -05 by Dan Romascanu (diff)
Genart Telechat review of -05 by Dan Romascanu (diff)
Secdir Last Call review of -03 by Rich Salz (diff)
Opsdir Last Call review of -03 by Sarah Banks (diff)
Assignment Reviewer Dan Romascanu 
State Completed
Review review-ietf-ippm-checksum-trailer-03-genart-lc-romascanu-2015-10-22
Reviewed rev. 03 (document currently at 06)
Review result Ready with Issues
Review completed: 2015-10-22

Review
review-ietf-ippm-checksum-trailer-03-genart-lc-romascanu-2015-10-22






I am the assigned Gen-ART reviewer for this draft. The General Area Review Team (Gen-ART) reviews all IETF documents being processed by the IESG for the IETF Chair.  Please treat these comments just like any other last call comments.




 




For more information, please see the FAQ at




 




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




 




Document: draft-ietf-ippm-checksum-trailer-03




Reviewer: Dan Romascanu




Review Date: 10/22/15




IETF LC End Date: N/A




IESG Telechat date: N/A




 




Summary:




 




This document is ready for publication. I have a couple of minor issues that need clarification and can be solved easily by minor editing.





 




Major issues:




 




Minor issues:




 







1.

      


It is not clear to me how backwards compatibility is ensured. How do implementations make distinction between OWAMP or TWAMP packets that use timestamp update and Checksum Complement rather than timestamp update
 and UDP checksum update? Is a mix of senders/receivers (for OWAMP) or senders/reflectors (for TWAMP) that some support and some do not support Checksum Complement possible?








2.

      


The last paragraph in section 4 reads: ‘The concept described in this document is intended to be used only in unauthenticated or in authenticated mode.’ This seems either a mistake, or I did not understand what
 it means and some clarification is needed. 




 




Nits/editorial comments: