Last Call Review of draft-ietf-trill-ecn-support-04
review-ietf-trill-ecn-support-04-tsvart-lc-tuexen-2018-02-04-00

Request Review of draft-ietf-trill-ecn-support
Requested rev. no specific revision (document currently at 07)
Type Last Call Review
Team Transport Area Review Team (tsvart)
Deadline 2018-02-05
Requested 2018-01-22
Authors Donald Eastlake, Bob Briscoe
Draft last updated 2018-02-04
Completed reviews Rtgdir Early review of -01 by Loa Andersson (diff)
Rtgdir Early review of -03 by Loa Andersson (diff)
Tsvart Last Call review of -04 by Michael Tüxen (diff)
Opsdir Last Call review of -05 by Sarah Banks (diff)
Genart Last Call review of -04 by Dan Romascanu (diff)
Assignment Reviewer Michael Tüxen
State Completed
Review review-ietf-trill-ecn-support-04-tsvart-lc-tuexen-2018-02-04
Reviewed rev. 04 (document currently at 07)
Review result Ready with Nits
Review completed: 2018-02-04

Review
review-ietf-trill-ecn-support-04-tsvart-lc-tuexen-2018-02-04

I've reviewed this document as part of the transport area directorate's ongoing
effort to review key IETF documents. These comments were written primarily for
the transport area directors, but are copied to the document's authors for
their information and to allow them to address any issues raised.
When done at the time of IETF Last Call, the authors should consider this
review together with any other last-call comments they receive.
Please always CC tsv-art@ietf.org if you reply to or forward this review.

This draft is basically ready for publication, but has nits that should be fixed before publication.

Nits:

Section 1:

Old text:
This can improve network efficiency through better flow
control without packet drops.

New text:
This can improve network efficiency through better congestion
control without packet drops.

Old text:
This specification provides for any ECN marking in the traffic at the
ingress to be copied into the TRILL Extension Header Flags Word.

New Text:
This specification specifies for any ECN marking in the traffic at the
ingress to be copied into the TRILL Extension Header Flags Word.

Section 2:

Old text:
after the Extesnion Flags Word.

New text:
after the Extension Flags Word

Section 3.3

Please define "3-bit ECN codepoint" and refer to Table 3 BEFORE using it.
This might result in swapping Table 2 and Table 3.