Re: [lisp] Mirja Kühlewind's Discuss on draft-ietf-lisp-gpe-05: (with DISCUSS and COMMENT)

Mirja Kuehlewind <ietf@kuehlewind.net> Wed, 08 January 2020 10:21 UTC

Return-Path: <ietf@kuehlewind.net>
X-Original-To: lisp@ietfa.amsl.com
Delivered-To: lisp@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id A2E5112006F; Wed, 8 Jan 2020 02:21:54 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.897
X-Spam-Level:
X-Spam-Status: No, score=-1.897 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, SPF_HELO_NONE=0.001, SPF_NONE=0.001, URIBL_BLOCKED=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 TwWPvUbRZ-qY; Wed, 8 Jan 2020 02:21:52 -0800 (PST)
Received: from wp513.webpack.hosteurope.de (wp513.webpack.hosteurope.de [IPv6:2a01:488:42:1000:50ed:8223::]) (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 F3705120018; Wed, 8 Jan 2020 02:21:51 -0800 (PST)
Received: from 200116b82ce1a4005104981634920ee5.dip.versatel-1u1.de ([2001:16b8:2ce1:a400:5104:9816:3492:ee5]); authenticated by wp513.webpack.hosteurope.de running ExIM with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) id 1ip8TC-0001GF-Mz; Wed, 08 Jan 2020 11:21:46 +0100
Content-Type: text/plain; charset=utf-8
Mime-Version: 1.0 (Mac OS X Mail 12.4 \(3445.104.11\))
From: Mirja Kuehlewind <ietf@kuehlewind.net>
In-Reply-To: <F07DBBF7-BAEB-4D88-8552-EB3A64AC72C2@cisco.com>
Date: Wed, 8 Jan 2020 11:21:45 +0100
Cc: The IESG <iesg@ietf.org>, "lisp-chairs@ietf.org" <lisp-chairs@ietf.org>, "lisp@ietf.org" <lisp@ietf.org>, "magnus.westerlund@ericsson.com" <magnus.westerlund@ericsson.com>, "draft-ietf-lisp-gpe@ietf.org" <draft-ietf-lisp-gpe@ietf.org>, Luigi Iannone <ggx@gigix.net>
Content-Transfer-Encoding: quoted-printable
Message-Id: <7B0AFA55-F5F0-4B7A-A898-A1E175CB096B@kuehlewind.net>
References: <153738612868.21424.5753365080841918983.idtracker@ietfa.amsl.com> <c31f2457-0803-6a98-5970-10acf9782e10@cisco.com> <F07DBBF7-BAEB-4D88-8552-EB3A64AC72C2@cisco.com>
To: "Fabio Maino (fmaino)" <fmaino@cisco.com>, "BRUNGARD, DEBORAH A (ATTLABS)" <db3546@att.com>
X-Mailer: Apple Mail (2.3445.104.11)
X-bounce-key: webpack.hosteurope.de;ietf@kuehlewind.net;1578478912;99c8adcf;
X-HE-SMSGID: 1ip8TC-0001GF-Mz
Archived-At: <https://mailarchive.ietf.org/arch/msg/lisp/QUQd5EzsYyaethUi0wgH6HnK47k>
Subject: Re: [lisp] =?utf-8?q?Mirja_K=C3=BChlewind=27s_Discuss_on_draft-ietf-?= =?utf-8?q?lisp-gpe-05=3A_=28with_DISCUSS_and_COMMENT=29?=
X-BeenThere: lisp@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: List for the discussion of the Locator/ID Separation Protocol <lisp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/lisp>, <mailto:lisp-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/lisp/>
List-Post: <mailto:lisp@ietf.org>
List-Help: <mailto:lisp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/lisp>, <mailto:lisp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 08 Jan 2020 10:21:55 -0000

Hi Fabio,

Thanks for all the work. Changes look good to me and I think my discuss comments are addressed.

One small comment/nit: I think you also should define the “Reserved” field in Figure 2. It’s not mention in the text, and even though the meaning is obvious, I assume it was an oversight that it's not described.

Given the large set of changes, it’s good that another wg last call took place. I think given more or less whole document has changes, it could be approbate to also have another IETF last call and put it back on a future telechat agenda. But I let Deborah decide about this. 

Deborah what's your plan here?

Mirja



> On 8. Jan 2020, at 00:02, Fabio Maino (fmaino) <fmaino@cisco.com> wrote:
> 
> Hi Mirja,
> It took quite some time, but I think we are finally making progress with the review of draft-ietf-lisp-gpe and the related LISP RFCbis drafts (https://datatracker.ietf.org/doc/draft-ietf-lisp-rfc6830bis/
> , https://datatracker.ietf.org/doc/draft-ietf-lisp-rfc6833bis/ ).
> 
> Could you please take a look at the latest rev -13 of https://datatracker.ietf.org/doc/draft-ietf-lisp-gpe/, and let us  know if we have addressed your comments?
> 
> Wrt lisp-gpe, compared with rev -05 that you last reviewed, we have done two main changes that might help addressing your DISCUSS: 
> 1.	We have introduced the concept of shim header, along the line of what Mirja suggested in her comment. The chairs thought that the change was significant enough to require a new last call with the WG, that we did after Singapore
> 2.	 We have introduced section 4 that, following what done in RFC8085 and RFC8086, defines the scope of applicability of LISP-GPE and makes considerations related with congestion control, UDP checksum, and ethernet payload encapsulation. 
> 
> Please, let me know if you have any further question or suggestion. 
> 
> I have attached a diff from rev -05 that is the one to which your ballot comments were referring to. 
> 
> Thanks,
> Fabio
> 
> 
> On 9/20/18, 1:22 PM, "Fabio Maino" <fmaino@cisco.com> wrote:
> 
>    Thanks for your notes Mirja.
> 
>    I'll publish an updated rev this evening to consolidate the changes that 
>    I believe we have agreed upon, and then I'll work on those that are 
>    still open.
> 
>    Please see below.
> 
> 
>    On 9/19/18 12:42 PM, Mirja Kühlewind wrote:
>> Mirja Kühlewind has entered the following ballot position for
>> draft-ietf-lisp-gpe-05: Discuss
>> 
>> When responding, please keep the subject line intact and reply to all
>> email addresses included in the To and CC lines. (Feel free to cut this
>> introductory paragraph, however.)
>> 
>> 
>> Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
>> for more information about IESG DISCUSS and COMMENT positions.
>> 
>> 
>> The document, along with other ballot positions, can be found here:
>> https://datatracker.ietf.org/doc/draft-ietf-lisp-gpe/
>> 
>> 
>> 
>> ----------------------------------------------------------------------
>> DISCUSS:
>> ----------------------------------------------------------------------
>> 
>> Thanks for addressing the TSV-ART review (and Magnus for doing the review)! I
>> assume that the proposed text will be incorporated in the next version. (Would
>> have been even better if those (larger) changes would have been added before
>> the doc was put on the telechat; please update as soon as possible so other AD
>> can review that text as well).
>> 
>> However, I think the text still needs to say more about HOW the PCP should be
>> mapped to DSCPs. RFC8325 doesn't provide guidelines but a mapping for 802.11.
>> Is the same mapping applicable here?
> 
>    Agree. As pointed out by Magnus' latest email there's more investigation 
>    needed here. I'll get back on this.
> 
>> 
>> Also, I'm not an expert for that part, but I guess there also is further
>> guidance needed on HOW to map the VID...?
> 
>    This is really straightforward, as the VID is a 12-bit field, and the 
>    IID is 24-bit. Implementation that I'm aware of typically carve a 
>    portion of the IID space to encode the VID.
> 
>> 
>> 
>> ----------------------------------------------------------------------
>> COMMENT:
>> ----------------------------------------------------------------------
>> 
>> Given this doc uses the last reserved bit in the lisp header, I would really
>> like to see more discussion how the data plane lisp can still be extended. I
>> think the solution is straight-forward (define a shim layer for the extension
>> and announce this capability in the Map-Reply), however, spelling this out
>> seems to be appropriate for this doc.
> 
>    Correct, that's the idea. I'll add a sentence that states that a 
>    lisp-gpe next protocol header can be used to extend the lisp data-plane 
>    functions.
> 
> 
>    Thanks,
>    Fabio
> 
>> 
>> 
> 
> 
> 
> <Diff_ draft-ietf-lisp-gpe-05.txt - draft-ietf-lisp-gpe-13.txt.pdf>