[Last-Call] Genart last call review of draft-ietf-6man-eh-limits-17

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

 



Reviewer: Peter Yee
Review result: Ready with Nits

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://wiki.ietf.org/en/group/gen/GenArtFAQ>.

Document: draft-ietf-6man-eh-limits-17
Reviewer: Peter Yee
Review Date: 2024-12-19
IETF LC End Date: 2024-12-05
IESG Telechat date: Not scheduled for a telechat

Summary: This is a fairly straightforward informational specification that
suggests how limits on IPv6 extension headers can be defined and applied. There
are some small wording issues that should be fixed prior to publication, but no
important issues obvious to this layer 3 non-expert (and tardy) reviewer.
[Ready with nits]

Major issues: None

Minor issues: None

Nits/editorial comments:

Page 6, 2nd paragraph, 1st sentence: insert “a” before the second occurrence of
“limit”.

Page 6, 2nd paragraph, 3rd sentence: insert “the” before “default”.

Page 7, 5th bullet item: delete a redundant “unless”.

Page 8, section 4, 1st bullet item, 5th sentence: add a comma after “limit”.

Page 9, 1st partial bullet item, 1st whole sentence: append a comma after
“limit”.

Page 10, 1st bullet item, last sentence: delete “the” before “it”.

Page 10, 2nd bullet item, last sentence: delete “the” before “it”.

Page 11, 1st (really long) indented paragraph: I find it odd to use code 8.
Considering the definition of this code in RFC 8883, I’d almost feel that an
update is needed to that RFC to expand the definition. As it, the definition is
just being stretched and appropriated here. Understanding that this is an
Informational specification, it might not be appropriate to make such an update
strictly from this document, so I’ll just leave my point out there.

Page 11, 1st non-indented paragraph: change “an” to “on”.

Page 11, 1st bullet item: append a comma after “Options”.

Page 12, 2nd bullet item, 2nd sentence: append a comma after “limit”.

Page 12, 3rd bullet item, 2nd sentence: append a comma after “the limit”.
Insert “the” before “length limit”.

Page 12, 3rd bullet item, 3rd sentence: change the comma after “exceeded” to a
semicolon. Append a comma after “however”.

Page 12, 4th bullet item, 2nd sentence: append a comma after “received”. (I
believe it’s still appropriate in the if-then construction here.) Append a
comma after “padding”

Page 12, 4th bullet item, 3rd sentence: change the comma after “exceeded” to a
semicolon. Append a comma after “however”.

Page 13, 1st bullet item, 2nd sentence: append a comma after “router”.

Page 13, 1st bullet item, 3rd sentence: change the comma after “exceeded” to a
semicolon. Append a comma after “however”.

Page 13, section 7, 2nd paragraph, 2nd sentence: change “MTU sized” to
“MTU-sized”.

Page 13, section 7, 2nd paragraph, last sentence: change “MTU sized” to
“MTU-sized”. Change “minimal sized” to “minimal-sized”.

Page 13, section 7, 3rd paragraph, 2nd sentence: insert “and” before
“intermediate”.

Page 14, 1st paragraph, last sentence: purely a suggestion: insert
“independently” before “set”.

Page 16, section A.2, 1st sentence: delete the first “limit”. Insert a hyphen
into “104 byte”.

Page 17, 1st non-indented paragraph, last sentence: change both occurrences of
“128 byte” to “128-byte”. Change “192 byte” to “192-byte”. Change “64 byte” to
“64-byte”.

Page 17, 2nd non-indented paragraph, 1st sentence: change “128 byte” to
“128-byte”.

Page 17, 2nd non-indented paragraph, 2nd sentence: change “128 byte” to
“128-byte”. Insert “a” before “104”. Insert a hyphen between that “104” and the
following “byte”.

Page 17, 2nd non-indented paragraph, 3rd sentence: change “32 byte” to
“32-byte”. Change “64 bytes” to “64-byte” (not the dropping of “s”).

Page 17, 1st phrase before bullet items: change “128 byte” to “128-byte”.

Page 17, 4th bullet item: append a comma after “i.e.”.

Page 18, 1st non-indented paragraph, 3rd sentence: delete “does”.

Page 18, section A.4, 2nd paragraph, 1st sentence: change “Open ended” to
“Open-ended”.

Page 18, section A.4, 3rd paragraph, 1st sentence: change “well defined” to
“well-defined”.

Page 18, section A.4, 3rd paragraph, 2nd sentence: insert “in” before
“performance”. Change “is” to “could be”. Insert “a” before “Denial”.

Page 18, section A.4, 4th paragraph, 1st sentence: change “out of order” to
“out-of-order”.



-- 
last-call mailing list -- last-call@xxxxxxxx
To unsubscribe send an email to last-call-leave@xxxxxxxx




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

  Powered by Linux