Return-Path: <stephane.litkowski@orange.com>
X-Original-To: bess@ietfa.amsl.com
Delivered-To: bess@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1])
 by ietfa.amsl.com (Postfix) with ESMTP id 3D417130EA4;
 Wed, 27 Feb 2019 02:21:58 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.598
X-Spam-Level: 
X-Spam-Status: No, score=-2.598 tagged_above=-999 required=5
 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7,
 SPF_PASS=-0.001, UNPARSEABLE_RELAY=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 dY7xVHPS5vkP; Wed, 27 Feb 2019 02:21:54 -0800 (PST)
Received: from orange.com (mta136.mail.business.static.orange.com
 [80.12.70.36])
 (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))
 (No client certificate requested)
 by ietfa.amsl.com (Postfix) with ESMTPS id D1295130E27;
 Wed, 27 Feb 2019 02:21:53 -0800 (PST)
Received: from opfednr07.francetelecom.fr (unknown [xx.xx.xx.71])
 by opfednr24.francetelecom.fr (ESMTP service) with ESMTP id 448Wv43cQxz1yqH;
 Wed, 27 Feb 2019 11:21:52 +0100 (CET)
Received: from Exchangemail-eme6.itn.ftgroup (unknown [xx.xx.13.82])
 by opfednr07.francetelecom.fr (ESMTP service) with ESMTP id 448Wv42gbzzFpWv;
 Wed, 27 Feb 2019 11:21:52 +0100 (CET)
Received: from OPEXCAUBMA3.corporate.adroot.infra.ftgroup
 ([fe80::90fe:7dc1:fb15:a02b]) by OPEXCAUBM5E.corporate.adroot.infra.ftgroup
 ([fe80::849f:f804:b713:d99a%21]) with mapi id 14.03.0435.000; Wed, 27 Feb
 2019 11:21:52 +0100
From: <stephane.litkowski@orange.com>
To: "draft-ietf-bess-nsh-bgp-control-plane@ietf.org"
 <draft-ietf-bess-nsh-bgp-control-plane@ietf.org>
CC: "bess@ietf.org" <bess@ietf.org>
Thread-Topic: Shepherd's review of draft-ietf-bess-nsh-bgp-control-plane-06
Thread-Index: AdTN15CohDC0LG//QgG1ys5nLdIbiQ==
Date: Wed, 27 Feb 2019 10:21:51 +0000
Message-ID: <6687_1551262912_5C7664C0_6687_242_18_9E32478DFA9976438E7A22F69B08FF924C199D40@OPEXCAUBMA3.corporate.adroot.infra.ftgroup>
Accept-Language: fr-FR, en-US
Content-Language: en-US
X-MS-Has-Attach: yes
X-MS-TNEF-Correlator: 
x-originating-ip: [10.114.13.247]
Content-Type: multipart/related;
 boundary="_004_9E32478DFA9976438E7A22F69B08FF924C199D40OPEXCAUBMA3corp_";
 type="multipart/alternative"
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/bess/kq0NSXkEPjrvXTUQaU5vCkRZx7w>
Subject: [bess] Shepherd's review of draft-ietf-bess-nsh-bgp-control-plane-06
X-BeenThere: bess@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: BGP-Enabled ServiceS working group discussion list <bess.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/bess>,
 <mailto:bess-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/bess/>
List-Post: <mailto:bess@ietf.org>
List-Help: <mailto:bess-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/bess>,
 <mailto:bess-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 27 Feb 2019 10:21:58 -0000


--_004_9E32478DFA9976438E7A22F69B08FF924C199D40OPEXCAUBMA3corp_
Content-Type: multipart/alternative;
 boundary="_000_9E32478DFA9976438E7A22F69B08FF924C199D40OPEXCAUBMA3corp_"


--_000_9E32478DFA9976438E7A22F69B08FF924C199D40OPEXCAUBMA3corp_
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable

Hi,

Here is my review of draft-ietf-bess-nsh-bgp-control-plane-06

General comment:

The document is globally well written with good examples that help the unde=
rstanding. However it requires some refinements in the normative language u=
sed: some statement should be normative but are not using upper case words.


Detailed comments:

Abstract:

-          Please expand "BGP" on first use

-          Please give a reference associated to "Network Service Header"

Introduction:

-          When "classifier" is used as an example of SF, do you refer to t=
he classifier used to steer the traffic onto an SFC ? If yes, I don't think=
 it could be considered as an SF.

-          I think the "conventional" approach described in the intro is re=
ally old school and even before SFC, we had some means, like dynamic routin=
g protocols and other mechanisms like separate routing tables, to steer the=
 traffic through a set of services. I agree that there is also some drawbac=
k associated with such approaches like the operational complexity of provis=
ioning.

Section 1.2:
It would be good to give a definition for the additional terms that are def=
ined in this document.

Section 2.1:



=B7         As the section is focused on dataplane, it would be good to ren=
ame it as "Reminder on NSH dataplane" or something like that. It does not p=
rovide a functional overview of the controlplane which is what was expected=
 based on the title.



=B7         "A special Service Function, called a Classifier, is located at=
 each
   ingress point to a service function overlay network.  It assigns the
   packets of a given packet flow to a specific Service Function Path.
"
Again, based on my understanding, the Classifier cannot really be considere=
d as an SF , at least this is a component out of the SFC which steers the t=
raffic onto an SFC.


=B7         "An unknown or invalid SPI SHALL be treated as an error and the=
 SFF

   MUST drop the packet.  Such errors SHOULD be logged, and such logs

   MUST be subject to rate limits."



I found strange to have such statement in this document which is focused on=
 the controlplane while this statement is a dataplane statement. Isn't it p=
art of RFC8300 Section 3 : "3.  Update the NSH: SFs MUST decrement the serv=
ice index by one.  If

       an SFF receives a packet with an SPI and SI that do not

       correspond to a valid next hop in a valid SFP, that packet MUST

       be dropped by the SFF."

The next paragraph deals also with normative dataplane statement which is I=
MO out of scope of this document.





Section 2.2

=B7         "The SFIR describes a particular instance of a

      particular Service Function". Would it be good to talk about "Service=
 Function Instance" rather than instance of a Service Function ? That's und=
erstandable, of course, however I think it's good to reuse the exact termin=
ology.

=B7         I think the SFT definition is appearing a bit late in this sect=
ion and in the document as it as been referenced already multiple times bef=
ore.

=B7         "Service Function Type (SFT) that

   is the category of SF that is supported by an SFF". Don't you mean SFI r=
ather than SFF ?

=B7         "Thus the SFF can be seen as a portal...". Would "gateway" be m=
ore suitable rather than "portal" ?

=B7         The Figure 1 is not really used in this section as part of the =
existing text. I would be better to have a companion text that explains the=
 figure.

=B7         I don't like (personal opinion), the "grouping" of SFIs in the =
Figure 1 as part of an SFT. What strikes me is that it could be confused wi=
th the usual representation of an SF composed of multiple SFIs. Again that'=
s just a personal feeling.



Section 3:

=B7         "they must use BGP Capabilities": is it a normative MUST ?



Section 3.1:

=B7         s/a two byte Type field and a six byte/a two bytes Type field a=
nd a six bytes/

=B7         "Two SFIs of the same SFT must be associated". Is it a normativ=
e MUST ? Same comment for next sentences in the paragraph (multiple occuren=
ces)

=B7         "The Service Function Type identifies a service function". I do=
n't think we can really say that, it identifies the type of service the SF =
is providing but not the SF itself.

=B7         "Each node hosting an SFI

   must originate an SFIR for each type of SF that it hosts, and it may

   advertise an SFIR for each instance of each type of SF." Is it really "a=
nd" ? I mean can we just summarize by using " Each node hosting an SFI MUST=
 originate an SFIR for each instance of each type of SF" ?

=B7         "A BGP Update containing one or more SFIRs will also include". =
Is it a MUST or SHOULD "also include"?

=B7         How is the nexthop encoded in the NLRI ?

Section 3.1.1

=B7         s/"It can be included"/"It MAY be included"/ ?

=B7         That would be great to give more details about the usage of poo=
ls

=B7         The encoding in Figure 4 is not matching the text. Figure 4 has=
 Type=3D0x80, Sub-type=3DTBD6, text says type=3DTBD6 and subtype=3D0x00

=B7         How is the pool ID managed ?

Section 3.1.2

=B7         Same comment about Figure 5 not matching the text.


Section 3.2:

=B7         s/a two byte Type field and a six byte/a two bytes Type field a=
nd a six bytes/

=B7         Same comment about normative language "must" vs "MUST"

=B7         How is the nexthop encoded in the NLRI ?



Section 3.2.1:

=B7         From a readability point of view, it would be good to tell in t=
he second sentence that the requested attribute is Optional Transitive (we =
can see it in the description but it is also good to have it in the intro o=
f the attribute).

=B7         "The presence rules and meanings are as follows.". It would be =
good to use normative language in the following sentences.

=B7         I don't see the "error handling" behavior associated with this =
attribute (discard, treat-as-withdraw...)



Section 3.2.1.1

=B7         "It may be present" or "It MAY be present" ?

Section 4.1

Having an SFF being part of a single overlay network is IMO a very limited =
use case, in such a case, I don't think that you need an RT at all. The SFF=
 could be part of a L3VPN which will be used as an underlay.
An SFF being "multitenant" is more valid and defacto requires to maintain s=
eparate forwarding states (VRF like...). This is mandatory to maintain the =
tenant isolation and RTs are very useful here to know the appropriate conte=
xt to put the routes in.

Section 7.1
While I understand that the node doing the classification can perform a dee=
p packet inspection to get an entropy indicator, any intermediate node cann=
ot set it again as the NSH header will be there.
Here is an example:


PacketsIn ---> Classifier ->SFF1 (SFI1) -> SFF2(SFI2) ->SFF3(SFI3)->Packets=
Out to dest

Classifier pushes NSH header onto the packets as well as the underlay tunne=
l (using an entropy indicator in UDP or ELI/EL). Where the packets comes in=
 at SFF1, IMO, it strips the underlay tunnels including the entropy indicat=
or. Which means that when the packet with NSH header will go to SFF2, it wi=
ll not have anymore entropy indicator and there is no way to build it again=
 as the NSH header prevents a new deep packet inspection. Am I missing some=
thing ? Maintaing a per packet state is IMO not doable , especially if the =
SFI modifies the packet content.

Section 8
I would have been good to have two SFIs with the same SFT on the same SFF i=
n the example.
I don't like the representation of RD using "192.0.2.1,1" as the "," can be=
 confusing with a regular separator. Why not using :"192.0.2.1:1" notation =
which is well known ?


Section 8.9.1:
How does an SFF know that an attached SFI is stateful ? I don't think it ca=
n know that.
I don't think that the fact that SFF2 is used in both direction is safe fro=
m a loadbalancing perspective.
If the hashing algorithm used by SFF2 is sensible to the order of the keys =
(like source vs dest address, or source vs dest port), it may provide a dif=
ferent SFI as a result of the hashing between the forward and the reverse f=
low.

Section 8.9.2:
How can the controller instruct the classifier how to place traffic ?
In addition, this could involve multiple classifiers that need to be coordi=
nated.
The ingress forward classifier and the ingress reverse classifier should us=
e SFPs within the same assoc.

Section 9:
Do we have to set limits on receiving nodes in term of number of states rec=
eived from the controller to mitigate some attack ?
The text talks about security of BGP, what kind of mechanism should be put =
in place ?
Do we have any interdomain considerations ?
This is a controller based approach, so the security of the controller itse=
lf is also important.


References:
Shouldn't RFC7665 be normative ?
I think that the mpls-sfc and mpls-sfc-encaps should also be normative as y=
ou are defining a controlplane to use them.



Brgds,



[Orange logo]<http://www.orange.com/>

Stephane Litkowski
Network Architect
Orange/SCE/EQUANT/OINIS/NET
Orange Expert Future Networks
phone: +33 2 23 06 49 83 <https://monsi.sso.francetelecom.fr/index.asp?targ=
et=3Dhttp%3A%2F%2Fclicvoice.sso.francetelecom.fr%2FClicvoiceV2%2FToolBar.do=
%3Faction%3Ddefault%26rootservice%3DSIGNATURE%26to%3D+33%202%2023%2028%2049=
%2083%20>  NEW !
mobile: +33 6 71 63 27 50 <https://monsi.sso.francetelecom.fr/index.asp?tar=
get=3Dhttp%3A%2F%2Fclicvoice.sso.francetelecom.fr%2FClicvoiceV2%2FToolBar.d=
o%3Faction%3Ddefault%26rootservice%3DSIGNATURE%26to%3D+33%206%2037%2086%209=
7%2052%20>  NEW !
stephane.litkowski@orange.com<mailto:stephane.litkowski@orange.com>


___________________________________________________________________________=
______________________________________________

Ce message et ses pieces jointes peuvent contenir des informations confiden=
tielles ou privilegiees et ne doivent donc
pas etre diffuses, exploites ou copies sans autorisation. Si vous avez recu=
 ce message par erreur, veuillez le signaler
a l'expediteur et le detruire ainsi que les pieces jointes. Les messages el=
ectroniques etant susceptibles d'alteration,
Orange decline toute responsabilite si ce message a ete altere, deforme ou =
falsifie. Merci.

This message and its attachments may contain confidential or privileged inf=
ormation that may be protected by law;
they should not be distributed, used or copied without authorisation.
If you have received this email in error, please notify the sender and dele=
te this message and its attachments.
As emails may be altered, Orange is not liable for messages that have been =
modified, changed or falsified.
Thank you.


--_000_9E32478DFA9976438E7A22F69B08FF924C199D40OPEXCAUBMA3corp_
Content-Type: text/html; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable

<html xmlns:v=3D"urn:schemas-microsoft-com:vml" xmlns:o=3D"urn:schemas-micr=
osoft-com:office:office" xmlns:w=3D"urn:schemas-microsoft-com:office:word" =
xmlns:m=3D"http://schemas.microsoft.com/office/2004/12/omml" xmlns=3D"http:=
//www.w3.org/TR/REC-html40">
<head>
<meta http-equiv=3D"Content-Type" content=3D"text/html; charset=3Diso-8859-=
1">
<meta name=3D"Generator" content=3D"Microsoft Word 14 (filtered medium)">
<!--[if !mso]><style>v\:* {behavior:url(#default#VML);}
o\:* {behavior:url(#default#VML);}
w\:* {behavior:url(#default#VML);}
.shape {behavior:url(#default#VML);}
</style><![endif]--><style><!--
/* Font Definitions */
@font-face
	{font-family:Wingdings;
	panose-1:5 0 0 0 0 0 0 0 0 0;}
@font-face
	{font-family:Wingdings;
	panose-1:5 0 0 0 0 0 0 0 0 0;}
@font-face
	{font-family:Calibri;
	panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
	{font-family:Tahoma;
	panose-1:2 11 6 4 3 5 4 4 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
	{margin:0in;
	margin-bottom:.0001pt;
	font-size:11.0pt;
	font-family:"Calibri","sans-serif";}
a:link, span.MsoHyperlink
	{mso-style-priority:99;
	color:blue;
	text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
	{mso-style-priority:99;
	color:purple;
	text-decoration:underline;}
pre
	{mso-style-priority:99;
	mso-style-link:"HTML Preformatted Char";
	margin:0in;
	margin-bottom:.0001pt;
	font-size:10.0pt;
	font-family:"Courier New";}
p.MsoAcetate, li.MsoAcetate, div.MsoAcetate
	{mso-style-priority:99;
	mso-style-link:"Balloon Text Char";
	margin:0in;
	margin-bottom:.0001pt;
	font-size:8.0pt;
	font-family:"Tahoma","sans-serif";}
p.MsoListParagraph, li.MsoListParagraph, div.MsoListParagraph
	{mso-style-priority:34;
	margin-top:0in;
	margin-right:0in;
	margin-bottom:0in;
	margin-left:.5in;
	margin-bottom:.0001pt;
	font-size:11.0pt;
	font-family:"Calibri","sans-serif";}
span.EmailStyle17
	{mso-style-type:personal-compose;
	font-family:"Calibri","sans-serif";
	color:windowtext;}
span.BalloonTextChar
	{mso-style-name:"Balloon Text Char";
	mso-style-priority:99;
	mso-style-link:"Balloon Text";
	font-family:"Tahoma","sans-serif";}
span.HTMLPreformattedChar
	{mso-style-name:"HTML Preformatted Char";
	mso-style-priority:99;
	mso-style-link:"HTML Preformatted";
	font-family:"Courier New";}
.MsoChpDefault
	{mso-style-type:export-only;}
@page WordSection1
	{size:8.5in 11.0in;
	margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
	{page:WordSection1;}
/* List Definitions */
@list l0
	{mso-list-id:862550681;
	mso-list-type:hybrid;
	mso-list-template-ids:449454502 -1479754310 67698691 67698693 67698689 676=
98691 67698693 67698689 67698691 67698693;}
@list l0:level1
	{mso-level-start-at:2;
	mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	mso-ansi-font-size:12.0pt;
	font-family:Symbol;
	mso-fareast-font-family:"Times New Roman";
	mso-bidi-font-family:"Courier New";
	color:windowtext;}
@list l0:level2
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l0:level3
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l0:level4
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l0:level5
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l0:level6
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l0:level7
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l0:level8
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l0:level9
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l1
	{mso-list-id:897326461;
	mso-list-type:hybrid;
	mso-list-template-ids:1152414534 -1291577370 67698691 67698693 67698689 67=
698691 67698693 67698689 67698691 67698693;}
@list l1:level1
	{mso-level-start-at:2;
	mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	mso-ansi-font-size:12.0pt;
	font-family:Symbol;
	mso-fareast-font-family:"Times New Roman";
	mso-bidi-font-family:"Courier New";
	color:windowtext;}
@list l1:level2
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l1:level3
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l1:level4
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l1:level5
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l1:level6
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l1:level7
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l1:level8
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l1:level9
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l2
	{mso-list-id:1000155913;
	mso-list-type:hybrid;
	mso-list-template-ids:1277610034 -1479754310 67698691 67698693 67698689 67=
698691 67698693 67698689 67698691 67698693;}
@list l2:level1
	{mso-level-start-at:2;
	mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	mso-ansi-font-size:12.0pt;
	font-family:Symbol;
	mso-fareast-font-family:"Times New Roman";
	mso-bidi-font-family:"Courier New";
	color:windowtext;}
@list l2:level2
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l2:level3
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l2:level4
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l2:level5
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l2:level6
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l2:level7
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l2:level8
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l2:level9
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l3
	{mso-list-id:1086920800;
	mso-list-type:hybrid;
	mso-list-template-ids:992910466 -1479754310 67698691 67698693 67698689 676=
98691 67698693 67698689 67698691 67698693;}
@list l3:level1
	{mso-level-start-at:2;
	mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	mso-ansi-font-size:12.0pt;
	font-family:Symbol;
	mso-fareast-font-family:"Times New Roman";
	mso-bidi-font-family:"Courier New";
	color:windowtext;}
@list l3:level2
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l3:level3
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l3:level4
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l3:level5
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l3:level6
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l3:level7
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l3:level8
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l3:level9
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l4
	{mso-list-id:1161696489;
	mso-list-type:hybrid;
	mso-list-template-ids:98706434 1222947714 67698691 67698693 67698689 67698=
691 67698693 67698689 67698691 67698693;}
@list l4:level1
	{mso-level-start-at:0;
	mso-level-number-format:bullet;
	mso-level-text:-;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Calibri","sans-serif";
	mso-fareast-font-family:Calibri;
	mso-bidi-font-family:"Times New Roman";}
@list l4:level2
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l4:level3
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l4:level4
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l4:level5
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l4:level6
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l4:level7
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l4:level8
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l4:level9
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l5
	{mso-list-id:1254977078;
	mso-list-type:hybrid;
	mso-list-template-ids:-1494856768 -1479754310 67698691 67698693 67698689 6=
7698691 67698693 67698689 67698691 67698693;}
@list l5:level1
	{mso-level-start-at:2;
	mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	mso-ansi-font-size:12.0pt;
	font-family:Symbol;
	mso-fareast-font-family:"Times New Roman";
	mso-bidi-font-family:"Courier New";
	color:windowtext;}
@list l5:level2
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l5:level3
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l5:level4
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l5:level5
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l5:level6
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l5:level7
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l5:level8
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l5:level9
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l6
	{mso-list-id:1309093806;
	mso-list-type:hybrid;
	mso-list-template-ids:-542048984 -1479754310 67698691 67698693 67698689 67=
698691 67698693 67698689 67698691 67698693;}
@list l6:level1
	{mso-level-start-at:2;
	mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	mso-ansi-font-size:12.0pt;
	font-family:Symbol;
	mso-fareast-font-family:"Times New Roman";
	mso-bidi-font-family:"Courier New";
	color:windowtext;}
@list l6:level2
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l6:level3
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l6:level4
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l6:level5
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l6:level6
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l6:level7
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l6:level8
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l6:level9
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l7
	{mso-list-id:1433818951;
	mso-list-type:hybrid;
	mso-list-template-ids:730123802 -1479754310 67698691 67698693 67698689 676=
98691 67698693 67698689 67698691 67698693;}
@list l7:level1
	{mso-level-start-at:2;
	mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	mso-ansi-font-size:12.0pt;
	font-family:Symbol;
	mso-fareast-font-family:"Times New Roman";
	mso-bidi-font-family:"Courier New";
	color:windowtext;}
@list l7:level2
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l7:level3
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l7:level4
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l7:level5
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l7:level6
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l7:level7
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l7:level8
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l7:level9
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l8
	{mso-list-id:2141068915;
	mso-list-type:hybrid;
	mso-list-template-ids:289557946 -1479754310 67698691 67698693 67698689 676=
98691 67698693 67698689 67698691 67698693;}
@list l8:level1
	{mso-level-start-at:2;
	mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	mso-ansi-font-size:12.0pt;
	font-family:Symbol;
	mso-fareast-font-family:"Times New Roman";
	mso-bidi-font-family:"Courier New";
	color:windowtext;}
@list l8:level2
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l8:level3
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l8:level4
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l8:level5
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l8:level6
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l8:level7
	{mso-level-number-format:bullet;
	mso-level-text:\F0B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l8:level8
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l8:level9
	{mso-level-number-format:bullet;
	mso-level-text:\F0A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
ol
	{margin-bottom:0in;}
ul
	{margin-bottom:0in;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext=3D"edit" spidmax=3D"1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext=3D"edit">
<o:idmap v:ext=3D"edit" data=3D"1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang=3D"EN-US" link=3D"blue" vlink=3D"purple">
<div class=3D"WordSection1">
<p class=3D"MsoNormal">Hi,<o:p></o:p></p>
<p class=3D"MsoNormal"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoNormal">Here is my review of draft-ietf-bess-nsh-bgp-control=
-plane-06<o:p></o:p></p>
<p class=3D"MsoNormal"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoNormal">General comment:<o:p></o:p></p>
<p class=3D"MsoNormal"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoNormal">The document is globally well written with good exam=
ples that help the understanding. However it requires some refinements in t=
he normative language used: some statement should be normative but are not =
using upper case words.<o:p></o:p></p>
<p class=3D"MsoNormal"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoNormal"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoNormal">Detailed comments:<o:p></o:p></p>
<p class=3D"MsoNormal"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoNormal">Abstract:<o:p></o:p></p>
<p class=3D"MsoListParagraph" style=3D"text-indent:-.25in;mso-list:l4 level=
1 lfo1"><![if !supportLists]><span style=3D"mso-list:Ignore">-<span style=
=3D"font:7.0pt &quot;Times New Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&=
nbsp;&nbsp;&nbsp;&nbsp;
</span></span><![endif]>Please expand &#8220;BGP&#8221; on first use<o:p></=
o:p></p>
<p class=3D"MsoListParagraph" style=3D"text-indent:-.25in;mso-list:l4 level=
1 lfo1"><![if !supportLists]><span style=3D"mso-list:Ignore">-<span style=
=3D"font:7.0pt &quot;Times New Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&=
nbsp;&nbsp;&nbsp;&nbsp;
</span></span><![endif]>Please give a reference associated to &#8220;Networ=
k Service Header&#8221;<o:p></o:p></p>
<p class=3D"MsoNormal"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoNormal">Introduction:<o:p></o:p></p>
<p class=3D"MsoListParagraph" style=3D"text-indent:-.25in;mso-list:l4 level=
1 lfo1"><![if !supportLists]><span style=3D"mso-list:Ignore">-<span style=
=3D"font:7.0pt &quot;Times New Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&=
nbsp;&nbsp;&nbsp;&nbsp;
</span></span><![endif]>When &#8220;classifier&#8221; is used as an example=
 of SF, do you refer to the classifier used to steer the traffic onto an SF=
C ? If yes, I don&#8217;t think it could be considered as an SF.<o:p></o:p>=
</p>
<p class=3D"MsoListParagraph" style=3D"text-indent:-.25in;mso-list:l4 level=
1 lfo1"><![if !supportLists]><span style=3D"mso-list:Ignore">-<span style=
=3D"font:7.0pt &quot;Times New Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&=
nbsp;&nbsp;&nbsp;&nbsp;
</span></span><![endif]>I think the &#8220;conventional&#8221; approach des=
cribed in the intro is really old school and even before SFC, we had some m=
eans, like dynamic routing protocols and other mechanisms like separate rou=
ting tables, to steer the traffic through a
 set of services. I agree that there is also some drawback associated with =
such approaches like the operational complexity of provisioning.<o:p></o:p>=
</p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">Section 1.2:<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">It would be good to give a definitio=
n for the additional terms that are defined in this document.<o:p></o:p></s=
pan></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">Section 2.1:<o:p></o:p></span></p>
<p class=3D"MsoNormal" style=3D"text-indent:.25in"><span style=3D"font-size=
:12.0pt;font-family:&quot;Times New Roman&quot;,&quot;serif&quot;"><o:p>&nb=
sp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l8 level1 lfo3">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"color:black">As the section is focused on =
dataplane, it would be good to rename it as &#8220;Reminder on NSH dataplan=
e&#8221; or something like that. It does not provide a functional overview =
of the controlplane which is what was expected based on the title.<o:p></o:=
p></span></pre>
<pre style=3D"margin-left:.5in"><span style=3D"color:black"><o:p>&nbsp;</o:=
p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l8 level1 lfo3">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"font-size:12.0pt;font-family:&quot;Times N=
ew Roman&quot;,&quot;serif&quot;">&#8220;</span><span style=3D"color:black"=
>A special Service Function, called a Classifier, is located at each<o:p></=
o:p></span></pre>
<p class=3D"MsoNormal"><span style=3D"font-size:10.0pt;font-family:&quot;Co=
urier New&quot;;color:black">&nbsp;&nbsp; ingress point to a service functi=
on overlay network.&nbsp; It assigns the<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:10.0pt;font-family:&quot;Co=
urier New&quot;;color:black">&nbsp;&nbsp; packets of a given packet flow to=
 a specific Service Function Path.<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">&#8220;<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">Again, based on my understanding, th=
e Classifier cannot really be considered as an SF , at least this is a comp=
onent out of the SFC which steers the traffic onto an SFC.<o:p></o:p></span=
></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l1 level1 lfo2">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"font-size:12.0pt;font-family:&quot;Times N=
ew Roman&quot;,&quot;serif&quot;">&#8220;</span><span style=3D"color:black"=
>An unknown or invalid SPI SHALL be treated as an error and the SFF<o:p></o=
:p></span></pre>
<pre><span style=3D"color:black">&nbsp;&nbsp; MUST drop the packet.&nbsp; S=
uch errors SHOULD be logged, and such logs<o:p></o:p></span></pre>
<pre><span style=3D"color:black">&nbsp;&nbsp; MUST be subject to rate limit=
s.&#8221;<o:p></o:p></span></pre>
<pre><span style=3D"color:black"><o:p>&nbsp;</o:p></span></pre>
<pre><span style=3D"color:black">I found strange to have such statement in =
this document which is focused on the controlplane while this statement is =
a dataplane statement. Isn&#8217;t it part of RFC8300 Section 3 : &#8220;3.=
&nbsp; Update the NSH: SFs MUST decrement the service index by one.&nbsp; I=
f<o:p></o:p></span></pre>
<pre><span style=3D"color:black">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; an SF=
F receives a packet with an SPI and SI that do not<o:p></o:p></span></pre>
<pre><span style=3D"color:black">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; corre=
spond to a valid next hop in a valid SFP, that packet MUST<o:p></o:p></span=
></pre>
<pre><span style=3D"color:black">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; be dr=
opped by the SFF.&#8221;<o:p></o:p></span></pre>
<pre><span style=3D"color:black">The next paragraph deals also with normati=
ve dataplane statement which is IMO out of scope of this document.<o:p></o:=
p></span></pre>
<pre><span style=3D"color:black"><o:p>&nbsp;</o:p></span></pre>
<pre><span style=3D"color:black"><o:p>&nbsp;</o:p></span></pre>
<pre><span style=3D"color:black">Section 2.2<o:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l7 level1 lfo5">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"color:black">&#8220;The SFIR describes a p=
articular instance of a<o:p></o:p></span></pre>
<pre><span style=3D"color:black">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; particular =
Service Function&#8221;. Would it be good to talk about &#8220;Service Func=
tion Instance&#8221; rather than instance of a Service Function ? That&#821=
7;s understandable, of course, however I think it&#8217;s good to reuse the=
 exact terminology. <o:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l3 level1 lfo4">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"color:black">I think the SFT definition is=
 appearing a bit late in this section and in the document as it as been ref=
erenced already multiple times before.<o:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l3 level1 lfo4">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"color:black">&#8220;Service Function Type =
(SFT) that<o:p></o:p></span></pre>
<pre><span style=3D"color:black">&nbsp;&nbsp; is the category of SF that is=
 supported by an SFF&#8221;. Don&#8217;t you mean SFI rather than SFF ?<o:p=
></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l3 level1 lfo4">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"color:black">&#8220;Thus the SFF can be se=
en as a portal&#8230;&#8221;. Would &#8220;gateway&#8221; be more suitable =
rather than &#8220;portal&#8221; ?<o:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l3 level1 lfo4">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"color:black">The Figure 1 is not really us=
ed in this section as part of the existing text. I would be better to have =
a companion text that explains the figure.<o:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l3 level1 lfo4">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"color:black">I don&#8217;t like (personal =
opinion), the &#8220;grouping&#8221; of SFIs in the Figure 1 as part of an =
SFT. What strikes me is that it could be confused with the usual representa=
tion of an SF composed of multiple SFIs. Again that&#8217;s just a personal=
 feeling.<o:p></o:p></span></pre>
<pre><span style=3D"color:black"><o:p>&nbsp;</o:p></span></pre>
<pre><span style=3D"color:black">Section 3:<o:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l0 level1 lfo6">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"color:black">&#8220;they must use BGP Capa=
bilities&#8221;: is it a normative MUST ?<o:p></o:p></span></pre>
<pre><span style=3D"color:black"><o:p>&nbsp;</o:p></span></pre>
<pre><span style=3D"color:black">Section 3.1:<o:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l0 level1 lfo6">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"color:black">s/a two byte Type field and a=
 six byte/a two bytes Type field and a six bytes/<o:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l0 level1 lfo6">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"font-size:12.0pt;font-family:&quot;Times N=
ew Roman&quot;,&quot;serif&quot;">&#8220;</span><span style=3D"color:black"=
>Two SFIs of the same SFT must be associated&#8221;. Is it a normative MUST=
 ? Same comment for next sentences in the paragraph (multiple occurences)<o=
:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l0 level1 lfo6">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"font-size:12.0pt;font-family:&quot;Times N=
ew Roman&quot;,&quot;serif&quot;">&#8220;</span><span style=3D"color:black"=
>The Service Function Type identifies a service function&#8221;. I don&#821=
7;t think we can really say that, it identifies the type of service the SF =
is providing but not the SF itself.<o:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l0 level1 lfo6">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"font-size:12.0pt;font-family:&quot;Times N=
ew Roman&quot;,&quot;serif&quot;">&#8220;</span><span style=3D"color:black"=
>Each node hosting an SFI<o:p></o:p></span></pre>
<pre><span style=3D"color:black">&nbsp;&nbsp; must originate an SFIR for ea=
ch type of SF that it hosts, and it may<o:p></o:p></span></pre>
<pre><span style=3D"color:black">&nbsp;&nbsp; advertise an SFIR for each in=
stance of each type of SF.&#8221; Is it really &#8220;and&#8221; ? I mean c=
an we just summarize by using &#8220; Each node hosting an SFI MUST origina=
te an SFIR for each instance of each type of SF&#8221; ?<o:p></o:p></span><=
/pre>
<p class=3D"MsoListParagraph" style=3D"text-indent:-.25in;mso-list:l0 level=
1 lfo6"><![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Sy=
mbol"><span style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Ti=
mes New Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
</span></span></span><![endif]><span style=3D"font-size:12.0pt;font-family:=
&quot;Times New Roman&quot;,&quot;serif&quot;">&#8220;A BGP Update containi=
ng one or more SFIRs will also include&#8221;. Is it a MUST or SHOULD &#822=
0;also include&#8221;?<o:p></o:p></span></p>
<p class=3D"MsoListParagraph" style=3D"text-indent:-.25in;mso-list:l0 level=
1 lfo6"><![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Sy=
mbol"><span style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Ti=
mes New Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
</span></span></span><![endif]><span style=3D"font-size:12.0pt;font-family:=
&quot;Times New Roman&quot;,&quot;serif&quot;">How is the nexthop encoded i=
n the NLRI ?<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">Section 3.1.1<o:p></o:p></span></p>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l0 level1 lfo6">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"color:black">s/&#8220;It can be included&#=
8221;/&#8221;It MAY be included&#8221;/ ?<o:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l0 level1 lfo6">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"color:black">That would be great to give m=
ore details about the usage of pools<o:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l0 level1 lfo6">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"color:black">The encoding in Figure 4 is n=
ot matching the text. Figure 4 has Type=3D0x80, Sub-type=3DTBD6, text says =
type=3DTBD6 and subtype=3D0x00<o:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l0 level1 lfo6">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"color:black">How is the pool ID managed ?<=
o:p></o:p></span></pre>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">Section 3.1.2<o:p></o:p></span></p>
<p class=3D"MsoListParagraph" style=3D"text-indent:-.25in;mso-list:l5 level=
1 lfo7"><![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Sy=
mbol"><span style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Ti=
mes New Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
</span></span></span><![endif]><span style=3D"font-size:12.0pt;font-family:=
&quot;Times New Roman&quot;,&quot;serif&quot;">Same comment about Figure 5 =
not matching the text.<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<pre><span style=3D"color:black">Section 3.2:<o:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l0 level1 lfo6">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"color:black">s/a two byte Type field and a=
 six byte/a two bytes Type field and a six bytes/<o:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l0 level1 lfo6">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"color:black">Same comment about normative =
language &#8220;must&#8221; vs &#8220;MUST&#8221;<o:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l0 level1 lfo6">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"font-size:12.0pt;font-family:&quot;Times N=
ew Roman&quot;,&quot;serif&quot;">How is the nexthop encoded in the NLRI ?<=
/span><span style=3D"color:black"><o:p></o:p></span></pre>
<pre><span style=3D"color:black"><o:p>&nbsp;</o:p></span></pre>
<pre><span style=3D"color:black">Section 3.2.1:<o:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l2 level1 lfo8">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"color:black">From a readability point of v=
iew, it would be good to tell in the second sentence that the requested att=
ribute is Optional Transitive (we can see it in the description but it is a=
lso good to have it in the intro of the attribute).<o:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l2 level1 lfo8">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"color:black">&#8220;The presence rules and=
 meanings are as follows.&#8221;. It would be good to use normative languag=
e in the following sentences.<o:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l2 level1 lfo8">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"color:black">I don&#8217;t see the &#8220;=
error handling&#8221; behavior associated with this attribute (discard, tre=
at-as-withdraw&#8230;)<o:p></o:p></span></pre>
<pre><span style=3D"color:black"><o:p>&nbsp;</o:p></span></pre>
<pre><span style=3D"color:black">Section 3.2.1.1<o:p></o:p></span></pre>
<pre style=3D"margin-left:.5in;text-indent:-.25in;mso-list:l6 level1 lfo9">=
<![if !supportLists]><span style=3D"font-size:12.0pt;font-family:Symbol"><s=
pan style=3D"mso-list:Ignore">=B7<span style=3D"font:7.0pt &quot;Times New =
Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; </span></span=
></span><![endif]><span style=3D"color:black">&#8220;It may be present&#822=
1; or &#8220;It MAY be present&#8221; ?<o:p></o:p></span></pre>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">Section 4.1<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">Having an SFF being part of a single=
 overlay network is IMO a very limited use case, in such a case, I don&#821=
7;t think that you need an RT at all. The SFF could be part of
 a L3VPN which will be used as an underlay.<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">An SFF being &#8220;multitenant&#822=
1; is more valid and defacto requires to maintain separate forwarding state=
s (VRF like&#8230;). This is mandatory to maintain the tenant isolation and
 RTs are very useful here to know the appropriate context to put the routes=
 in.<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">Section 7.1<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">While I understand that the node doi=
ng the classification can perform a deep packet inspection to get an entrop=
y indicator, any intermediate node cannot set it again as
 the NSH header will be there.<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">Here is an example:<o:p></o:p></span=
></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;=
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">PacketsIn -</span><span style=3D"fon=
t-size:12.0pt;font-family:Wingdings">=E0</span><span style=3D"font-size:12.=
0pt;font-family:&quot;Times New Roman&quot;,&quot;serif&quot;"> Classifier =
-&gt;SFF1
 (SFI1) -&gt; SFF2(SFI2) -&gt;SFF3(SFI3)-&gt;PacketsOut to dest<o:p></o:p><=
/span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">Classifier pushes NSH header onto th=
e packets as well as the underlay tunnel (using an entropy indicator in UDP=
 or ELI/EL). Where the packets comes in at SFF1, IMO, it
 strips the underlay tunnels including the entropy indicator. Which means t=
hat when the packet with NSH header will go to SFF2, it will not have anymo=
re entropy indicator and there is no way to build it again as the NSH heade=
r prevents a new deep packet inspection.
 Am I missing something ? Maintaing a per packet state is IMO not doable , =
especially if the SFI modifies the packet content.<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">Section 8<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">I would have been good to have two S=
FIs with the same SFT on the same SFF in the example.<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">I don&#8217;t like the representatio=
n of RD using &#8220;192.0.2.1,1&#8221; as the &#8220;,&#8221; can be confu=
sing with a regular separator. Why not using :&#8221;192.0.2.1:1&#8221; not=
ation which is well known
 ?<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">Section 8.9.1:<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">How does an SFF know that an attache=
d SFI is stateful ? I don&#8217;t think it can know that.<o:p></o:p></span>=
</p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">I don&#8217;t think that the fact th=
at SFF2 is used in both direction is safe from a loadbalancing perspective.=
<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">If the hashing algorithm used by SFF=
2 is sensible to the order of the keys (like source vs dest address, or sou=
rce vs dest port), it may provide a different SFI as a result
 of the hashing between the forward and the reverse flow.<o:p></o:p></span>=
</p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">Section 8.9.2:<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">How can the controller instruct the =
classifier how to place traffic ?<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">In addition, this could involve mult=
iple classifiers that need to be coordinated.<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">The ingress forward classifier and t=
he ingress reverse classifier should use SFPs within the same assoc.<o:p></=
o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">Section 9:<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">Do we have to set limits on receivin=
g nodes in term of number of states received from the controller to mitigat=
e some attack ?<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">The text talks about security of BGP=
, what kind of mechanism should be put in place ?<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">Do we have any interdomain considera=
tions ?<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">This is a controller based approach,=
 so the security of the controller itself is also important.<o:p></o:p></sp=
an></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">References:<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">Shouldn&#8217;t RFC7665 be normative=
 ?<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">I think that the mpls-sfc and mpls-s=
fc-encaps should also be normative as you are defining a controlplane to us=
e them.<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;">Brgds,<o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p>&nbsp;</o:p></span></p>
<p class=3D"MsoNormal"><a href=3D"http://www.orange.com/"><span style=3D"fo=
nt-size:12.0pt;font-family:&quot;Times New Roman&quot;,&quot;serif&quot;;co=
lor:blue;text-decoration:none"><img border=3D"0" width=3D"40" height=3D"40"=
 id=3D"Picture_x0020_1" src=3D"cid:image001.jpg@01D4CDE0.58C5BF70" alt=3D"O=
range logo"></span></a><span style=3D"font-size:12.0pt;font-family:&quot;Ti=
mes New Roman&quot;,&quot;serif&quot;"><o:p></o:p></span></p>
<p class=3D"MsoNormal" style=3D"line-height:15.0pt"><span style=3D"font-siz=
e:12.0pt;font-family:&quot;Times New Roman&quot;,&quot;serif&quot;">&nbsp;<=
o:p></o:p></span></p>
<p class=3D"MsoNormal"><b><span style=3D"font-size:10.0pt;font-family:&quot=
;Arial&quot;,&quot;sans-serif&quot;;color:black">Stephane Litkowski
</span></b><span style=3D"font-size:12.0pt;font-family:&quot;Times New Roma=
n&quot;,&quot;serif&quot;"><br>
</span><span style=3D"font-size:10.0pt;font-family:&quot;Arial&quot;,&quot;=
sans-serif&quot;;color:black">Network Architect
</span><span style=3D"font-size:12.0pt;font-family:&quot;Times New Roman&qu=
ot;,&quot;serif&quot;"><br>
</span><span style=3D"font-size:10.0pt;font-family:&quot;Arial&quot;,&quot;=
sans-serif&quot;;color:black">Orange/SCE/EQUANT/OINIS/NET</span><span style=
=3D"font-size:12.0pt;font-family:&quot;Times New Roman&quot;,&quot;serif&qu=
ot;"><o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:10.0pt;font-family:&quot;Ar=
ial&quot;,&quot;sans-serif&quot;;color:black">Orange Expert Future Networks=
</span><span style=3D"font-size:12.0pt;font-family:&quot;Times New Roman&qu=
ot;,&quot;serif&quot;"><o:p></o:p></span></p>
<p class=3D"MsoNormal"><span style=3D"font-size:10.0pt;font-family:&quot;Ar=
ial&quot;,&quot;sans-serif&quot;;color:black">phone:
</span><span lang=3D"FR" style=3D"font-size:10.0pt;font-family:&quot;Arial&=
quot;,&quot;sans-serif&quot;;color:black"><a href=3D"https://monsi.sso.fran=
cetelecom.fr/index.asp?target=3Dhttp%3A%2F%2Fclicvoice.sso.francetelecom.fr=
%2FClicvoiceV2%2FToolBar.do%3Faction%3Ddefault%26rootservice%3DSIGNATURE%26=
to%3D&#43;33%202%2023%2028%2049%2083%20"><span style=3D"color:black">&#43;33
 2 23 <b>06</b> 49 83 </span></a>&nbsp;NEW&nbsp;!</span><span lang=3D"FR" s=
tyle=3D"font-size:12.0pt;font-family:&quot;Times New Roman&quot;,&quot;seri=
f&quot;"><br>
</span><span lang=3D"FR" style=3D"font-size:10.0pt;font-family:&quot;Arial&=
quot;,&quot;sans-serif&quot;;color:black">mobile:
<a href=3D"https://monsi.sso.francetelecom.fr/index.asp?target=3Dhttp%3A%2F=
%2Fclicvoice.sso.francetelecom.fr%2FClicvoiceV2%2FToolBar.do%3Faction%3Ddef=
ault%26rootservice%3DSIGNATURE%26to%3D&#43;33%206%2037%2086%2097%2052%20">
<span style=3D"color:black">&#43;33 6 71 63 27 50 </span></a>&nbsp;NEW&nbsp=
;!</span><span lang=3D"FR" style=3D"font-size:12.0pt;font-family:&quot;Time=
s New Roman&quot;,&quot;serif&quot;"><br>
<a href=3D"mailto:stephane.litkowski@orange.com"><span style=3D"font-size:1=
0.0pt;font-family:&quot;Arial&quot;,&quot;sans-serif&quot;;color:#FF6600">s=
tephane.litkowski@orange.com</span></a>
<o:p></o:p></span></p>
<p class=3D"MsoNormal"><o:p>&nbsp;</o:p></p>
</div>
<PRE>______________________________________________________________________=
___________________________________________________

Ce message et ses pieces jointes peuvent contenir des informations confiden=
tielles ou privilegiees et ne doivent donc
pas etre diffuses, exploites ou copies sans autorisation. Si vous avez recu=
 ce message par erreur, veuillez le signaler
a l'expediteur et le detruire ainsi que les pieces jointes. Les messages el=
ectroniques etant susceptibles d'alteration,
Orange decline toute responsabilite si ce message a ete altere, deforme ou =
falsifie. Merci.

This message and its attachments may contain confidential or privileged inf=
ormation that may be protected by law;
they should not be distributed, used or copied without authorisation.
If you have received this email in error, please notify the sender and dele=
te this message and its attachments.
As emails may be altered, Orange is not liable for messages that have been =
modified, changed or falsified.
Thank you.
</PRE></body>
</html>

--_000_9E32478DFA9976438E7A22F69B08FF924C199D40OPEXCAUBMA3corp_--

--_004_9E32478DFA9976438E7A22F69B08FF924C199D40OPEXCAUBMA3corp_
Content-Type: image/jpeg; name="image001.jpg"
Content-Description: image001.jpg
Content-Disposition: inline; filename="image001.jpg"; size=1093;
 creation-date="Wed, 27 Feb 2019 10:21:51 GMT";
 modification-date="Wed, 27 Feb 2019 10:21:51 GMT"
Content-ID: <image001.jpg@01D4CDE0.58C5BF70>
Content-Transfer-Encoding: base64

/9j/4AAQSkZJRgABAQEAYABgAAD/2wBDAAgGBgcGBQgHBwcJCQgKDBQNDAsLDBkSEw8UHRofHh0a
HBwgJC4nICIsIxwcKDcpLDAxNDQ0Hyc5PTgyPC4zNDL/2wBDAQkJCQwLDBgNDRgyIRwhMjIyMjIy
MjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjL/wAARCAAoACgDASIA
AhEBAxEB/8QAHwAAAQUBAQEBAQEAAAAAAAAAAAECAwQFBgcICQoL/8QAtRAAAgEDAwIEAwUFBAQA
AAF9AQIDAAQRBRIhMUEGE1FhByJxFDKBkaEII0KxwRVS0fAkM2JyggkKFhcYGRolJicoKSo0NTY3
ODk6Q0RFRkdISUpTVFVWV1hZWmNkZWZnaGlqc3R1dnd4eXqDhIWGh4iJipKTlJWWl5iZmqKjpKWm
p6ipqrKztLW2t7i5usLDxMXGx8jJytLT1NXW19jZ2uHi4+Tl5ufo6erx8vP09fb3+Pn6/8QAHwEA
AwEBAQEBAQEBAQAAAAAAAAECAwQFBgcICQoL/8QAtREAAgECBAQDBAcFBAQAAQJ3AAECAxEEBSEx
BhJBUQdhcRMiMoEIFEKRobHBCSMzUvAVYnLRChYkNOEl8RcYGRomJygpKjU2Nzg5OkNERUZHSElK
U1RVVldYWVpjZGVmZ2hpanN0dXZ3eHl6goOEhYaHiImKkpOUlZaXmJmaoqOkpaanqKmqsrO0tba3
uLm6wsPExcbHyMnK0tPU1dbX2Nna4uPk5ebn6Onq8vP09fb3+Pn6/9oADAMBAAIRAxEAPwD1O48f
afbXMsDWl0WjcoSAuDg49aj/AOFiad/z53f5L/jXB6r/AMhe9/67v/M1Ur5Geb4pSaTX3H2dPJcJ
KCbT+89H/wCFiad/z53f5L/jR/wsTTv+fO7/ACX/ABrziip/tnFd19xX9iYPs/vPR/8AhYmnf8+d
3+S/40V5xRR/bOK7r7g/sTB9n95b1X/kL3v/AF3f+ZqpW3Po17qGo3s0Cx7DdPGpeQLvbOdq56mo
R4d1EmEFYVaZdyI0oDY9cda450KspNqLO2niKUYJOSvbv5GVRWwPC+qmR4zFErI/l4aUDc2A2B68
GmDw7qTQrIIkJZVYR+YN+GOAdvuan6tW/lf3FfWqH86+8yqK2bjQG0+EvqU/2dmyIgieYGI6gkHg
0VM6M4O0tGVCtCavF3R1o8Oa7BcTtbXdh5TztPGJYyxjY9xxwcUxfDniNZ1m+32DOIPs+WjJymc8
8dc96KK+y/s+l3f3s+J/tKt2X3Ie+geJZJlla/0/eswmH7s/eChfT0FXDo2rDTRCktoLoIqfaCxy
NrZGPlz+GaKKpYGmr6vXzJlj6sraLTyMrUPCOuant+03lh8pJ+RGXJPUniiiisZ5Thpvmldv1N4Z
xioLljZL0R//2Q==

--_004_9E32478DFA9976438E7A22F69B08FF924C199D40OPEXCAUBMA3corp_--

