Re: [nfsv4] draft-mzhang-nfsv4-sequence-id-calibration-01

[Thomas Haynes <[e-mail]>]

Some nits on draft-mzhang-nfsv4-sequence-id-calibration-01:

1) Please mark the Operation numbers as needing assignment before publication.

BTW: You can look at https://github.com/loghyr/nfsv42_xdr for an unofficial copy of the current XDR.


2) In the ARGUMENTS sections, when you have things like:

struct SEQUENCE_QUERY4args {

   sessionid4 sqa_sessionid;

   slotid4 sqa_slotid;

   };

Please fix these up like this example from my delstid:

    <sourcecode name="" type="" markers="true"><![CDATA[
///
/// /*
///  * attributes for the delegation times being
///  * cached and served by the "client"
///  */
/// typedef nfstime4        fattr4_time_deleg_access;
/// typedef nfstime4        fattr4_time_deleg_modify;
///
 ]]>
    </sourcecode>

Please follow the convention of allowing the XDR to be stripped out by parsing the ‘///‘


3) For the Requirements Language, please use the bcp14 tag

“MUST” -> "<bcp14>MUST</bcp14>”

Note: You need to use this tag every place you use the keywords.