Re: Tsvart last call review of draft-ietf-trill-ecn-support-04

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Michael,

Thanks for the comments, see below.

On Sun, Feb 4, 2018 at 1:54 PM, Michael Tüxen <tuexen@xxxxxxxxxxxxxx> wrote:
> Reviewer: Michael Tüxen
> Review result: Ready with Nits
>
> 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@xxxxxxxx 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.

OK.

> 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.

Well, it specifies how to copy the ECN marking for any traffic and it
mandates that it be copied from IP traffic but it doesn't and can't
mandate that it be copied from all arbitrary protocols that might have
ECN marking...  Also, I don't like "... specification specifies ..."

How about:
"This document specifies how ECN marking in traffic at the ingress is
copied into the TRILL Extension Header Flags Word and requires such
copying for IP traffic."

> Section 2:
>
> Old text:
> after the Extesnion Flags Word.
>
> New text:
> after the Extension Flags Word

OK.

> 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.

We'll look into doing that.

Thanks,
Donald
===============================
 Donald E. Eastlake 3rd   +1-508-333-2270 (cell)
 155 Beaver Street, Milford, MA 01757 USA
 d3e3e3@xxxxxxxxx





[Index of Archives]     [IETF Annoucements]     [IETF]     [IP Storage]     [Yosemite News]     [Linux SCTP]     [Linux Newbies]     [Fedora Users]