Re: [Roll] draft-ietf-roll-efficient-npdao review

Rahul Jadhav <rahul.ietf@gmail.com> Thu, 27 September 2018 08:55 UTC

Return-Path: <rahul.ietf@gmail.com>
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 A7EA2130E27; Thu, 27 Sep 2018 01:55:50 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.998
X-Spam-Level:
X-Spam-Status: No, score=-1.998 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com
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 304E8rTss_vH; Thu, 27 Sep 2018 01:55:48 -0700 (PDT)
Received: from mail-ua1-x931.google.com (mail-ua1-x931.google.com [IPv6:2607:f8b0:4864:20::931]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 070B0130E1D; Thu, 27 Sep 2018 01:55:48 -0700 (PDT)
Received: by mail-ua1-x931.google.com with SMTP id r15-v6so670326uao.1; Thu, 27 Sep 2018 01:55:47 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=PT6ikekQIi6QvvZwAcoVe+AJVBKLZOaqRsZx+WIXJI4=; b=bLgyjpib2vXXSJe1Ubu9VWcdTrvMsg7/sLEe8wXB12+qhHYKKY+6kPyrF9rtZIZcFG gycUtlR5Vm+epLXm+B2BmBu3/rFr8dw1cGb95yWvAGZYmaiRnvKnxCTxapLSYYUtFynD tqp+9fMFgOUynHQaYt/9j1B6uGxVpaJuUpTGuBZebqJSN7p4WZzJRBokuIrGinHsjH1i EYKm9lhCZ55IHDWgayp/OHDSymugbZOoZ1QKdIU8uSMF1f0Lj8d0TVGCYT+HFG3WkwGk fLenxq8bUrFBaaZnb7yw+YuT7EomU4L0tlznzTLMhywAjqF1sqmoljfMw/Xn7rj40BRh LpFg==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=PT6ikekQIi6QvvZwAcoVe+AJVBKLZOaqRsZx+WIXJI4=; b=X/fUax3JKKYkeeze99BiXZfUCriYwbzcbDL9nxrBG2qf/O4yYwWtgsWLHTQ3spmSLu kzofQ/57d40DqjQjZ0vMonMpSnB/3EHQd07RIKcqoZBpEZPAAaeppWMy5Q80EqQy4hIm dlGehwJoD3dmtckqUn4S5AdFldOQM1ItekwuJ4QtZbwvhCuhw/su9HASzx78SaowPmIY kYbl8UgTejF5+PC7KLfcBgX2d2eZ0fW2a07dqIIjlT26ANUY8LEi+wwG33QZWV32/TZG vtaH6jBBlK6n9PmiA6eMji5tQpgctA0QpPPXMCnQ1AgwgCxLsKFH7HmqfqsGWLvlJ3EI zNWQ==
X-Gm-Message-State: ABuFfogrVQKtPGGMvUlkw+p6B6Pj0pyV4PO9irAIrWjDE6nuOqypKozi qdHxEmVWikcK3iuvyPj9sgN+0tMI0dIxP1ZKkCgFd4qJ
X-Google-Smtp-Source: ACcGV60RHmuHKDG0udtNSq7o3Lp8NL2XE+D9OcK+ywt1nXAv8Hc2/z50TXDoFFK4lBIIGFCMDSvu0ILyTR2AWjzMmp8=
X-Received: by 2002:ab0:4ade:: with SMTP id t30-v6mr3214414uae.35.1538038546880; Thu, 27 Sep 2018 01:55:46 -0700 (PDT)
MIME-Version: 1.0
References: <65ac8a93279212d8757d1ecf24ac9e42@bbhmail.nl> <CAO0Djp0HAMccSR+G6Zd=0TTPSmo0+mKv_XxAQG95vqQ+rDeN1w@mail.gmail.com> <E78532A5-D8E5-4394-921B-C67527099615@cisco.com>
In-Reply-To: <E78532A5-D8E5-4394-921B-C67527099615@cisco.com>
From: Rahul Jadhav <rahul.ietf@gmail.com>
Date: Thu, 27 Sep 2018 14:25:35 +0530
Message-ID: <CAO0Djp1r+2wTnbNv+7VB8wHH9=qRKEyaczrDz=HxVn021mAvfQ@mail.gmail.com>
To: "Pascal Thubert (pthubert)" <pthubert@cisco.com>
Cc: "consultancy@vanderstok.org" <consultancy@vanderstok.org>, Routing Over Low power and Lossy networks <roll@ietf.org>, draft-ietf-roll-efficient-npdao@ietf.org
Content-Type: multipart/alternative; boundary="0000000000003cdda00576d6817b"
Archived-At: <https://mailarchive.ietf.org/arch/msg/roll/au19Bobkqquz_ftxcoYMYBhe2BQ>
Subject: Re: [Roll] draft-ietf-roll-efficient-npdao review
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, 27 Sep 2018 08:55:51 -0000

Hello Pascal,

Thank you for the review. Have incorporated your comments and updated the
draft (-07).
Please find my response inline.

Best,
Rahul

On Sat, 8 Sep 2018 at 16:46, Pascal Thubert (pthubert) <pthubert@cisco.com>
wrote:

> Dear all
>
> I made my own review and find the document mostly ready. I agree that
> Peter’s comments need handling to make the best text cleaner. The
> introduction could also be tightened and the English improved a bit, but I
> guess the rfc editor will help us there.
>
> A word on DAO vs. NPDAO with a same sequence would clarify that the DAO is
> valid and the route installed or kept depending on the order of the 2
> messages. A DAO received after a NPDAO but with an older sequence is
> ignored. This is classical RPL, but mentioning it could still be useful.
>

[RJ]: Have updated Section 4.3.3 "Path Sequence Number in DCO"   to
explicitly state the sequence number handling.

>
> One last thing could be a bit more details on when to place options and
> why.
>

[RJ]: Have updated Section 4.2 to explain this.


>
> Otherwise all good for me!
>
> Pascal
>
> Le 6 sept. 2018 à 18:07, Rahul Jadhav <rahul.ietf@gmail.com> a écrit :
>
> Many thanks Peter for the review. We will work on these comments and
> update the draft and provide detailed response soon.
>
> Thanks,
> Rahul
>
> On Thu, 6 Sep 2018 at 6:40 PM, Peter van der Stok <stokcons@bbhmail.nl>
> wrote:
>
>> Hi authors,
>>
>> Many thanks for this work and solving an open important problem.
>>
>> being the shepherd of this draft, I looked at the text to look for
>> improvements to facilitate the reading by the IESG.
>> I found the document difficult to read due to the use of multiple terms
>> for the same concept.
>> I would appreciate that the first 3 sections are improved, such that I
>> can continue the review afterwards without me wondering about my choices in
>> the interpretation of the text.
>>
>> For example: is a sub-node the same thing as a child in the node? and
>> what is a dependent node?
>> Some suggestions to improve the document are written below.
>>
>> Can you use, for example, the terms Switching-node (S-node)
>> After-switching parent (A-parent) and Before-switching parent (B-parent);
>> that will simplify the text and improve readability.
>> Please use consistently throughout one DODAG terminology like parent,
>> child, ancestor or another preferred terminology.
>> What does "target" mean: an endpoint of a path, a field in a packet, or
>> the S-node?
>> I thought that there can be many common ancestors on two paths with same
>> source and destination, and one of them is the first common ancestor. BTW
>> is the first one important; The rest of the text does not seem to rely on
>> it.
>>
>> In section 1.4; to which what subtree do you refer?
>>
>> In section 8.2.1 of RFC6550 it is mentioned that there can be a set of
>> preferred parents.
>> However, I have the impression that your assumption is the existence of a
>> single preferred parent, Some explanatory text would help.
>>
>> Some textual suggestions:
>> Every first introduction of an acronym must be introduced with the full
>> name; for example: DAO must be written out both in the abstract as in the
>> text for the first use; please look for all the first instances in the text.
>> In section 1.1 it will help if the list of used terms of RFC6550 is added.
>>
>> Section 3; a reminder that this applies only to storing mode may help.
>>
>> 3.1 How can transmission be tolerant to a link failure when the link has
>> disappeared completely or the node has been removed; some explanation is
>> needed here. It would help if the assumptions are listed: like there is
>> another path via ....
>>
>> In section 3.1, 3.2 and 3.3 it should not be necessary to repeat what is
>> written in section 2.1, 2.2 and 2.3, just a reference to the corresponding
>> section will do.
>>
>> Section 7 is rather short; I doubt that it will be accepted in this form.
>> There have been earlier comments on the insufficiency of the security
>> considerations in RPL documents,
>>
>> I hope I conveyed my problems sufficiently, and look forward to a new
>> version to continue the review.
>>
>> All the best,
>>
>> Peter
>>
>>
>> --
>> Peter van der Stok
>> vanderstok consultancy
>> mailto: consultancy@vanderstok.org, stokcons@bbhmail.nl
>> www: www.vanderstok.org
>> tel NL: +31(0)492474673     F: +33(0)966015248
>>
>