Re: [Roll] AD Review of draft-ietf-roll-turnon-rfc8138-07

[Alvaro Retana <[e-mail]>]

On July 21, 2020 at 5:53:53 AM, Pascal Thubert wrote:


Pascal:

Hi!


...
> Agreed for the registry part.
> All we want is to say that the code should check for MOP<7 before it uses
> the flag.
> To prepare for this the current draft onlyt changes the name of the
> registry as below.

Changing the name of the registry will result in having to Update the
other documents that have option flags because they don't include a
limitation.  It would also probably require an additional registry for
MOP >= 7...just for completeness.  By the time we're done with that
justification we're already half way to RLPv2 -- in this document!

Let's leave the registry unchanged.


> "
> 6. IANA Considerations
>
> IANA is requested to assign a new option flag from the Registry for
> the "DODAG Configuration Option Flags" that was created for [RFC6550]
> as follows:
>
> +---------------+---------------------------------+-----------+
> | Bit Number | Capability Description | Reference |
> +---------------+---------------------------------+-----------+
> | 2 (suggested) | Turn on RFC8138 Compression (T) | THIS RFC |
> +---------------+---------------------------------+-----------+
>
> Table 1: New DODAG Configuration Option Flag
>
> The DODAG Configuration Option Flags defined so far will be obsolete
> for RPL Mode of Operation (MOP) above and including 7.
>
> IANA is requested to update the name of the Registry from "DODAG
> Configuration Option Flags" to "DODAG Configuration Option Flags for
> RPL MOP 0..6".
>
> When MOP values of 7 and more are defined, a new registry will be
> needed.
> "
>
> Should I remove the text after the table or is it OK?

Yes, please.



> > This should simplify more. Thoughts?
> >
> >
>
> Please keep in mind that we want the core written today to disregard the
> flag (I guess that means use a default that is TBD) for MOP >= 7. Said this
> way, we still need text, and we need to define the default. I suspect it
> should be "ON" but I need to check with the ML.

This default won't be in this document, right?



...
> > The Abstract still says that the document Updates rfc6550:
> >
> > Suggestion>
> > This document updates RFC 8138 by defining a bit in the RPL DODAG
> > Configuration Option to indicate whether compression is used within the
> > RPL Instance, and specify the behavior of when the bit is set and reset.
> >
> >
>
> Did you mean
> "
> This document updates RFC 8138 by defining a bit in the RPL DODAG
> Configuration Option to indicate whether compression is used within the
> RPL Instance, and specify the behavior of RFC 8138-capable nodes
> when the bit is set and reset.
>
> "
>
> ?

Sure.


Thanks!

Alvaro.