Re: [Roll] Review [draft-rahul-roll-mop-ext-00]

[Rahul Arvind Jadhav <[e-mail]>]

Hello All,

We have submitted a new version (https://tools.ietf.org/html/draft-rahul-roll-mop-ext-01) based on feedback from Georgios. Thanks Georgios.

Updates include:

1.       A new section describing “Handling MOPex”. It is now possible to not have MOPex option with base MOP = 7. If MOPex is not present and base MOP is 7, than final MOP is 7 (i.e. MOPex is assumed to be 0). This avoids having to send MOPex in case of MOP=7.

2.       Restructuring of few sections

3.       Updated IANA considerations (new registries added for MOPex and Capabilities).

Regards,
Rahul


From: Georgios Z.Papadopoulos [mailto:[e-mail]]
Sent: 06 June 2019 20:26
To: Rahul Arvind Jadhav <[e-mail]>
Cc: Routing Over Low power and Lossy networks <[e-mail]>; Pascal Thubert (pthubert) <[e-mail]>
Subject: Re: Review [raft-rahul-roll-mop-ext-00]

Hi Rahul,

Please find my responses inline:

Best,
Georgios

On Jun 5, 2019, at 02:53, Rahul Arvind Jadhav <[e-mail]<mailto:[e-mail]>> wrote:

Thank you Georgios for the review and feedback.
Please find my responses inline.

Regards,
Rahul


   REQ2:  Optional capabilities handshake.  Capabilities are features,

          possibly optional, which could be handshaked between the nodes

          and the root within an RPL Instance.

[GP] Maybe few words (to give minimum background) earlier in the Introduction about the "capabilities handshake”? Otherwise it appears too unexpectedly.

[RJ] Yes. Should have introduced capabilities in Introduction section. Will do.

[GP] Great.



   REQ3:  Backwards compatibility.  The new options and new fields in

          the DIO message should be backward compatible i.e. if there

          are nodes which support old MOPs they could still operate in

          their own instances.



   REQ4:  Capabilities handshake could be optionally added with existing

          MOPs.  Capabilities been optional in nature could be put to

          use with existing MOPs.  Capabilities and MOP-extension is

          mutually independent i.e. a DIO can have a capabilities

          option, MOP-extension option or both in the same message.

[GP] Structurally, I would push REQ4 before REQ3. Since both REQ2 and 4 are related with Capabilities handshake.

[RJ] Or I could swap REQ3 and REQ2 which will make related requirements club together? But yes, I get the point of structuring things together and will update in next ver.

[GP] Yes, it also make sense.


3.  Extended MOP Control Message Option

[GP] A structural comment here, I do not see a Subsection 3.2 and so on, then I do not see why there should be 3.1.

[RJ] Will structure this properly. Should add a separate ‘Extended-MOP Base Rules’ section (which will be Section 3.2).

[GP] Great.



   This document reserves existing MOP value 7 to be used as an

   extender.

[GP] To make it clear, the value 7 did not represent something till today?

   DIO messages with MOP value of 7 MUST refer to the

   Extended MOP (MOPex) option in the DIO message.  If the MOPex option

   is absent in the DIO whose MOP is 7, then the DIO message MUST be

   silently discarded.

[RJ] MOP=7 does not represent anything today. We have added IANA consideration to reserve MOP=7. This reminds me, I need to add Extended-MOP-Value and Capabilities flag registry in the IANA consideration.

[GP] I see, good.



[GP] with the current implementation if this value is 7, what would be the case? It will be silently discarded as well?

[RJ] Thanks for raising this. I tried to check what happens as per 6550. As per 6550 Section 6.3.1, the node can still join the network with unknown MOP as a leaf node. This might have implications and may need handling.

[GP] Keep us posted if you have some news on it, please.





        0                   1                   2                   3

        0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1

       +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

       |   Type = TODO |           Extended-MOP-value                  |

       +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+



                       Figure 1: Extended MOP Option

[GP] Is it possible to have this Extended MOP option Figure within the actual DIO message?

[RJ] There are 7 bits in Reserved field in DIO base message. But I’m not sure if we can use those bits since otherwise there won’t be any Reserved space left for future play.

[GP] I see.






4.1.  Capability Control Message Option

[GP] Is there room both for MOPex and for Capability options? Or it will either be one or the other one?

[RJ] The draft allows either or both options. Will make this explicit in next ver.

[GP] Good.