Skip to main content

Last Call Review of draft-ietf-roll-trickle-mcast-05
review-ietf-roll-trickle-mcast-05-genart-lc-holmberg-2013-11-24-00

Request Review of draft-ietf-roll-trickle-mcast
Requested revision No specific revision (document currently at 12)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2013-10-24
Requested 2013-10-17
Authors Jonathan Hui , Richard Kelsey
I-D last updated 2013-11-24
Completed reviews Genart Last Call review of -05 by Christer Holmberg (diff)
Genart Last Call review of -07 by Christer Holmberg (diff)
Genart Telechat review of -11 by Christer Holmberg (diff)
Secdir Last Call review of -05 by Tero Kivinen (diff)
Opsdir Last Call review of -09 by Benoît Claise (diff)
Assignment Reviewer Christer Holmberg
State Completed
Request Last Call review on draft-ietf-roll-trickle-mcast by General Area Review Team (Gen-ART) Assigned
Reviewed revision 05 (document currently at 12)
Result Almost ready
Completed 2013-11-24
review-ietf-roll-trickle-mcast-05-genart-lc-holmberg-2013-11-24-00

I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART,
please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>



Document:                         draft-ietf-roll-trickle-mcast-05



Reviewer:                           Christer Holmberg



Review Date:                     24 November 2013



IETF LC End Date:             24 October 2013



IETF Telechat Date:         N/A



Summary:  The document is well written, but there are some minor editorial nits
that the authors may want to consider addressing before publication.



Major Issues: None



Minor Issues: None



Editorial nits:



Section 1:

------------



Q_1_1:



I assume "LLN" stands for "Low power and Lossy Networks", but there is no
extension anywhere. Please insert "Low power and Lossy Networks (LLN)" on first
occurance.





Section 3:

------------



Q_3_1:



In a few places the text says "this protocol". I would suggest to replace that
with "MPL".





Section 4:

-----------



Q_4_1:



In section 4, I would suggest to add a reference to section 4.1 after "within
an MPL Domain".



Yes, EVEN if "MPL Domain" is defined in the following section :)





Q_4_2:



I would suggest to add something in front of "Overview" in the subject of
section 4.3. Overview of what? :)





Section 5:

------------



In section 5.5, there is text saying:



    "Following [RFC6206], it is RECOMMENDED that all MPL Forwarders use

       the same values for the Trickle Parameters above for a given MPL

       Domain."



It is a little unclear to me what is "followed". RFC 6206 does not say anything
about MPL Forwardeds and MPL Domain. It talks about the trickle algorithm.



So, you need to say what is recommended in 6206, and how that applies to MPL.



It may be obvious to people involved in this work, but as an outsider I get a
little confused :)





Section 13:

-------------



Q_13_1:



I would suggest to not use roman numbers.





Regards,



Christer