I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at < http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>. Please resolve these comments along with any other Last Call comments you may receive. Document: draft-ietf-nfsv4-rfc3530bis-dot-x-22.txt Reviewer: Elwyn Davies Review Date: 2014-11-12 IETF LC End Date: 2014-10-06 IESG Telechat date: 2014-12-04 Summary: The main aims of this review were to 1. Check that the extracted .x file actually compiled (it didn't) - fix below. 2. To check the text corresponded with draft-ietf-nfsv4-rfc3530bis-33/34 I found a few minor glitches but otherwise this draft is almost ready. Major Issues: (well not really major) The extracted .x file does not currently compile with rpcgen. The cause is the definition of linktext4 (at line 154 of the extracted file) which was just fixed in draft-ietf-nfsv4-rfc3530bis-34 but not in this one. s/linktext4/linktext4<>/ Minor Issues: The type used for the lease_time attribute in -dot-x-22 is uint32_t, i.e., typedef uint32_t fattr4_lease_time; In 3530bis the type shown in Table 2 is nfs_lease4 but there is no mapping from this type to a base integer type in either document. This needs cleaning up. ======================== Differences between rfc3530-33/34 and rfc35230bus-dot-x-22: ----------------------------------------------------------- The items below need fixing up in 3530bis (apart from possibly the lease_time type) to make them consistent with -dot-x-22. - Error codes NFS4ERR_CB_PATH_DOWN and NFS4ERR_SHARE_DENIED are missing from table 5 in rfc3530-33 and there are no corresponding sections in Section 13.1.1 of rfc3530-33. However, they are mentioned in the text and in Tables 6 and 8. BTW All these tables ought to have titles (assuming they remain after Barry's comments). Also the ordering in Table 5 is slightly out of dictionary sort order if anyone cares. - Required attributes (Table 2 in rfc3530bis): + change attribute type s/b changeid4 + lease_time attribute the type used in -dot-x is uint32_t - nfs_lease4 used in 3530bis-33 is not defined in dot-x or copied into 3530bis. + fs_locations attribute type s/b fs_locations4 (not fs_locations) + owner attribute type s/b utf8str_mixed (not utf8<>) + owner_group attribute type s/b utf8str_mixed (not utf8<>)