Hi!

I have reviewed this document as part of the Operational directorate's ongoing effort to review all IETF documents being processed by the IESG. These comments were written primarily for the benefit of the operational area directors. Document editors and WG chairs should treat these comments just like any other last call comments.

This document is on the Standards Track, and specifies two mechanisms for Carrying PIM-SM in ASM mode Trees over P2MP mLDP LSPs.

Summary: Almost ready

Major:

None.

Minor:

Introduction:

CMP: Overall, the introduction is a bit hard to parse. It contains a number of assumptions and expectations of the reader and notes. I suggest these paragraphs be taken out into a separate subsection of the Intro.

   The first mechanism, described in Section 3, is optional for
   implementations, but the second mechanism, described in Section 4, is
   mandatory for all implementations claiming conformance to this
   specification.

CMP: This being a STD Track document, is “mandatory” the same as “REQUIRED” (see Section 1.1)? Is “optional” the same as “OPTIONAL”?

CMP: A nit here as well, the pointers to the Sections have a wrong offset (should be S2 and S3).


Nits:

CMP: Sometimes the text talks about “Source Active Auto-Discovery” and others about “BGP Source Active Auto-Discovery”. See example below, I think it should normalize on one of them:
 2.1 Originating Source Active Auto-Discovery Routes (Mechanism 1) ..  5
 2.2 Receiving BGP Source Active Auto-Discovery Route by LSR  .......  6


Hope this helps.

Thanks,

-- Carlos.
_______________________________________________
OPS-DIR mailing list
OPS-DIR at ietf.org


https://www.ietf.org/mailman/listinfo/ops-dir