[pim] draft-ietf-pim-sr-p2mp-policy-14 ietf last call Rtgdir review

[Linda Dunbar via Datatracker <[e-mail]>]

Document: draft-ietf-pim-sr-p2mp-policy
Title: Segment Routing Point-to-Multipoint Policy
Reviewer: Linda Dunbar
Review result: Has Nits

Summary: This document is technically solid and aligns well with existing MPLS
and SR OAM mechanisms.

Some Nits though:

Abstract: “via a controller-based mechanisms” → remove plural: “via a
controller-based mechanism”

Section 3.1: “The appropriate respond is sent back…” → should be: “The
appropriate response is sent back…”

Section 6 (Security): “Overall, the security needs for P2MP policy ping is same
as [RFC8029].”  — grammatical fix: “...are the same as...”

Section 5 (IANA): “This sub-type value is assigned from the standards Action of
range...”   — is it better with : “...from the Standards Action range of ...”

The following sentence is difficult to follow (Section 3.1.3):
"For example, when a P2MP Policy Ping or Traceroute packet enters an Unicast SR
domain, it MUST be processed on the two interconnecting Replication Segments,
based on the Replication SID and its TTL value.".

Best Regards, Linda Dunbar