Re: [6lo] [Gen-art] Genart telechat review of draft-ietf-6lo-fragment-recovery-12

Alissa Cooper <alissa@cooperw.in> Wed, 19 February 2020 20:40 UTC

Return-Path: <alissa@cooperw.in>
X-Original-To: 6lo@ietfa.amsl.com
Delivered-To: 6lo@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 11084120810; Wed, 19 Feb 2020 12:40:21 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.701
X-Spam-Level:
X-Spam-Status: No, score=-2.701 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_LOW=-0.7, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=cooperw.in header.b=lPTQW/Dv; dkim=pass (2048-bit key) header.d=messagingengine.com header.b=YdOwknFG
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 yMPlk9c_IxzS; Wed, 19 Feb 2020 12:40:19 -0800 (PST)
Received: from wout5-smtp.messagingengine.com (wout5-smtp.messagingengine.com [64.147.123.21]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 10A7F12082D; Wed, 19 Feb 2020 12:40:19 -0800 (PST)
Received: from compute7.internal (compute7.nyi.internal [10.202.2.47]) by mailout.west.internal (Postfix) with ESMTP id 43670346; Wed, 19 Feb 2020 15:40:17 -0500 (EST)
Received: from mailfrontend2 ([10.202.2.163]) by compute7.internal (MEProxy); Wed, 19 Feb 2020 15:40:17 -0500
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cooperw.in; h= content-type:mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; s=fm2; bh=/ OB0dMEubzBRjTaYWkStktdouh+yJF/XExxD0a02zBk=; b=lPTQW/DvYPNT4t9pf PboYUDDBah0mT+g8fYJ66atAaB9gNTpznPCaNGJSslHVYoJ3f/Iw+fHtlKYtXli8 MKoSQK2CgOU+Fb+SBflHGsky2oy/dh3enn58k5vcBTX8NjS3DQvZBf2tQ1fkOM5d 1WRvaH0TwrPu46fUuw5wL1XlqSaFyFlgy7BQJugHfQaWWNevzNXw6bPUjg4scDHV BqkNEuCXgm0N/V+8xV8MU40BnfH8oo5h0m5JM+Nuv9nm3B+uwbMn2+V57k3DO6EF K0L2tuvFasaNceEG9AP19hSWtnNpWFfuEp/Ph64A86eqJthY/WZ9Rf/SGa+lTePN Oc0XQ==
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm2; bh=/OB0dMEubzBRjTaYWkStktdouh+yJF/XExxD0a02z Bk=; b=YdOwknFG3P06VQiaXYHwy3Qtn7zG6ISr9DapiPBuSTn41Y7CZ2fAPqeq4 G4HwYO69DYQVbe8dbf3xEk4OZVmi1ayAip1UyGmKfluRlx5+MYXAd+T2jbLIKIlg tR2n3r2j4YBJANans75GDhXovWuF11FZstWwhwg0CjoCFrxKdiMtelxwk+3PuodY 90c3q8q4tb5Ti8miYl7lX/CC+NXCt8Q+18QGhgV8h43akNFjN5JJGmaYWjjZw+EE pGHRUsC2lwUjiButAtt0H3LWZSm0TlVB+KudEDPeE3jG42lQX3RUii1jXgxpEJTU 6I/uHRqoxoX5jGHvsbkcLgYJt2zQg==
X-ME-Sender: <xms:MJ1NXhqDOzRFUnkOGXP-4hraPbsDMsRLGAcEidBoFbyu8gagpW0QPw>
X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedugedrkedtgddugeduucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurheptggguffhjgffgffkfhfvofesthhqmhdthhdtjeenucfhrhhomheptehlihhs shgrucevohhophgvrhcuoegrlhhishhsrgestghoohhpvghrfidrihhnqeenucffohhmrg hinhepihgvthhfrdhorhhgnecukfhppedujeefrdefkedruddujedrkedvnecuvehluhhs thgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomheprghlihhsshgrsegtoh hophgvrhifrdhinh
X-ME-Proxy: <xmx:MJ1NXtSxY5lRJ7yBdxRbEbPvdavAC1MBAUgOvJ07U8_3Qf78toYq0g> <xmx:MJ1NXhqXDNWbVDVdiizNDJJh4dc38jcA8G-leUyOD1Ckczm4fMsSug> <xmx:MJ1NXjySHUxMq3I6_IgjD-B5X2HGqxMlOJNGDNaSjBzcRw-C6Swa3Q> <xmx:MJ1NXhKq5oJhDGX7kPHqQNYxro82m-gxLcQ_KXInESxlx8nwS5C5mA>
Received: from rtp-alcoop-nitro2.cisco.com (unknown [173.38.117.82]) by mail.messagingengine.com (Postfix) with ESMTPA id B051B30606E9; Wed, 19 Feb 2020 15:40:15 -0500 (EST)
Content-Type: text/plain; charset=utf-8
Mime-Version: 1.0 (Mac OS X Mail 11.5 \(3445.9.1\))
From: Alissa Cooper <alissa@cooperw.in>
In-Reply-To: <MN2PR11MB356552E520447243C9C44940D8110@MN2PR11MB3565.namprd11.prod.outlook.com>
Date: Wed, 19 Feb 2020 15:40:16 -0500
Cc: "gen-art@ietf.org" <gen-art@ietf.org>, "last-call@ietf.org" <last-call@ietf.org>, "draft-ietf-6lo-fragment-recovery.all@ietf.org" <draft-ietf-6lo-fragment-recovery.all@ietf.org>, "6lo@ietf.org" <6lo@ietf.org>
Content-Transfer-Encoding: quoted-printable
Message-Id: <03DF1F75-9A96-400D-9E44-999EAECF5F8F@cooperw.in>
References: <158188989907.5918.5443458035448081186@ietfa.amsl.com> <MN2PR11MB356552E520447243C9C44940D8110@MN2PR11MB3565.namprd11.prod.outlook.com>
To: "Pascal Thubert (pthubert)" <pthubert@cisco.com>, Peter Yee <peter@akayla.com>
X-Mailer: Apple Mail (2.3445.9.1)
Archived-At: <https://mailarchive.ietf.org/arch/msg/6lo/UXZZCsLYQx9uKH9Eb4K2c7vp3hA>
Subject: Re: [6lo] [Gen-art] Genart telechat review of draft-ietf-6lo-fragment-recovery-12
X-BeenThere: 6lo@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "Mailing list for the 6lo WG for Internet Area issues in IPv6 over constrained node networks." <6lo.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/6lo>, <mailto:6lo-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/6lo/>
List-Post: <mailto:6lo@ietf.org>
List-Help: <mailto:6lo-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/6lo>, <mailto:6lo-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 19 Feb 2020 20:40:21 -0000

Peter, thanks for your review. Pascal, thanks for your response. I entered a No Objection ballot.

Best,
Alissa


> On Feb 18, 2020, at 10:21 AM, Pascal Thubert (pthubert) <pthubert@cisco.com> wrote:
> 
> Many thanks for your review Peter!
> 
> I applied all your typo fix suggestions that are not discussed below.
> 
>> Page 14, 3rd paragraph, 1st sentence: change “a same” to one of “any”, “any
>> single”, or something similar.  These suggestions are based on the assumption
>> that fragments from disparate datagrams are not intermingled, otherwise
>> you’ll need some other description of what you want to do in order to allow
>> fragments to get a few hops away.
> 
> You're right, that can be any frame. Proposed change to:
> "
> When a single frequency is used by contiguous hops, the sender should insert a delay between the frames (e.g., frames carrying fragments) that are sent to the same next hop. The delay should cover multiple transmissions so as to let a frame progress a few hops and avoid hidden terminal issues.
> "
> 
>> Page 18, section 7.1, inter-frame gap, 2nd sentence: change “a same” to “the
>> same” in both places in the sentence.  Overall, I couldn’t quite parse this
>> sentence.  “may be subject to receive while transmitting” left me guessing as
>> to what exactly you wanted to convey here.  Please rewrite this sentence.
> 
> Actually I did a few lines ahead but forgot to erase those : ) There's also section 4.2.
> So why nit just erase the weird sentence? We then get
> 
> "
>   An implementation must control the rate at which it sends packets
>   over the same path to allow the next hop to forward a packet before
>   it gets the next.  In a wireless network that uses the same frequency
>   along a path, more time must be inserted to avoid hidden terminal
>   issues between fragments (more in Section 4.2).
> 
>   This is controlled by the following parameter:
> 
>   inter-frame gap:  Indicates the minimum amount of time between
>      transmissions.  The inter-frame gap protects the propagation of
>      one transmission before the next one is triggered and creates a
>      duty cycle that controls the ratio of air time and memory in
>      intermediate nodes that a particular datagram will use.
> "
> 
> 
> 
>> Page 19, OptFragmentSize
> Delete “of” before “the Hop Limit”.
> 
> Unsure: The Hop Limit is a field that can be compressed at the first hop and will need to be expanded later.
> 
> 
> Many thanks for your care, Peter. I looked up and fixed other occurrences of the similar French and I hope the doc reads better now. 
> 
> If you agree with the points above I'll submit a new round asap, maybe together with another review.
> 
> All the best,
> 
> Pascal
> 
> _______________________________________________
> Gen-art mailing list
> Gen-art@ietf.org
> https://www.ietf.org/mailman/listinfo/gen-art