The MPLS WG Archive[Date Prev][Date Next][Thread Prev][Thread Next] [Date Index][Thread Index][Author Index][Subject Index] [mpls] Summary: mpls wg review of thedraft-ietf-l2vpn-vpls-ldp-06.txt
All,
the mpls wg review of the draft-ietf-l2vpn-vpls-ldp-06.txt
has ended.
The comments sent to the list has been scarce.
We have received a few comments off-line, and also
comments from four reviewers that we asked to review
the documents. We want to thank Bob T, Ina, Eric G and
Dimitri for their reviews. Part of their comments are
very useful as "wg chair comments" and will be sent to
ADs and authors.
Below are the major points captured, based on the
reviewer input.
Targeted LDP sessions
----------------------
One recurring theme is that a choice to use LDP to signaling
vpls de-multiplexors will lead to a full mesh of targeted
ldp session between PEs. It has been appointed out that this
will potentially cause scaling problems.
WG-chair comment: LDP was never intended to support a full mesh of
targeted LDP sessions.
If LDP is used in this way it must clearly be stated that this is a
factor that will limit the scaling of the vpls application,
However this is nothing the changes or affects the workings of
other usages of LDP. This is local to this draft and should be
handled here.
MAC List TLV:
-------------
A second point is made that the added TLV "MAC List TLV"
is an example of using LDP for something it was not intended
for. One issue here is that the already, as compared to a routing
protocol, long LDP convergence time could be even longer. This
will have effects on both "traditional" LDP convergence and the
vpls application.
The encoding of this TLV is not clearly defined.
First the diagram of the TLV depicts MAC addresses that are
only 4 bytes long.
Second, given the historic confusion between IEEE and IETF
encodings some discussion of bit and byte order and some
pointers to documents such as RFC1042 would be in order.
WG-chair comment:
The MPLS working group is concerned about this issue. However
we need a better understanding of the dynamics involved before
a recommendation can be made.
The text on the usage of this TLV is very difficult to parse.
Some clean-up is needed. Further use of this TLV should be
incorporated into the examples.
Use of Generalized PWid FEC Element
-----------------------------------
Several fields are not clearly specified.
E.g. in section 4.1.1:
>> Control bit (C): Depending on whether, on that particular PW, the
>> control word is desired or not, the control bit may be specified.
simply say:
This bit is used to signal use of the control word as specified in
[PWE3-CTRL].
>> PW type: The allowed PW types in this version are Ethernet and
>> Ethernet VLAN.
The phrase "in this version" should be deleted.
>> AGI Type field
Either a value should be specified (along with a corresponding format)
or it should be stated how values are to be selected.
>> - Requested VLAN ID: If the PW type is Ethernet VLAN, this
>> parameter may be used to signal the insertion of the
>> appropriate VLAN ID.
This looks under specified.
>> Section 7. Operation of a VPLS
Since FEC 128 has been moved to an appendix, this section should be
recast in terms of FEC 129.
>> Section 8.2.2. Failure detection and recovery
The hello message only detect control plane failures and not at a
particularly fast rate. If fast detection is needed, then
VCCV or BFD on the PSN tunnel should be employed.
MD5 and security
----------------
A third point that has been made is that the security
considerations of the LDP specification were never designed
to meet the requirements of VPLS.
WG-chair comment: This would definitely be of great concern,
but again it does not effect the LDP as a protocol but is
limited to the usage of LDP in this application.
/Loa and George
--
Loa Andersson
Principal Networking Architect
Acreo AB phone: +46 8 632 77 14
Isafjordsgatan 22 mobile: +46 739 81 21 64
Kista, Sweden email: loa.andersson@acreo.se
loa@pi.se
_______________________________________________
mpls mailing list
mpls@lists.ietf.org
https://www1.ietf.org/mailman/listinfo/mpls
|
|