[Roll] Francesca Palombini's No Objection on draft-ietf-roll-aodv-rpl-10: (with COMMENT)

[Francesca Palombini via Datatracker <[e-mail]>]

Francesca Palombini has entered the following ballot position for
draft-ietf-roll-aodv-rpl-10: No Objection

When responding, please keep the subject line intact and reply to all
email addresses included in the To and CC lines. (Feel free to cut this
introductory paragraph, however.)


Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
for more information about DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-roll-aodv-rpl/



----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

Thank you for the work on this document. I have some minor comments/questions
below.

Francesca

1. -----

Section 2.

FP: Thank you for this very extensive (and useful) terminology section. I would
suggest to add a sentence to say that the reader is expected to be familiar
with RFC 6550 terminology. Alternatively, it might be good to add terms defined
there and used in this document, such as DODAG and DODAGID, into this section
as well. It also might improve readability to add references to documents when
appropriate (for example, DIO could reference RFC 6550).

2. -----

   to OrigNode.  Intermediate routers join the Paired DODAGs based on
   the Rank as calculated from the DIO message.  Henceforth in this

FP: Please add a reference to where Rank is first defined, and/or add it to the
terminology.

3. -----

   Target Prefix / Address
      (variable-length field) An IPv6 destination address or prefix.
      The Prefix Length field contains the number of valid leading bits
      in the prefix.  The length of the field is the least number of
      octets that can contain all of the bits of the Prefix, in other
      words Floor((7+(Prefix Length))/8) octets.  The remaining bits in

FP: "Floor((7+(Prefix Length))/8)" I am not sure where the "7+" comes from.
Noting that the Prefix Length is 7-bit long, I am tempted to say that the
number of octets calculated here also includes Prefix Length, however that is
not clear from the sentence above ("The length of the field" - I assume the
field refers to the Target Prefix / Address only). I think some clarification
is necessary.