Re: [mpls] Update to the bullets on the first page of my review of draft-song-mpls-extension-header-09

Loa Andersson <loa@pi.nu> Tue, 30 August 2022 15:13 UTC

Return-Path: <loa@pi.nu>
X-Original-To: mpls@ietfa.amsl.com
Delivered-To: mpls@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 39846C15C519; Tue, 30 Aug 2022 08:13:55 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.908
X-Spam-Level:
X-Spam-Status: No, score=-1.908 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, NICE_REPLY_A=-0.001, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=0.001, SPF_NONE=0.001, T_SCC_BODY_TEXT_LINE=-0.01] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([50.223.129.194]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id X2imG4WXUMpE; Tue, 30 Aug 2022 08:13:53 -0700 (PDT)
Received: from pipi.pi.nu (pipi.pi.nu [83.168.239.141]) (using TLSv1.1 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 4F0E4C15948D; Tue, 30 Aug 2022 08:10:36 -0700 (PDT)
Received: from [83.227.6.201] (c-c906e353.020-236-73746f24.bbcust.telenor.se [83.227.6.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: loa@pi.nu) by pipi.pi.nu (Postfix) with ESMTPSA id E7B7636854E; Tue, 30 Aug 2022 17:10:33 +0200 (CEST)
Message-ID: <b718cd04-c822-b744-0ca2-9c3e47ea2f62@pi.nu>
Date: Tue, 30 Aug 2022 17:10:33 +0200
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.13.0
Content-Language: en-CA
To: Tianran Zhou <zhoutianran@huawei.com>, "mpls@ietf.org" <mpls@ietf.org>
Cc: "draft-song-mpls-extension-header@ietf.org" <draft-song-mpls-extension-header@ietf.org>, "mpls-chairs@ietf.org" <mpls-chairs@ietf.org>, "pals-chairs@ietf.org" <pals-chairs@ietf.org>, DetNet Chairs <detnet-chairs@ietf.org>
References: <98356ad3-ec52-3a5c-ee31-1ee604d4b5db@pi.nu> <dc03203bf7ee49caafccdb70221edc1a@huawei.com>
From: Loa Andersson <loa@pi.nu>
In-Reply-To: <dc03203bf7ee49caafccdb70221edc1a@huawei.com>
Content-Type: text/plain; charset="UTF-8"; format="flowed"
Content-Transfer-Encoding: 8bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/mpls/dDOehiKnl1eCGfii4-XKezxfREs>
Subject: Re: [mpls] Update to the bullets on the first page of my review of draft-song-mpls-extension-header-09
X-BeenThere: mpls@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: Multi-Protocol Label Switching WG <mpls.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mpls>, <mailto:mpls-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/mpls/>
List-Post: <mailto:mpls@ietf.org>
List-Help: <mailto:mpls-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mpls>, <mailto:mpls-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 30 Aug 2022 15:13:55 -0000

Tianran,

Inline please.

On 2022-08-30 02:26, Tianran Zhou wrote:
> Hi Loa,
> Thanks very much for your review.
> Some quick answer to your major concerns.
> Please see in line with my reply.
> 
> Best,
> Tianran
> 
> -----Original Message-----
> From: Loa Andersson [mailto:loa@pi.nu]
> Sent: Tuesday, August 30, 2022 12:14 AM
> To: mpls@ietf.org
> Cc: draft-song-mpls-extension-header@ietf.org; mpls-chairs@ietf.org; pals-chairs@ietf.org; DetNet Chairs <detnet-chairs@ietf.org>
> Subject: Update to the bullets on the first page of my review of draft-song-mpls-extension-header-09
> 
> Folks,
> 
> I just sent out a mail with my review of draft-song-mpls-extension-header-09.
> 
> It was very quickly pointed that I have a bunch of typos in the three bullets on the first page. The three bullets should have been:
> 
> •  I’m concerned about the inclusion of text on  “accessing the
>      original protocol type and payload” in this document. Accessing
>      protocol types (except for the first nibble) and payload is out of
>      scope for MPLS, and thus for MNA.
> 
> 
> ZTR> As you mentioned the "first nibble" is in the scope of MPLS. Since we have HEH, we can use some bits to quickly identify the original protocol.

If the document meant that checking the the first nibble is ok, but that 
we should not go further it should say that, not talk about "accessing 
the original protocol header and the payload.

Something that has been said so many times that I'm starting t feel hoarse:

- the first nibble in MPLS does not give a positive identification on
   "original protocol" or "payload", what it does is giving an indication
   that the payload and protocol is not IPv4 or IPv6 and load sharing
   should be avoided (RFC4928).
   For example a Ethernet Pseudowire without a control word could give
   you any result if you think the first nibble is determinism.

> 
> •  I’m still concerned that we are not considering the processing order
>      between NAIs. I believe that for NAIs using post stack data only the
>      EHs could be inserted in the right order after the HEH. However, what
>      if the NAI relies on ISD ancillary data; and that NAI must be
>      processed after one of the post stack data NAI, but before another?
>      How is that handled?
> 
> ZTR> Do you mean the case when the ISD is not enough, hence requires a PSD? 
No.

> I am not sure if there is such use case. 
Neither am I, but since that is not what I said, I think it is moot.

IMHO, if we anyway need PSD,we can put everything in the PSD. Why we 
need to split a feature into different parts?

The result of the poll was that ISD is allowed, but that the use of ISD 
need to be carefully carefully motivated on a case by case basis. The 
ISD is there, it may be used, we can't just wish it away.

What I have been talking about is the processing order between different 
NAs. The result of the processing might be effected by the order.

I tried to bring this up a couple of times, but not found to much interest.

Consider the following thought experiment.

Let us assume that we have have a packet that needs to have three NAs 
persormed, NA-a, NA-b and NA-c.

If all NAs uses post stack data, the method described in the order you 
want then executed, that could be any order e.g. NA-a, NA-b and NA-c, or 
NA-a, NA-c and NA-b or even NA-c, NA-b and NA-a, or what ever, you just 
have to determine the order.

What John (and to some extent Kireeti) proposed also have problems, if I 
understand what was proposed is that the indicators for the NAs should 
be put in a flag field in the order they were defined (or at least 
numbered in the IANA registry). If NA-a has a lower number that NA-b 
which is lower numbered than NA-c, the execution order will be NAa, NA-b 
and NA-c. And as I see it in current specifications for the proposal 
from John and Kireeti, and we can't change that. Though I could think of 
methods to change, but lets wait with that a bit.

Now let us say that NA-b uses ISD.

Let us assume that the processing order if NA-a, NA-b and NA-c.

The problem with draft-song-mpls-extension-header is that it will do 
NA-a and NA-c, but not NA-b.

The proposal from John/Kireeti would work, as long as the order of 
execution an specification is the same. But I don't think we can rely on 
that.

Hope this helps.

/Loa




> 
> •  The authors have started the process of transforming this into an MNA
>      document, but the transformation is incomplete.
> 
> 
> Sorry about this!
> 
> /Loa
> 
> 

-- 
Loa Andersson                        email: loa@pi.nu
Senior MPLS Expert                          loa.pi.nu@gmail.com
Bronze Dragon Consulting             phone: +46 739 81 21 64