[nfsv4] Draft is updated following your comments, Thanks//答复: draft-mzhang-nfsv4-sequence-id-calibration-01

["yangjing (U)" <[e-mail]>]

Hi Thomas，

Thanks for you detailed instructions.
The draft is updated as https://www.ietf.org/archive/id/draft-mzhang-nfsv4-sequence-id-calibration-02.txt

Regards
Jing

发件人: Thomas Haynes [mailto:[e-mail]]
发送时间: 2023年4月7日 2:21 AM
收件人: yangjing (U) <[e-mail]>
抄送: [e-mail]
主题: Re: draft-mzhang-nfsv4-sequence-id-calibration-01



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.