Last Call Review of draft-ietf-tcpm-dctcp-07
review-ietf-tcpm-dctcp-07-genart-lc-levin-2017-06-29-00
| Request | Review of | draft-ietf-tcpm-dctcp |
|---|---|---|
| Requested revision | No specific revision (document currently at 10) | |
| Type | Last Call Review | |
| Team | General Area Review Team (Gen-ART) (genart) | |
| Deadline | 2017-06-15 | |
| Requested | 2017-06-01 | |
| Authors | Stephen Bensley , Dave Thaler , Praveen Balasubramanian , Lars Eggert , Glenn Judd | |
| I-D last updated | 2017-06-29 | |
| Completed reviews |
Genart Last Call review of -07
by Orit Levin
(diff)
Secdir Last Call review of -07 by Catherine Meadows (diff) Opsdir Last Call review of -07 by Joe Clarke (diff) |
|
| Assignment | Reviewer | Orit Levin |
| State | Completed | |
| Request | Last Call review on draft-ietf-tcpm-dctcp by General Area Review Team (Gen-ART) Assigned | |
| Reviewed revision | 07 (document currently at 10) | |
| Result | Ready w/nits | |
| Completed | 2017-06-29 |
review-ietf-tcpm-dctcp-07-genart-lc-levin-2017-06-29-00
I am the assigned Gen-ART reviewer for this draft. The General Area Review Team (Gen-ART) reviews all IETF documents being processed by the IESG for the IETF Chair. Please treat these comments just like any other last call comments. For more information, please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>. Document: draft-ietf-tcpm-dctcp-07 Reviewer: Orit Levin Review Date: 2017-06-17 IETF LC End Date: 2017-06-15 IESG Telechat date: 2017-06-22 Summary: This draft is ready for publication with nits: Section 3.2 "... an implementation may choose to send two ACKs, ..." Suggest changing "may" to either "can" or "MAY", as fitted. Section 3.4 "... like ssthresh must be changed ..." Suggest changing "must" to either "need" or "MUST", as fitted. Section 3.5 "... may result in..." Suggest changing "may" to "might". Section 4.1 "An implementation should decide when to use DCTCP." Suggest replacing "should decide" with "needs to know". "... there is additional management and configuration overhead required..." suggest replacing with "additional management and configuration functionality is needed". "Potential solutions ..." suggest replacing "Potential" with either "Possible" or "Known". Section 4.2 "... a scaled updated must ensure ..." Suggest replacing "must" with either "MUST" or "needs to". Section 7 "... the first versions..." Suggest replacing with "initial drafts". Section 8 "Since most servers run virtualized in datacenters, in addition ..." Suggest replacing with "Beyond the security considerations related to virtual servers, " Thanks, Orit.