Re: [Isis-wg] New version of draft-wei-isis-tlv

[[e-mail]]

Hi, IS-IS experts,

02 version has been posted just now. Modified according to Tony Li's comments. 

Best Regards,
Zhenqiang Li
13911635816
Department of Network Technology
China Mobile Research Institute
2010-03-08

--------------------------------------------------------------------------------

发件人： Tony Li
发送时间： 2010-03-06 01:24:42
收件人： [e-mail]; 'isis-wg'
抄送： [e-mail]; Chris Hopps; [e-mail]; [e-mail]
主题： Re: [Isis-wg] New version of draft-wei-isis-tlv




On 3/5/10 4:23 AM, "[e-mail]"
<[e-mail] > wrote:

> Hello ISIS fans,
> 
> A new version of draft-wei-isis-tlv was submitted to IETF just now. Comments
> are welcome. Thank you all for your discussion here.


Hi,

I've reviewed the new version of the draft and have a few comments:

1) I believe that we have some amount of agreement that the TLV for
backtracking purges is of some value.  However, the situations where it does
add value are all of the legitimate cases to initiate a purge.  As has been
noted, the implementations that generate a purge on a checksum error not
operating within the spec and should be repaired.  It is wholly unreasonable
to expect that an implementation that has this defect is going to
subsequently implement the new TLV.  Thus, the discussion of the purge on
checksum case is not a motivator for the addition of this TLV.  As such, I
recommend that the text discussing this case be removed.

2) You state "ISIS protocol is vulnerable to purge packet propagation."  The
use of the word "vulnerability" implies that there is a defect, and further
implies that there is a security defect.  Neither of these is true.  I
recommend that you reword this sentence to something like: "The IS-IS
protocol floods purges throughout an area, regardless of which IS initiated
the purge.  If a network operator would like to investigate the cause of the
purge, it is difficult to determine the origin of the purge."

3) Your section on security considerations is insufficient.  We need to
clearly specify how this TLV is used when constructing a purge.  What is
needed here is not very difficult.  Something similar to: "If this TLV is
used in conjunction with IS-IS authentication mechanisms, then the purge is
constructed by removing the original contents of the LSP, leaving only the
LSP header, adding this TLV and then adding the IS-IS authentication TLV.
This document amends the behavior of [RFC5304] and [RFC5310]."

4) Editorial stuff: you appear to be having some formatting issues, whereby
you only get one section per page.  May I recommend xml2rfc as a fine tool?
Also, your draft needs some extensive proofreading.

Regards,
Tony