The MPLS WG Archive[Date Prev][Date Next][Thread Prev][Thread Next] [Date Index][Thread Index][Author Index][Subject Index] [mpls] Last Call on "LSP Ping"
> This is one of the best written I-Ds for a long time!
>
> A couple of nits.
>
> Section 4.4
> Note: in all the error codes listed in this draft a stack-depth of 0
> means "no value specified". This allows compatibility with existing
> implementations which do not use the Return Subcode field.
> s/draft/document/
Thanks.
> Section 7
> There are several specific codepoints defined in the draft and marked as
> "to be defined by IANA". These are specific values within the new name
> spaces defined in section 7. I think it would help IANA if you listed all
> of the defined values in a central place (perhaps with references to the
> defining sections).
Good idea. Will include.
> Appendix A.
> I'm sure this one has been out and round the block several times.
> Notwithstanding that the appendix is non-normative, why is it included? I
> thought we were doing no new work (apart from drafts listed in RFC3468)
> related to CR-LDP.
Is there anyone who objects to removing Appendix A?
> Section 3.5
> The Error Code TLV
> I think no-one came up with a use for this. Since we try not to define
> things unless we have an actual use, perhaps this TLV should be retired.
I'll leave it to Kireeti to defend as I also suggested taking it out.
> And a silly question.
>
> Suppose I have an RSVP-TE LSP that starts on an IPv6 LSR and ends on an
> IPv4 LSR (or vice versa).
> So I have an IPv6 Sender Template and IPv4 Session (or vice versa).
> How do I encode the Target FEC Stack?
> Or perhaps this is just an unrealistic deployment?
Hmmm. Good luck getting a notify message back! Not something I was
planning to support.
========================================================================
George Swallow Cisco Systems (978) 936-1398
1414 Massachusetts Avenue
Boxborough, MA 01719
_______________________________________________
mpls mailing list
mpls@lists.ietf.org
https://www1.ietf.org/mailman/listinfo/mpls
|
|