[Last-Call] draft-ietf-tcpm-prr-rfc6937bis-14 ietf last call Genart review

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

 



Document: draft-ietf-tcpm-prr-rfc6937bis
Title: Proportional Rate Reduction for TCP
Reviewer: Russ Housley
Review result: Almost Ready

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-tcpm-prr-rfc6937bis-14
Reviewer: Russ Housley
Review Date: 2025-05-27
IETF LC End Date: 2025-06-06
IESG Telechat date: Not scheduled for a telechat

Summary: Almost Ready


Major Concerns:  None


Minor Concerns:

The Abstract and Section 1 say that this document "updates" the
experimental PRR algorithm.  This wording is confusing because this
document obsoletes RFC 6937.  The Abstract and Section 1 should
state that this document obsoletes RFC 6937.  Avoiding the use of
"updates" is desirable.

The 7th paragraph of Section 5 begins with "A final change"; yet the
8th paragraph talks about another adaptation to PRR.

The last paragraph of Section 5 is not really about changes since the
publication of RFC 6937.  I'm not sure where this information belongs.


Nits:

Section 5: s/(i.e. sndcnt is 0)/(i.e., sndcnt is 0)/

Section 5: s/RTO/retransmission timeout (RTO)/

Section 5: s/sets cwnd = ssthresh/sets cwnd to ssthresh/

Section 5:  s/ECN/Explicit Congestion Notification (ECN)/

Section 5: s/AQMs/ approaches to Active Queue Management (AQM)/

Section 9: Figure 1 is too wide.  Can segment 22 be omitted?  If so,
the text that follows would say: "ACK#22 (not shown) carries ...".

Section 12: s/Janey C.  Hoe/Janey C. Hoe/



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