[Roll] Part 2: comment resolutions for AD Review of draft-ietf-roll-aodv-rpl-07

Charlie Perkins <charles.perkins@earthlink.net> Thu, 07 May 2020 19:52 UTC

Return-Path: <charles.perkins@earthlink.net>
X-Original-To: roll@ietfa.amsl.com
Delivered-To: roll@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 93E523A0CF3; Thu, 7 May 2020 12:52:26 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.098
X-Spam-Level:
X-Spam-Status: No, score=-2.098 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=earthlink.net; domainkeys=pass (2048-bit key) header.from=charles.perkins@earthlink.net header.d=earthlink.net
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id QapCztw_5wTS; Thu, 7 May 2020 12:52:23 -0700 (PDT)
Received: from elasmtp-dupuy.atl.sa.earthlink.net (elasmtp-dupuy.atl.sa.earthlink.net [209.86.89.62]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 2B0943A0CE7; Thu, 7 May 2020 12:52:23 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=earthlink.net; s=dk12062016; t=1588881143; bh=qbTXKO2luPxYnVLBh5WpJtXsBHdOFCbhAsho oj6Xmxc=; h=Received:To:Cc:From:Subject:Message-ID:Date:User-Agent: MIME-Version:Content-Type:Content-Transfer-Encoding: Content-Language:X-ELNK-Trace:X-Originating-IP; b=U/b3+HEDe2IGoXrh AZ+aMqMPuMbxBxS+zoagx80mOwODV7gG5Z370w86k/rrACudUG/q6aALEPGY4VP0MNy KGZiQ+dvQE+MT1FfZwQuaaLJ6tHkB7dx6RoKZejCfNkBEfHoV7+p/FTXj57XhmU8PaG emtyBvYmYibZNoe4cLF++9S4S0MMKwya+3heYdu6uFHScQh1LeGXxxhm1tf3vg9BzDD jD9NxNpyKc3ucIiAYBQihe7EVBwp9wPjSemJnlV+AwnTMeiJYHenF3c9aeT0lP404oH rI7hhl96D2LUuR0iTHrMWiY7+/czyzBA/89LwZSkfj6UuZ9fENHFLINGrg==
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=dk12062016; d=earthlink.net; b=jBil1tsDNIy+90xgZrIUKK4NSrb3DHIYy1vX+M74JI0Liu83pllEEvaGv0ai2wJdju1AmtpEi3M7GHdZJVf8nACdJeZCRJfDOGKPVliSb2dlkRmjD5L9hqWK4wZ6pmb0rqDg1DYjRwaOwQxakEXHx+NRdHSZ2MY7vH7Ij+RLm87Hm1/Pw1cQdF2sQ3IsJemQZSF94+y0jDHq6xFddN464hF0QGH4atvsybLZW6SpLBrEEDHR9nofUw0Gbfb7FHd6p2lPAXNvVKAvNv+t8/g6UdedRwDyMnAXEls9rdDpsBL+NMLed1AWGly0itkdH39vCQqPOx4fyPIYcb1ONwkKzg==; h=Received:To:Cc:From:Subject:Message-ID:Date:User-Agent:MIME-Version:Content-Type:Content-Transfer-Encoding:Content-Language:X-ELNK-Trace:X-Originating-IP;
Received: from [99.51.72.196] (helo=[192.168.1.82]) by elasmtp-dupuy.atl.sa.earthlink.net with esmtpsa (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128) (Exim 4) (envelope-from <charles.perkins@earthlink.net>) id 1jWmZC-0006lx-9d; Thu, 07 May 2020 15:52:22 -0400
To: Routing Over Low power and Lossy networks <roll@ietf.org>
Cc: "draft-ietf-roll-aodv-rpl@ietf.org" <draft-ietf-roll-aodv-rpl@ietf.org>
From: Charlie Perkins <charles.perkins@earthlink.net>
Message-ID: <81fbfde3-df64-311e-392d-7c091a06fd80@earthlink.net>
Date: Thu, 07 May 2020 12:52:20 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Transfer-Encoding: 8bit
Content-Language: en-US
X-ELNK-Trace: 137d7d78656ed6919973fd6a8f21c4f2d780f4a490ca6956846b590522b13c95c07a736cb524653e68e9189f9bf973d8350badd9bab72f9c350badd9bab72f9c
X-Originating-IP: 99.51.72.196
Archived-At: <https://mailarchive.ietf.org/arch/msg/roll/9EdhU3zjhaSbodw19OWC24JHLaU>
Subject: [Roll] Part 2: comment resolutions for AD Review of draft-ietf-roll-aodv-rpl-07
X-BeenThere: roll@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Routing Over Low power and Lossy networks <roll.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/roll>, <mailto:roll-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/roll/>
List-Post: <mailto:roll@ietf.org>
List-Help: <mailto:roll-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/roll>, <mailto:roll-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 07 May 2020 19:52:27 -0000

  Hello folks,

Today we have submitted a revised version 8 for our 
draft-ietf-roll-aodv-rpl.  Here are the rest of some resolutions for 
comments made by our AD for the improvement of draft-ietf-roll-aodv-rpl-07.

Regards,
Charlie P.


 > ...
 > 672      Step 3:
 >
 > 674         If the 'H' bit is set to 1, then the router (TargNode or
 > 675         intermediate) MUST build the upward route entry 
accordingly.  The
 > 676         route entry MUST include at least the following items: Source
 > 677         Address, InstanceID, Destination Address, Next Hop, 
Lifetime, and
 > 678         Sequence Number.  The Destination Address and the 
InstanceID can
 > 679         be respectively learned from the DODAGID and the 
RPLInstanceID of
 > 680         the RREQ-DIO, and the Source Address is copied from the ART
 > 681         Option.  The next hop is the preferred parent.  The 
lifetime is
 > 682         set according to DODAG configuration and can be extended 
when the
 > 683         route is actually used.  The sequence number represents the
 > 684         freshness of the route entry, and it is copied from the 
Orig SeqNo
 > 685         field of the RREQ option.  A route entry with same source and
 > 686         destination address, same InstanceID, but stale sequence 
number,
 > 687         SHOULD be deleted.
 >
 > [major] "...MUST build the upward route entry accordingly."  I was 
going to ask what does "accordingly" mean, but the next sentence 
indicates what it has to include.  Instead of having two sentences 
trying to specify the same thing, please collapse them into one.
Done.
 >
 > [minor] This route is to OrigNode, correct?  Please say so.  I know 
that it has been mentioned that the RREQ-Instance is used for that -- 
remind the reader.
Done.
 >
 > [major] "...the Source Address is copied from the ART Option."   What 
is the "Source Address"?  I ask because I assumed that it is the address 
used by the local router to send data to the OrigNode, but the only 
address in the ART Option is the "Target Prefix".  What am I missing?

I used your text as follows:

                   ...the Source Address is the address used by the 
local router to
         send data to the OrigNode
 >
 > [nit] Please be consistent in how names are used.  For example, the 
paragraph above uses both "Next Hop" and "next hop" to refer to the same 
thing.

Fixed.


 >
 > [minor] "The lifetime is set according to DODAG configuration and can 
be extended when the route is actually used."  I think that this is a 
little confusing, because you seem to be talking about route lifetime 
and not the L (which I interpreted as lifetime too) value in the RREQ 
Option, right?  Please be specific.

Fixed.


 >
 > [nit] s/entry with same source/entry with the same source

Fixed.


 >
 > [major] "A route entry with same source and destination address, same 
InstanceID, but stale sequence number, SHOULD be deleted."  When would 
it be ok to not delete the entry?  IOW, why is that not a MUST?

Changed to MUST.


 >
 > 689         If the 'H' bit is set to 0, an intermediate router MUST 
include
 > 690         the address of the interface receiving the RREQ-DIO into the
 > 691         address vector.
 >
 > [major] Step 3 (at least the first paragraph) seems to be about 
building a route entry.  What is different if the H bit is set to 0?  
How is the route entry built?

I think that if H=0 there is no route entry, since source routing is to 
be used.


 >
 > [minor] This last paragraph talks about forwarding the RREQ, so 
perhaps it fits better in the next step.
Done, even though Step 4 now discusses more than just forwarding the RREQ.
 >
 > 693      Step 4:
 >
 > 695         An intermediate router transmits a RREQ-DIO via link-local
 > 696         multicast.  TargNode prepares a RREP-DIO.
 >
 > [minor] "TargNode prepares a RREP-DIO."  Please add a forward 
reference to §6.3.

Done.


 >
 > 698    6.2.2.  Additional Processing for Multiple Targets
 >
 > 700      If the OrigNode tries to reach multiple TargNodes in a 
single RREQ-
 > 701      instance, one of the TargNodes can be an intermediate router 
to the
 > 702      others, therefore it SHOULD continue sending RREQ-DIO to 
reach other
 > 703      targets.  In this case, before rebroadcasting the RREQ-DIO, a
 > 704      TargNode MUST delete the Target Option encapsulating its own 
address,
 > 705      so that downstream routers with higher ranks do not try to 
create a
 > 706      route to this TargetNode.
 >
 > [major] "...one of the TargNodes can be an intermediate router to the 
others, therefore it SHOULD continue sending RREQ-DIO to reach other 
targets."  When would a TargNode choose not to forward the RREQ?  IOW, 
why is that not a MUST?

Changed to MUST.


 >
 > 708      An intermediate router could receive several RREQ-DIOs from 
routers
 > 709      with lower ranks in the same RREQ-instance but have 
different lists
 > 710      of Target Options.  When rebroadcasting the RREQ-DIO, the
 > 711      intersection of these lists SHOULD be included.  For 
example, suppose
 > 712      two RREQ-DIOs are received with the same RPLInstance and 
OrigNode.
 > 713      Suppose further that the first RREQ has (T1, T2) as the 
targets, and
 > 714      the second one has (T2, T4) as targets.  Then only T2 needs 
to be
 > 715      included in the generated RREQ-DIO.  If the intersection is 
empty, it
 > 716      means that all the targets have been reached, and the router 
SHOULD
 > 717      NOT send out any RREQ-DIO.  Any RREQ-DIO message with 
different ART
 > 718      Options coming from a router with higher rank is ignored.
 >
 > [major] "...the intersection of these lists SHOULD be included."  
When would a node not include the intersection?  IOW, why is this not a 
MUST?

Changed to MUST.
 >
 > [major] "If the intersection is empty...the router SHOULD NOT send 
out any RREQ-DIO."  When would it be ok to sent the RREQ out? If there 
are no targets, then it seems like you would never want to.  IOW, why is 
that not a MUST NOT?

Changed to MUST NOT.
 >
 > [minor] I'm not sure about the intersection logic above...but maybe 
I'm missing something.  The example seems to imply that every RREQ 
should include all outstanding targets (?)...so that comparing the first 
one (T1, T2) with the second (T2, T4), implies that T1 has been 
found...is that correct?  If so, then it looks like T4 is still a valid 
target, but the intersection wouldn't include it.  What am I missing?

It is assumed that OrigNode was the eventual origination point for both 
the example incoming RREQs.  So the inference is that one path 
discovered a route for T4, and the other path discovered a route for T1, 
and neither path discovered a route for T2.  And, one might speculate, 
both paths discovered a route for T3 which isn't mentioned in the example.


 >
 > [minor] Related:  The specification above only applies to the period 
of time between receiving the first RREQ and the initial rebroadcast, 
right?  IOW, if a RREQ is received and rebroadcast....and then a second 
RREQ is received (with a different list of targets), then the 
rebroadcast should not include just the intersection, right?

That is correct.  The following text was added to clarify this point:

                                                           For the
         purposes of determining the intersection with previous incoming
         RREQ-DIOs, the intermediate router maintains a record of the
         targets that have been requested associated with the RREQ-Instance.


 >
 > 720    6.3.  Generating Route Reply (RREP) at TargNode
 >
 > 722    6.3.1.  RREP-DIO for Symmetric route
 >
 > 724      If a RREQ-DIO arrives at TargNode with the 'S' bit set to 1, 
there is
 > 725      a symmetric route along which both directions can fulfill the
 > 726      requirements.  Other RREQ-DIOs might later provide 
asymmetric upward
 > 727      routes (i.e.  S=0).  Selection between a qualified symmetric 
route
 > 728      and an asymmetric route that might have better performance is
 > 729      implementation-specific and out of scope.  If the 
implementation uses
 > 730      the symmetric route, the TargNode MAY delay transmitting the 
RREP-DIO
 > 731      for duration RREP_WAIT_TIME to await a better symmetric route.
 >
 > [major] RREP_WAIT_TIME is not defined anywhere.

By default it is set to equal 1/4 of the value determined by the L bit.


 >
 > [major] "...a better symmetric route."  What makes a route better?

I changed the sentence to say lower Rank.
 >
 > 733      For a symmetric route, the RREP-DIO message is unicast to 
the next
 > 734      hop according to the accumulated address vector (H=0) or the 
route
 > 735      entry (H=1).  Thus the DODAG in RREP-Instance does not need 
to be
 > 736      built.  The RPLInstanceID in the RREP-Instance is paired as 
defined
 > 737      in Section 6.3.3.  In case the 'H' bit is set to 0, the address
 > 738      vector received in the RREQ-DIO MUST be included in the 
RREP-DIO.
 > 739      TargNode increments its current sequence number and uses the
 > 740      incremented result in the Dest SeqNo in the ART option of 
the RREQ-
 > 741      DIO.  The address of the OrigNode MUST be encapsulated in 
the ART
 > 742      Option and included in this RREP-DIO message.
 >
 > 744    6.3.2.  RREP-DIO for Asymmetric Route
 >
 > 746      When a RREQ-DIO arrives at a TargNode with the 'S' bit set 
to 0, the
 > 747      TargNode MUST build a DODAG in the RREP-Instance rooted at 
itself in
 > 748      order to discover the downstream route from the OrigNode to the
 > 749      TargNode.  The RREP-DIO message MUST be re-transmitted via 
link-local
 > 750      multicast until the OrigNode is reached or MaxRank is exceeded.
 >
 > [minor] The previous section talked about delaying the RREP. Should 
that be considered here too?

A similar delay specification has been added.


 >
 > 752      The settings of the fields in RREP option and ART option are 
the same
 > 753      as for the symmetric route, except for the 'S' bit.
 >
 > 755    6.3.3.  RPLInstanceID Pairing
 >
 > [minor] Pairing only applied to symmetric routes, right? Please say so.

I think it is O.K. to mandate pairing for asymmetric routes.


 >
 > 757      Since the RPLInstanceID is assigned locally (i.e., there is no
 > 758      coordination between routers in the assignment of 
RPLInstanceID), the
 > 759      tuple (OrigNode, TargNode, RPLInstanceID) is needed to uniquely
 > 760      identify a discovered route.  The upper layer applications 
may have
 > 761      different requirements and they can initiate the route 
discoveries
 > 762      simultaneously.  Thus between the same pair of OrigNode and 
TargNode,
 > 763      there can be multiple AODV-RPL instances.  To avoid any 
mismatch, the
 > 764      RREQ-Instance and the RREP-Instance in the same route 
discovery MUST
 > 765      be paired somehow, e.g. using the RPLInstanceID.
 >
 > [minor] "The upper layer applications may have different requirements 
and they can initiate the route discoveries simultaneously."  The 
applications don't initiate route discovery...  Application requirements 
are mentioned several times in this document, but it is not clear to me 
how those requirements are reflected in the RREQ.

Fixed.


 >
 > [major] "..MUST be paired somehow, e.g. using the RPLInstanceID."  
There is no clear Normative action here. s/MUST/must

I think that the reformulated sentence uses MUST appropriately.


 >
 >
 > ...
 > 782    6.4.  Receiving and Forwarding Route Reply
 >
 > [-] Some of the comments above apply to this section too.

I hope they have been addressed but if not please advise.


 >
 >
 > ...
 > 803         If the constraints are not fulfilled, the router MUST NOT 
join the
 > 804         DODAG; the router MUST discard the RREQ-DIO, and does not 
execute
 > 805         the remaining steps in this section.
 >
 > [nit] s/and does not execute/and not execute

Fixed.


 >
 >
 > ...
 > 813      Step 3:
 >
 > 815         If the 'H' bit is set to 1, then the router (OrigNode or
 > 816         intermediate) MUST build a downward route entry. The 
route entry
 > 817         SHOULD include at least the following items: OrigNode 
Address,
 > 818         InstanceID, TargNode Address as destination, Next Hop, 
Lifetime
 > 819         and Sequence Number.  For a symmetric route, the next hop 
in the
 > 820         route entry is the router from which the RREP-DIO is 
received.
 > 821         For an asymmetric route, the next hop is the preferred 
parent in
 > 822         the DODAG of RREQ-Instance.  The InstanceID in the route 
entry
 > 823         MUST be the original RPLInstanceID (after subtracting the 
Shift
 > 824         field value).  The source address is learned from the ART 
Option,
 > 825         and the destination address is learned from the DODAGID.  The
 > 826         lifetime is set according to DODAG configuration and can be
 > 827         extended when the route is actually used.  The sequence 
number
 > 828         represents the freshness of the route entry, and is 
copied from
 > 829         the Dest SeqNo field of the ART option of the RREP-DIO.  
A route
 > 830         entry with same source and destination address, same 
InstanceID,
 > 831         but stale sequence number, SHOULD be deleted.
 >
 > [major] The description in §6.2.1 says that the "route entry MUST 
include...".  Why is SHOULD used in this case?  When is it ok to not 
include these items?  Should the same apply to §6.2.1?

Changed to MUST here also.


 >
 >
 > ...
 > 851    7.  Gratuitous RREP
 >
 > [minor] This section introduces T and O (instead of 
TargNode/OrigNode) to explain the operation.  That is not a bad thing, 
but I think that having consistent terminology is a really good thing.

Changed to TargNode / OrigNode.


 >
 >
 > ...
 > 872      In case of hop-by-hop routing, R MUST unicast the received 
RREQ-DIO
 > 873      hop-by-hop to T.  The routers along the route SHOULD build 
new route
 > 874      entries with the related RPLInstanceID and DODAGID in the 
downward
 > 875      direction.  Then T MUST unicast the RREP-DIO hop-by-hop to 
R, and the
 > 876      routers along the route SHOULD build new route entries in 
the upward
 > 877      direction.  Upon receiving the unicast RREP-DIO, R sends the
 > 878      gratuitous RREP-DIO to the OrigNode as defined in Section 6.3.
 >
 > [major] I don't understand how the "routers along the route" can do 
anything if the messages are unicast...??

The intention is for the routers along the way to be part of the new 
RPLInstance and DODAG.  They don't have to change the next hop information.


 >
 > 880    8.  Operation of Trickle Timer
 >
 > 882      The trickle timer operation to control 
RREQ-Instance/RREP-Instance
 > 883      multicast is similar to that in P2P-RPL [RFC6997].
 >
 > [major] "The trickle timer operation...is similar to that in P2P-RPL 
[RFC6997]."  Similar, or the same??
 >
 > Note that if it is the same, then rfc6997 would have to be a 
Normative Reference.

TODO: rewrite section 8.


 >
 > 885    9.  IANA Considerations
 >
 > 887    9.1.  New Mode of Operation: AODV-RPL
 >
 > 889      IANA is required to assign a new Mode of Operation, named 
"AODV-RPL"
 > 890      for Point-to-Point(P2P) hop-by-hop routing under the RPL 
registry.
 > 891      The value of TBD1 is assigned from the "Mode of Operation" space
 > 892      [RFC6550].
 >
 > [major] NEW>
 >    IANA is asked to assign a new Mode of Operation, named "AODV-RPL"
 >    for Point-to-Point(P2P) hop-by-hop routing from the "Mode of 
Operation"
 >    Registry [RFC6550].
 >
Done.


 >
 > 894 +-------------+---------------+---------------+
 > 895                 |    Value    |  Description  | Reference   |
 > 896 +-------------+---------------+---------------+
 > 897                 |   TBD1 (5)  |   AODV-RPL    | This document |
 > 898 +-------------+---------------+---------------+
 >
 > 900                           Figure 6: Mode of Operation
 >
 > 902    9.2.  AODV-RPL Options: RREQ, RREP, and Target
 >
 > 904      Three entries are required for new AODV-RPL options "RREQ", 
"RREP"
 > 905      and "ART" with values of TBD2 (0x0A), TBD3 (0x0B) and TBD4 
(0x0C)
 > 906      from the "RPL Control Message Options" space [RFC6550].
 >
 > [major] NEW>
 >    IANA is asked to assign three new AODV-RPL options "RREQ", "RREP" and
 >    "ART", as described in Figure 7 from the "RPL Control Message Options"
 >    Registry [RFC6550].
 >
 > 908 +-------------+------------------------+---------------+
 > 909             |    Value    |        Meaning         | Reference   |
 > 910 +-------------+------------------------+---------------+
 > 911             | TBD2 (0x0A) |      RREQ Option       | This document |
 > 912 +-------------+------------------------+---------------+
 > 913             | TBD3 (0x0B) |      RREP Option       | This document |
 > 914 +-------------+------------------------+---------------+
 > 915             | TBD3 (0x0C) |       ART Option       | This document |
 > 916 +-------------+------------------------+---------------+
 >
 > 918                           Figure 7: AODV-RPL Options
 >
 > 920    10.  Security Considerations
 >
 > 922      The security mechanisms defined in section 10 of [RFC6550] and
 > 923      section 11 of [RFC6997] can also be applied to the control 
messages
 > 924      defined in this specification.  The RREQ-DIO and RREP-DIO 
both have a
 > 925      secure variant, which provide integrity and replay 
protection as well
 > 926      as optional confidentiality and delay protection.
 >
 > [major] rfc6997/§11 mostly talks about the processing of the new 
messages defined there.  How does that apply to this document?
 >
 > [major] rfc6997 says that section "10 of [RFC6550] describe RPL's 
security framework...This security framework can also be used in P2P-RPL 
after taking into account the constraints specified in Section 11."  How 
does that apply to this document if both "section 10 of [RFC6550] and 
section 11 of [RFC6997]" are mentioned above?
TODO: Instead of citing RFC 6997, we need to adapt the text to be 
specific for AODV-RPL messages.
 >
 > [minor] §3 talks about the fact that an RREQ-DIO is a DIO message 
with the rREQ Option (and there's similar text for the RREP-DIO).  
However, I think it's confusing to the reader to mention that there are 
secure variants.  I think that expanding the description (at the end of 
§3) of what exactly the *-DIO messages are (and that the definition 
includes the secure variants) would go a long way.

Is it O.K. if we note that RREQ and RREP have secure variants, without 
having to reproduce the entire section?


 >
 > 928      AODV-RPL can operate in the three security modes defined in
 > 929      [RFC6550].  AODV-RPL messages SHOULD use a security mode at 
least as
 > 930      strong as the security mode used in RPL.
 >
 > [major] "AODV-RPL messages SHOULD use a security mode at least as 
strong as the security mode used in RPL."  What does that mean? As I 
asked before, what is the relationship in the network between RPL and 
AODV-RPL.  I have been assuming that the Options are simply included as 
an "add-on" to the base RPL already running, but this statement seems to 
imply that they are completely independent if they can have different 
security modes.   ??

AODV-RPL does not require RPL to be running.  All routes could be 
point-to-point.  I think they can be completely independent but still 
use the same route table.  As you note, this will require some 
bookkeeping to make sure that route entries discovered by secure 
messaging are distinguishable from route entries discovered by unsecured 
messaging.


 >
 > 932      o  Unsecured.  In this mode, RREQ-DIO and RREP-DIO are used 
without
 > 933         any security fields as defined in section 6.1 of 
[RFC6550].  The
 > 934         control messages can be protected by other security 
mechanisms,
 > 935         e.g. link-layer security.  This mode SHOULD NOT be used 
when RPL
 > 936         is using Preinstalled mode or Authenticated mode (see below).
 >
 > [major] There is a Normative contradiction: (above) "This mode SHOULD 
NOT be used when RPL is using Preinstalled mode or Authenticated mode 
(see below)." ...and... (below) "Unsecured messages MUST be dropped."  
It seems to me that maybe s/SHOULD NOT/MUST NOT
Agreed.
 >
 > [major] Related:  There's no indication on what should be done with 
unsecured messages in Authenticated mode.  I'm assuming the same drop 
action.
 >
 > 938      o  Preinstalled.  In this mode, AODV-RPL uses secure 
RREQ-DIO and
 > 939         RREP-DIO messages, and a node wishing to join a secured 
network
 > 940         will have been pre-configured with a shared key.  A node 
can use
 > 941         that key to join the AODV-RPL DODAG as a host or a router.
 > 942         Unsecured messages MUST be dropped.  This mode SHOULD NOT 
be used
 > 943         when RPL is using Authenticated mode.
 >
 > [major] When is it ok to use this mode with Authenticated mode?  IOW, 
why is that not a MUST NOT?

Changed to MUST NOT.


 >
 > ...
 > 961    11.  Future Work
 >
 > [minor] This section sounds appropriate for an Experimental document 
and not one in the Standards Track.
 >
 > [major] I would expect some of the items below to be specified in a 
Standards Track document.  For instance, "the initial state of a link" 
seems pretty basic. Put another way, what should be the settings (for 
the items mentioned here)?
 >
 > 963      There has been some discussion about how to determine the 
initial
 > 964      state of a link after an AODV-RPL-based network has begun 
operation.
 > 965      The current draft operates as if the links are symmetric until
 > 966      additional metric information is collected.  The means for 
making
 > 967      link metric information is considered out of scope for 
AODV-RPL.  In
 > 968      the future, RREQ and RREP messages could be equipped with 
new fields
 > 969      for use in verifying link metrics.  In particular, it is 
possible to
 > 970      identify unidirectional links; an RREQ received across a
 > 971      unidirectional link has to be dropped, since the destination 
node
 > 972      cannot make use of the received DODAG to route packets back 
to the
 > 973      source node that originated the route discovery operation.  
This is
 > 974      roughly the same as considering a unidirectional link to 
present an
 > 975      infinite cost metric that automatically disqualifies it for 
use in
 > 976      the reverse direction.
 >
 > [major] "The current draft operates as if the links are symmetric 
until additional metric information is collected."  §6 mandates a check 
to determine the state symmetry.  How is that (unspecified) check 
related to this text?  It seems to be that it is the same thing; is it?

In AODVv2, and to a certain extent in AODV, there is text that goes into 
some detail about how to determine link symmetry.  That could be adapted 
for incorporation into AODV-RPL.  Alternatively, and perhaps preferable, 
we could specify that link symmetry may be determined by procedures that 
are out of scope for the AODV-RPL specification.  Would that be O.K.

We could also change the AODV-RPL document to specify that whether or 
not a link is symmetric is not assumed to be symmetric by default.  That 
would be O.K. with me if the determination of link symmetry is out of 
scope.  For instance, some links could do this at layer 2.


 >
 > 978    12.  Contributors
 >
 > [minor] In general Contributors are listed list before the Author's 
Address at the bottom [rfc7322].

Fixed.


 >
 >
 > ...
 > 1060    Appendix A.  Example: ETX/RSSI Values to select S bit
 >
 > [minor] Please expand ETX/RSSI on first mention (§5).
Done.
 >
 > 1062      We have tested the combination of "RSSI(downstream)" and "ETX
 > 1063      (upstream)" to determine whether the link is symmetric or 
asymmetric
 > 1064      at the intermediate nodes.  The example of how the ETX and RSSI
 > 1065      values are used in conjuction is explained below:
 >
 > [style nit] Don't write in the first person ("We have...").

Fixed.


 >
 > [minor] It would be really nice to provide a reference for these tests.
TODO: provide a citation.
 >
 > [minor] Add references for ETX/RSSI.
TODO: provide a citation.
 >
 > [nit] s/conjuction/conjunction
Fixed.
 >
 >
 > ...
 > 1083      We tested the operations in this specification by making the
 > 1084      following experiment, using the above parameters.  In our 
experiment,
 > 1085      a communication link is considered as symmetric if the ETX 
value of
 > 1086      NodeA->NodeB and NodeB->NodeA (See Figure.8) are, say, 
within 1:3
 > 1087      ratio.  This ratio should be taken as a notional metric for 
deciding
 > 1088      link symmetric/asymmetric nature, and precise definition of 
the ratio
 > 1089      is beyond the scope of the draft.  In general, NodeA can 
only know
 > 1090      the ETX value in the direction of NodeA -> NodeB but it has 
no direct
 > 1091      way of knowing the value of ETX from NodeB->NodeA.  Using 
physical
 > 1092      testbed experiments and realistic wireless channel propagation
 > 1093      models, one can determine a relationship between RSSI and ETX
 > 1094      representable as an expression or a mapping table. Such a
 > 1095      relationship in turn can be used to estimate ETX value at 
nodeA for
 > 1096      link NodeB--->NodeA from the received RSSI from NodeB.  
Whenever
 > 1097      nodeA determines that the link towards the nodeB is 
bi-directional
 > 1098      asymmetric then the "S" bit is set to "S=0".  Later on, the 
link from
 > 1099      NodeA to Destination is asymmetric with "S" bit remains to "0".
 >
 > [nit] s/Figure.8/Figure 8
 >
 > [nit] s/within 1:3 ratio/within 1:3 a ratio
 >
 > [nit] s/, and precise definition of the ratio is beyond the scope of 
the draft./.    Not needed, this is just an example.

Fixed, fixed, and fixed.


 >
 > 1101    Appendix B.  Changelog
 >
 > [nit] Add a note to the RFC Editor to remove this section before 
publication.
 >
Done.

Also added a new section about Changes between revisions ...-07 and ...-08.