Re: [Gen-art] Genart last call review of draft-ietf-6tisch-6top-protocol-10

Xavi Vilajosana Guillen <xvilajosana@uoc.edu> Thu, 22 March 2018 08:41 UTC

MIME-Version: 1.0
In-Reply-To: <1795952350.395820.1520737954029.JavaMail.root@canet.uoc.es>
References: <1795952350.395820.1520737954029.JavaMail.root@canet.uoc.es>
From: Xavi Vilajosana Guillen <xvilajosana@uoc.edu>
Date: Thu, 22 Mar 2018 09:41:18 +0100
Message-ID: <CAC9+vPiqHw1F7Nc+J5vQJreahHDfusBtgVMn03W=xa+TJN2HSw@mail.gmail.com>
To: Brian Carpenter <brian.e.carpenter@gmail.com>
Cc: gen-art@ietf.org, draft-ietf-6tisch-6top-protocol.all@ietf.org, tisch <6tisch@ietf.org>
Content-Type: multipart/alternative; boundary="00000000000079f9740567fc4587"
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/SqNJeXiuPtxkN01NHQkorsepmY8>
Subject: Re: [Gen-art] Genart last call review of draft-ietf-6tisch-6top-protocol-10
Precedence: list

Dear Brian,

after the WG meeting we proceed to resolve your pointed issue. Thanks so
much for going through the draft again.

We will publish v11 with the following update on the text as you suggested.
We hope this clarifies your point.


In section 3.1.1.  2-step 6P Transaction
we added:
Race conditions MAY happen when a timeout expires while a 6P
       Response is on the air.  Other inconsistencies can also happen
       when the last L2 ACK for a 6P Response is lost or when one of the
       nodes is power cycled.  6P provides an inconsistency detection
       mechanism described in Section 3.4.6.1 to cope with such
       situations.


In section 3.1.2.  3-step 6P Transaction
we added:
  Race conditions MAY happen when a timeout expires while a 6P
       Confirmation is on the air.  Other inconsistencies can also
       happen when the last L2 ACK for a 6P Confirmation is lost or when
       one of the nodes is power cycled.  6P provides an inconsistency
       detection mechanism described in Section 3.4.6.1 to cope with
       such situations.


kind regards
Xavi


2018-03-11 4:11 GMT+01:00 Brian Carpenter <brian.e.carpenter@gmail.com>:

> Reviewer: Brian Carpenter
> Review result: Ready
>
> 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 treat these comments just
> like any other last call comments.
>
> For more information, please see the FAQ at
>
> <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.
>
> Document: draft-ietf-6tisch-6top-protocol-10
> Reviewer: Brian Carpenter
> Review Date: 2018-03-10
> IETF LC End Date: 2018-03-26
> IESG Telechat date: 2018-04-05
>
> Summary: Ready
>
> Comment:
>
> Most of my previous comments have been fixed, thanks. I still disagree
> with the authors on one point, but not enough to delay the draft:
>
> In section 3.1.1 "2-step 6P Transaction" there seems to be a rare race
> condition
> if A's timeout expires while B's Response is in flight. This will be
> detected
> later as an inconsistency (section 3.4.6.2). The authors don't think it's
> necessary
> to mention this in 3.1.1. IMHO it would be useful to mention. (Similarly
> for
> section 3.1.2, 3-step transaction.)
>
>
>


-- 
Dr. Xavier Vilajosana
Wireless Networks Lab

*Internet Interdisciplinary Institute (IN3)Professor*
(+34) 646 633 681
xvilajosana@uoc.edu <usuari@uoc.edu>
http://xvilajosana.org
http://wine.rdi.uoc.edu
Parc Mediterrani de la Tecnologia
Av Carl Friedrich Gauss 5, B3 Building
08860 Castelldefels (Barcelona). Catalonia. Spain
[image: Universitat Oberta de Catalunya]

[Gen-art] Genart last call review of draft-ietf-6… Brian Carpenter
Re: [Gen-art] Genart last call review of draft-ie… Xavi Vilajosana Guillen
Re: [Gen-art] Genart last call review of draft-ie… Brian E Carpenter
Re: [Gen-art] Genart last call review of draft-ie… Brian E Carpenter
Re: [Gen-art] Genart last call review of draft-ie… Alissa Cooper
Re: [Gen-art] Genart last call review of draft-ie… Xavi Vilajosana Guillen