> Mirja Kühlewind has entered the following ballot position for
> draft-ietf-lisp-rfc6830bis-16: 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-rfc6830bis/
Thanks for your comments Mirja.
> ----------------------------------------------------------------------
> DISCUSS:
> ----------------------------------------------------------------------
>
> I have a couple of smaller discuss points with should be straight-forward to
> address and one more high-level discussion point that might not have a
> solution
> (depending on the deployment status of LISP I guess) but I would like to at
> least have a discussion. I start with the straight-forward onces:
>
> 1) Unfortunately ECN decapsulation is slightly more complicated than described
> in section 5.3. Please check section 3.2 in rfc6040 and revise accordingly
> (maybe also provide a pointer to rfc6040 instead or in addition to rfc3168)!
> (Also it seems like the text on ECN is simply just twice in sec 5.3; not sure
> that is helpful).
Well it doesn’t have to be. I am a bit hesitant to just point to another long
complicated RFC. This text has gone through review for quite a long time and
many ECN experts decided we should write it this way. And this text did go
through IESG review when RFC6830 was made experimental RFC.
> 2) Further, also in sec 5.3:
> "The inner-header 'Differentiated Services Code Point' (DSCP) field
> (or the 'Traffic Class' field, in the case of IPv6) SHOULD be
> copied from the outer-header DSCP field ('Traffic Class' field, in
> the case of IPv6) considering the exception listed below."
> However, I didn't find any exceptions listed later in the doc. However,
> setting
> the DSCP field might also be matter of local policy. E.g. if DSCP is not used
> for a different purpose in the receiver side LISP network, it could make sense
> to restore/keep the original value in the inner header.
This is a dangling reference where the “text below” was removed or reworded. So
I will remove this reference.
> 3) Sec 7.1. only takes about ICMPv6 "Packet Too Big" packets while
> "IPv4-encapsulated packet with the DF bit set to 1" should be addressed as
> well.
This is discussed in length. I don’t know how you could have missed this.
> 4) I would like to see another sentence in section 12 explicitly stating that
> the source port SHOULD be the same for all packet belong to the same flow.
Done.
> 5) Sec 5.3 says "Both N- and V-bits MUST NOT be set in the same packet."
> What happens if both bits are set? The 'Nonce/Map-Version' is just ignored, or
> maybe the packet should be dropped or something? Please clarify in the doc!
If they are both set, there is text in the docuemnt that already says:
<t hangText="N:">The N-bit is the nonce-present bit. When
this bit is set to 1, the low-order 24 bits of the first 32
bits of the LISP header contain a Nonce. See <xref
target="echo-nonce"/> for details. Both N- and V-bits MUST
NOT be set in the same packet. If they are, a decapsulating
ETR MUST treat the 'Nonce/Map-Version' field as having a
Nonce value present.</t>
> 6) And now the more-discussion-needed point:
> So my underlying concern is the same as brought up by the TSV-ART review that
> lisp information are not end-to-end integrity protected or authenticated.
I would like you to be more specific. Beacuse there is a lot of security in the
protocol and we believe the current drafts, in their entirety, inicdate so.
> However, while briefly thinking about how this could be eventually realized, I
> noticed that there is actually no mechanism to extend the LISP header in any
Right, by design so it is efficient for hardware AND software forwarding. But
we do have the LISP-GPE header that can be used for extensions. But that has
limited deployment.
> way. There is no version, no option and the LISP header seems to have a fixed,
We decdied as a working group that the UDP port number would indicate what
header follows and therefore what LISP version is used.
> implicitly specified length without an explicit length field. This seems too
That is right, the header is 8 bytes, fixed, by design.
> late to add any kind of extensibility mechanism at this stage of the protocols
> lifetime, however, I would still like to discuss if there was any discussion
> about extensibility, what was the reason to chose this approach, and potential
> if some background about the choice should be given in the doc.
Well the original authors didn’t believe in having a lot of options in
protocols, otherwise it will cause more un-interoperability. But has time has
gone on, I believe we made the right choices, and have discovered that LISP is
quite extensible, and even 10 years after the the original design, we can
continue to add more LISP use-cases with the same basic protocol.
> ----------------------------------------------------------------------
> COMMENT:
> ----------------------------------------------------------------------
>
> 1) In sec 7.1 I would recommend to provide a pointer to RFC4459 and align the
> language to more what RFC4459 recommends: OLD "This behavior is performed by
> the ITR when the source host originates
> a packet with the 'DF' field of the IP header set to 0."
> MAYBE:
> "This behavior MAY be performed by the ITR only when the source host
> originates
> a packet with the 'DF' field of the IP header set to 0.
Changed. Thanks for the suggestion.
> 2) Sec 4: "...this document recommends that a maximum of two
> LISP headers can be prepended to a packet."
> MAYBE:
> "it is RECOMMENDED that a maximum of two
> LISP headers can be prepended to a packet.”
Changed.
Thanks again,
Dino
_______________________________________________
lisp mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/lisp
