Document:   draft-ietf-ippm-twamp-08.txt
Reviewer: Suresh Krishnan [suresh.krishnan@ericsson.com]
Review Date:  30 June 2008
IETF LC date: 26 June 2008

Summary: This draft is almost ready for publication but I have some comments.

Minor
=====

* MBZ undefined

I am not aware of any standard definition of MBZ in the IETF. Given this, the document needs to define MBZ in its terminology. Something like

"The bits marked MBZ MUST be set to zero by senders and MUST be ignored by receivers."

* Section 3.5 Creating Test Sessions
 From my reading of this document I understand that the Request-TW-Session command defined here is based on the Request-Session command from RFC4656. This is not made explicit in this document. It would also be helpful to have a command format ASCII art picture before going into defining things. I also think that this section goes too much into what OWAMP does, and I feel it distracts the reader. e.g. The Conf-Sender and the Conf-Receiver text.

* Section 3.8

I think that the number of sessions field should be set to 0 since there are no session description records that follow. Right now this field is undefined in the draft.