Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Barry Leiba's No Objection on draft-ietf-roll-turnon-rfc8138-11: (wit…
…h COMMENT) Hello Barry Many thanks for your time and review. This is always appreciated : ) > ---------------------------------------------------------------------- > COMMENT: > ---------------------------------------------------------------------- > > Just a couple of very minor comments: > > “RPL” should be expanded on first use. > We should probably ask the RFC Editor to mark “DAG” and “DODAG” as “well > known”, but they are not yet so marked, so “DODAG” should be expanded on > first use. Yes, I expanded a bit the introduction. Th efirst 2 paragraphs now say " The design of Low Power and Lossy Networks (LLNs) is generally focused on saving energy, which is the most constrained resource of all. The routing optimizations in the "Routing Protocol for Low Power and Lossy Networks" [RFC6550] (RPL) such as routing along a Destination-Oriented Directed Acyclic Graph (DODAG) to a Root Node and the associated packet compression technique [RFC8138] derive from that primary concern. Enabling [RFC8138] requires a Flag Day where the network is upgraded and rebooted. Otherwise, if acting as a Leaf, a node that does not support the compression would fail to communicate; if acting as a router it would drop the compressed packets and black-hole a portion of the network. This specification enables a hot upgrade where a live network is migrated. During the migration, the compression remains inactive, until all nodes are upgraded. " > — Section 5.3 — > > It is RECOMMENDED to only deploy nodes that support [RFC8138] in a > network where the compression is turned on. > > I think I misread this the first time; it’s ambiguous, so please reword it to make > this clear. What is it that’s recommended?: 1. In a network where compression > is turned on, only deploy nodes that support 8138? 2. Don’t deploy nodes that > support 8138 unless compression is turned on? The former, 1. Interestingly, you used the same words. This is hitting my limits as a non-native. I changed to the double negative that I wanted to avoid, hoping it is more readable in the end: " Nodes that do not support [RFC8138] SHOULD NOT be deployed in a network where the compression is turned on. If that it done, the node [RFC8138] can only operate as a RUL. " > > — Section 7 — > > An attacker in the middle of the network may reset the "T" flag > > Thank you for this phrasing; I like it. > I got help. Help is good. Take care, Pascal
- Loading branch information
Showing
2 changed files
with
67 additions
and
68 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters