Telechat Review of draft-ietf-6lo-btle-14
review-ietf-6lo-btle-14-genart-telechat-davies-2015-07-10-00

Request Review of draft-ietf-6lo-btle
Requested rev. no specific revision (document currently at 17)
Type Telechat Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2015-07-07
Requested 2015-06-26
Authors Johanna Nieminen, Teemu Savolainen, Markus Isomaki, Basavaraj Patil, Zach Shelby, Carles Gomez
Draft last updated 2015-07-10
Completed reviews Genart Last Call review of -13 by Elwyn Davies (diff)
Genart Telechat review of -14 by Elwyn Davies (diff)
Secdir Last Call review of -13 by Chris Lonvick (diff)
Secdir Telechat review of -14 by Chris Lonvick (diff)
Opsdir Last Call review of -13 by Niclas Comstedt (diff)
Assignment Reviewer Elwyn Davies
State Completed
Review review-ietf-6lo-btle-14-genart-telechat-davies-2015-07-10
Reviewed rev. 14 (document currently at 17)
Review result Ready
Review completed: 2015-07-10

Review
review-ietf-6lo-btle-14-genart-telechat-davies-2015-07-10

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

Please wait for direction from your document shepherd
or AD before posting a new version of the draft.

Document: draft-ietf-6lo-btle-14.txt
Reviewer: Elwyn Davies
Review Date: 2015/07/07
IETF LC End Date: 2015/06/09
IESG Telechat date: 2015/07/09

Summary:


Ready for proposed standard.  Thanks for sorting out the points raised 


in my last call review.  The section on subnet usage is very helpful.  


There are a few very minor nits that I spotted on reading the new 


version that can be passed to the RFC Editor if no other changes are needed.




Major issues:
None.

Minor issues:
None.

Nits/editorial comments:
s3, para 2: s/inter- peripheral/inter-peripheral/ [Remove one space]


s3.2.2, last para: "(see Section 3.2.2)" is self-referential!  I think 


you mean 3.2.3.



s3.2.4.2, last para (Item 3): s/if most recently/if the most recently/


s3.2.4, para 5: To avoid the slight confusion which I had in the 


previous version, I suggest a modification to this para to make it clear 


that the link local address can only occur on comms to the 6LBR and not 


through it, thus:



OLD:
   When a 6LN is sending a packet to or through a 6LBR, it MUST fully
   elide the source address if it is a link-local address.  A non-link-
   local source address 6LN has registered with ARO to the 6LBR for the
   indicated prefix MUST be fully elided if the source address is the
   latest address 6LN has registered for the indicated prefix.
NEW:
   When a 6LN is sending a packet to a 6LBR, it MUST fully


   elide the source address if it is a link-local address.  For other 


packets to or


   through a 6LBR with a non-link-local source address that the 6LN has 


registered


   with ARO to the 6LBR for the indicated prefix, the source address 


MUST be fully


   elided if it is the latest address that the 6LN has registered for 


the indicated prefix.



END


s5:  Need to expand UART acronym in new Direct Test Mode text (USB is on 


the well-known list, assuming it is the Universal Serial Bus meaning).



s5, last para: s/service attacks/service attack/