Re: [Gen-art] Genart telechat review of draft-ietf-lwig-energy-efficient-08

"Carles Gomez Montenegro" <carlesgo@entel.upc.edu> Tue, 16 January 2018 11:58 UTC

Return-Path: <carlesgo@entel.upc.edu>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 3A5C612DA00; Tue, 16 Jan 2018 03:58:59 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.2
X-Spam-Level:
X-Spam-Status: No, score=-4.2 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3] autolearn=ham autolearn_force=no
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 meMocNA-n_Iw; Tue, 16 Jan 2018 03:58:56 -0800 (PST)
Received: from violet.upc.es (violet.upc.es [147.83.2.51]) (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 3F92E12D838; Tue, 16 Jan 2018 03:58:53 -0800 (PST)
Received: from entelserver.upc.edu (entelserver.upc.es [147.83.39.4]) by violet.upc.es (8.14.4/8.14.4/Debian-4.1ubuntu1) with ESMTP id w0GBwl5M018593; Tue, 16 Jan 2018 12:58:47 +0100
Received: from webmail.entel.upc.edu (webmail.entel.upc.edu [147.83.39.6]) by entelserver.upc.edu (Postfix) with ESMTP id B21891D53C1; Tue, 16 Jan 2018 12:58:46 +0100 (CET)
Received: from 147.83.113.31 by webmail.entel.upc.edu with HTTP; Tue, 16 Jan 2018 12:58:35 +0100
Message-ID: <c32e0c65b7f419d26bc81cdb21b58415.squirrel@webmail.entel.upc.edu>
In-Reply-To: <151289205656.22064.11321559673578764236@ietfa.amsl.com>
References: <151289205656.22064.11321559673578764236@ietfa.amsl.com>
Date: Tue, 16 Jan 2018 12:58:35 +0100
From: Carles Gomez Montenegro <carlesgo@entel.upc.edu>
To: Peter Yee <peter@akayla.com>
Cc: lwip@ietf.org, draft-ietf-lwig-energy-efficient.all@ietf.org, gen-art@ietf.org
User-Agent: SquirrelMail/1.4.21-1.fc14
MIME-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: 8bit
X-Priority: 3 (Normal)
Importance: Normal
X-Virus-Scanned: clamav-milter 0.99.2 at violet
X-Virus-Status: Clean
X-Greylist: ACL matched, not delayed by milter-greylist-4.3.9 (violet.upc.es [147.83.2.51]); Tue, 16 Jan 2018 12:58:47 +0100 (CET)
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/BQ_axTIS3Cn90OOUAhJGHUufzjo>
Subject: Re: [Gen-art] Genart telechat review of draft-ietf-lwig-energy-efficient-08
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 16 Jan 2018 11:58:59 -0000

Dear Peter,

Thanks a lot for your helpful review.

We plan to incorporate your suggestions in the next revision of the draft.

Best regards,

Carles


> Reviewer: Peter Yee
> Review result: Ready with Issues
>
> I am the assigned Gen-ART reviewer for this draft. The General Area
> Review Team (Gen-ART) reviews all IETF documents being processed
> by the IESG for the IETF Chair. Please wait for direction from your
> document shepherd or AD before posting a new version of the draft.
>
> For more information, please see the FAQ at
>
> <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.
>
> Document: draft-ietf-lwig-energy-efficient-08
> Reviewer: Peter Yee
> Review Date: 2017-12-09
> IETF LC End Date: None
> IESG Telechat date: 2018-01-11
>
> Summary: This Informational Track specification discusses aspects of
> network
> protocol design that can negatively affect energy consumption of
> constrained
> nodes.  It's generally informative, although it doesn't provide much
> specific
> guidance on how its recommendations can be accomplished.  Ready with
> Issues.
>
> Major issues:
>
> Minor issues:
>
> Page 8, section 3.6.1, section title: this whole section refers to IEEE
> 802.11,
> but there is no reference to primary specification given.  Furthermore, a
> specific version of IEEE 802.11 should be cited.  The current version is
> IEEE
> 802.11-2016.  Reference to IEEE 802.11v is made further down in this
> section,
> but in fact IEEE 802.11v was incorporated and subsumed into IEEE
> 802.11-2012
> and need not be separately cited any longer.  IEEE considers IEEE 802.11v
> to be
> a superseded standard.
>
> Page 9, 6th paragraph: this paragraph should be expanded and clarified.
> It
> makes the implied assumption that the protocol is operating on a
> constrained
> node, but there are (generally) two sides to a protocol.  The remote side
> may
> not be aware of the constrained nature of the device with which it is
> communicating.  And in fact, it may be communicating with a mix of
> constrained
> and unconstrained devices, so without some form of signaling, it will not
> necessarily know to synchronize with parameters that are complementary to
> the
> needs of the constrained device.  While I understand what the paragraph is
> saying, it gives the topic short shrift.
>
> Nits/editorial comments:
>
> General:
>
> Append a comma after occurrences of i.e. and e.g.
>
> Specific:
>
> Page 1, Abstract, 2nd sentence: delete the comma.
>
> Page 3, 2nd full paragraph, 1st sentence: insert a space before each of
> "[RFC6775]" and "[RFC4944]".
>
> Page 3, section 1.1: insert "and" between "[RFC7228]" and
> "[I-D.bormann-lwig-7228bis]".
>
> Page 8, section 3.4, 3rd sentence: the sentence refers to "the next
> subsection", but in fact that should be a point to subsection 3.6.  Either
> re-order subsections 3.4 and 3.5 to make the reference correct or change
> to an
> explicit "subsection 3.6".
>
> Page 8, section 3.6.1, 1st paragraph, 5th sentence: change "the latter" to
> "it".
>
> Page 9, 5th paragraph: change "client initiated" to "client-initiated".
> Change
> "network assisted" to "network-assisted".  Change "Idel" to "Idle".
> Change
> both occurrences of "save" to "saving".
>
> Page 9, section 3.6.2, 1st paragraph, 3rd sentence: delete the second
> "IPv6".
>
> Page 10, 1st full paragraph, 2nd sentence: change "since" to "from".
>
> Page 10, section 3.6.3, last sentence: chance "functionalites" to
> "functionalities".
>
> Page 13, 1st paragraph, 1st sentence: append a comma after "At the MAC
> level".
>
> Page 13, 1st paragraph, 2nd sentence: insert "a" before "paging".
>
> Page 13, 1st paragraph, 9th sentence: change "check" to "checking".
>
> Page 13, 2nd paragraph, 6th sentence: change "dummybearer" to "dummy
> bearer".
>
> Page 13, 2nd paragraph, 7th sentence: insert "a" before "longer".  Change
> "a"
> before "energy" to "an".
>
> Page 14, section 4, 2nd paragraph, 1st sentence: change "energy-efficient"
> to
> "energy efficient".
>
> Page 14, section 4, 2nd paragraph, 6th sentence: change "then" to "the"
> before
> "6LoWPAN".
>
> Page 15, 1st partial paragraph, 1st full sentence: change "alternative" to
> "alternatively".
>
> Page 15, section 5, 2nd paragraph, 6th sentence: change "energy-friendly"
> to
> "energy friendly".
>
> Page 15, section 5, 3rd paragraph, 2nd sentence: replace "allows to
> provide"
> with "provides".
>
> Page 17, section 6.4, 1st sentence: substitute "encodes" for "allows to
> encode".
>
> Page 18, section 9, 2nd paragraph: insert "The" before "Authors" and make
> "Authors" lowercase.
>
> Page 18, section 9, 3rd paragraph, 1st sentence: insert "the" before
> "IESG".
> Change "IETF87" to "IETF 87".
>
> Page 18, section 9, 3rd paragraph, 2nd sentence: Change "Jaeglli" to
> "Jaeggli".
>
> Page 19, section 11: change "the energy efficient" to "energy-efficient".
>
> Page 19, IEEE80211v reference: drop the period after "Management".
>
>