Re: [Tsv-art] Tsvart last call review of draft-ietf-6lo-minimal-fragment-07
Joerg Ott <ott@in.tum.de> Thu, 30 January 2020 11:20 UTC
Return-Path: <ott@in.tum.de>
X-Original-To: tsv-art@ietfa.amsl.com
Delivered-To: tsv-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 7F08D120129; Thu, 30 Jan 2020 03:20:51 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.198
X-Spam-Level:
X-Spam-Status: No, score=-4.198 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] 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 MvyMdyp97lAv; Thu, 30 Jan 2020 03:20:48 -0800 (PST)
Received: from mail-out2.informatik.tu-muenchen.de (mail-out2.in.tum.de [131.159.0.36]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 960DD120104; Thu, 30 Jan 2020 03:20:45 -0800 (PST)
Received: by mail.in.tum.de (Postfix, from userid 107) id 5B7DF1C07D0; Thu, 30 Jan 2020 12:20:43 +0100 (CET)
Received: (Authenticated sender: ott) by mail.in.tum.de (Postfix) with ESMTPSA id 051981C07CD; Thu, 30 Jan 2020 12:20:40 +0100 (CET) (Extended-Queue-bit tech_dbyaq@fff.in.tum.de)
To: "Pascal Thubert (pthubert)" <pthubert@cisco.com>, Joerg Ott <jo@acm.org>, "tsv-art@ietf.org" <tsv-art@ietf.org>
Cc: "draft-ietf-6lo-minimal-fragment.all@ietf.org" <draft-ietf-6lo-minimal-fragment.all@ietf.org>, "last-call@ietf.org" <last-call@ietf.org>, "6lo@ietf.org" <6lo@ietf.org>
References: <158037733663.10174.9948288306890965922@ietfa.amsl.com> <MN2PR11MB3565E7C6948978B820FF95C1D8040@MN2PR11MB3565.namprd11.prod.outlook.com>
From: Joerg Ott <ott@in.tum.de>
Message-ID: <15f1f8cb-aad2-2c8f-4b02-122f919c6cbc@in.tum.de>
Date: Thu, 30 Jan 2020 12:20:39 +0100
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:68.0) Gecko/20100101 Thunderbird/68.4.1
MIME-Version: 1.0
In-Reply-To: <MN2PR11MB3565E7C6948978B820FF95C1D8040@MN2PR11MB3565.namprd11.prod.outlook.com>
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Language: en-US
Content-Transfer-Encoding: 8bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/tsv-art/_PWvXQA1w4gbCEmvKLxEVxm_DB4>
Subject: Re: [Tsv-art] Tsvart last call review of draft-ietf-6lo-minimal-fragment-07
X-BeenThere: tsv-art@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Transport Area Review Team <tsv-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/tsv-art>, <mailto:tsv-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/tsv-art/>
List-Post: <mailto:tsv-art@ietf.org>
List-Help: <mailto:tsv-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/tsv-art>, <mailto:tsv-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 30 Jan 2020 11:20:51 -0000
Hi Pascal, wow, this is a super-fast fix. Looks all good to me. Best, Jörg On 30.01.20 12:01, Pascal Thubert (pthubert) wrote: > Many thanks Joerg! > > Please see below: > >> Generally, the document is in good state and almost ready to publish. A few >> nits, one on protocol state management below: >> >> Nits: >> >> The text uses both "6lo fragments" and "6LoWPAN fragments". > > Changed to 6LoWPAN fragments everywhere > > >> >> Section 5, end of 1st para: >> "Since the datagram_tag is uniquely associated to the source Link-Layer >> address of the fragment, the forwarding node MUST assign a new >> datagram_tag from its own namespace for the next hop and rewrite the >> fragment header of each fragment with that datagram_tag." >> >> This sentence is correct but it comes after the description of handling >> subsequent fragments, rather than the first one, and subsequent ones should, >> of course, not receive a new datagram_tag. Maybe move the sentence further >> up or make explicit reference to the first fragment. > > Great point, moved up. > >> >> Sect. 5, bullet list, "* a datagram_size" >> Just as a remark, this does not seem to be used. It *could* be used to check if >> a fragment beyond the end of the packet arrives, but has otherwise no >> (documented) meaning. The draft should spell out its purpose. > > Since we did not use it the simplest is to remove it 😊 > >> >> Sect. 5, bullet list, "a timer that allows discarding the stale FF state after some >> timeout" >> Surely needed, but no advice is given. There is generally no explicit statement >> when to discard the state. What should an implementation do to interoperate, >> given that upstream multiplexing of packet fragments from multiple sources >> can yield diverse intervals between consecutive fragments? Would this be the >> same timer previously used for reassembly? If so, maybe just state this. > > Great point again! Does the below work, added to the bullet? > " > The duration of the timer should be longer than that which covers the reassembly at the receiving end point. > " > >> >> Sect. 7, 2nd bullet: "attck" -> "attack" >> > > Fixed! > > > Many thanks again, Joerg. I posted 09 with the above. Please let me know if we are OK? > > All the best > > Pascal > > > > > >
- [Tsv-art] Tsvart last call review of draft-ietf-6… Joerg Ott via Datatracker
- Re: [Tsv-art] Tsvart last call review of draft-ie… Pascal Thubert (pthubert)
- Re: [Tsv-art] Tsvart last call review of draft-ie… Joerg Ott