Draft: draft-ietf-ippm-owdp-14.txt 
Reviewer: Mark Allman [mallman@icir.org]
Date: Wednesday 6/8/2005 11:40 PM CST

Summary: This document is pretty solid.  I have a few comments...

  + On page 8 it would seem like the mode value should be chosen from
    the mode values advertised in the message given on page 7.  Right?
    I think it'd be good to say this.

  + The MBZ fields are often mentioned in the context of filling them in
    with a "string" of zeros.  I think a better word could be chosen
    here.  I understand that we're not really placing a string in the
    packet.  But, more explicitly stating that each bit must be of value
    zero would be nice.  (This is a nit and maybe something that could
    be clarified by the RFC editor.)

  + Another nit... "uptime" seems like the wrong term.  I think
    "StartTime" would be better since this is an absolute time and not a
    relative time.  I.e., it's when the process started, not how long it
    has been running.  (Right?)  (Again, could be fixed with an RFC
    editor note, I am sure.)

  + I am baffled as to the purpose of the IZP field.  I think there
    needs to be a better paragraph as to what the purpose of this field
    really is.

So, I'd really like the first and last comments to be addressed and
don't think they are all that hard.  I think the last comment is mostly
a matter of understanding what the field is for.   I.e., I think I could
implement without the understanding.  The first item is just a matter of
nailing down a corner.