Document: draft-green-secsh-ecc-08.txt
Reviewer: Miguel A. Garcia
Review Date: 2009-07-14 
IETF LC End Date: 2009-08-10


Summary: The document is ready for publication as a standard track RFC.


Major issues: none


Minor issues:

- In Section 9.1, second sentence, the text reads: "These curve should always be enabled ...." I think the "should" should be elevated to a "SHOULD" in this case.

- The IANA Considerations section needs more work. In particular, it does not name the registry where IANA has to operate. I am assuming that this registry is "Secure Shell (SSH) Protocol Parameters). That registry includes several subregistries, of which, this draft is presumably adding
  new values to the "Public Key Algorithm Names" and "Key Exchange Method Names". Please clearly explain what needs to be registered where.

Nits/editorial comments: A couple of minor nits, in case you need to create a new version of the document

- The third paragraph of Section 6.1 contains an 8-line long sentence. I think this sentence could be simplified and chunked into smaller sentences, because it is a bit hard to read such long sentences.

- The first paragraph in Section 7 reads ... "... without being granted IANA permission". If we are accurate, IANA does not grant any permission. 
Instead IANA acts as a registry. So, most likely you want to say "... 
without requiring an IANA registration process" or something like that.

- Title of Section 9.2, missing white space separated two words.