Skip to main content

Last Call Review of draft-ietf-aqm-codel-07
review-ietf-aqm-codel-07-genart-lc-gont-2017-03-25-00

Request Review of draft-ietf-aqm-codel
Requested revision No specific revision (document currently at 10)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2017-03-27
Requested 2017-03-13
Authors Kathleen Nichols , Van Jacobson , Andrew McGregor , Jana Iyengar
I-D last updated 2017-03-25
Completed reviews Secdir Last Call review of -07 by Yoav Nir (diff)
Genart Last Call review of -07 by Fernando Gont (diff)
Assignment Reviewer Fernando Gont
State Completed
Request Last Call review on draft-ietf-aqm-codel by General Area Review Team (Gen-ART) Assigned
Reviewed revision 07 (document currently at 10)
Result Ready w/issues
Completed 2017-03-25
review-ietf-aqm-codel-07-genart-lc-gont-2017-03-25-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

<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.

Document: draft-ietf-aqm-codel-07
Reviewer: Fernando Gont
Review Date: 2017-03-25
IETF LC End Date: 2017-03-27
IESG Telechat date: 2017-04-13

Summary:
The document is well written. However, there seem to be too many details that
are left out in other associated documents. Whereas such details (including
figures) are needed to understand this document, they should be included here.
Either provide full explanations or summarize the outcome without details
(there's text that somehow relies on the reading finding such figures
elsewhere).

Major issues:
* Section 5.1, page 16:
>    A more detailed explanation with many pictures can be found in
>    http://www.ietf.org/proceedings/84/slides/slides-84-tsvarea-4.pdf .

Unless I'm missing something, if this document is specifying CoDel, then all
such details should be here.

Minor issues:
* General:
The document would benefit from a terminology section. For example, only well
into the I-D one finds definitions of terms such as "sojourn times" or "target
setpoint". Not sure if those terms are supposed to be trivial/obvious to most
people reading this document, but their non-definition left me trying to figure
out what the terms were about before I was ale to better understand what you
were describing.

Page 3, Section 1:
>    o  simple and efficient implementation (can easily span the spectrum
>       from low-end, linux-based access points and home routers up to
>       high-end commercial router silicon)

There seems to be an unnecesary implicit "judgement" here, associating
linux-based with low-end, and commercial with high end. I suggest you modify
the text.

Page 10:
>            count_ = (delta > 1 && now - drop_next_ < 16*interval_)?

For clarity, I'd use additional parenthesis here.

Nits/editorial comments:
* Page 17:
>  The power vs. f curve for any AIMD TCP is monotone decreasing.

Please expand the acronym.

* Page 17:
>    simulation that this result holds for Reno, Cubic, and
>    Westwood[TSV84].

Missing space.

Thanks!
Fernando