[pim] draft-ietf-pim-pfm-forwarding-enhancements-05 telechat Rtgdir review

[Carlos Pignataro via Datatracker <[e-mail]>]

Document: draft-ietf-pim-pfm-forwarding-enhancements
Title: PIM Flooding Mechanism and Source Discovery Enhancements
Reviewer: Carlos Pignataro
Review result: Has Issues

Hello,
I have been selected as the Routing Directorate reviewer for this draft. The
Routing Directorate seeks to review all routing or routing-related drafts as
they pass through IETF last call and IESG review, and sometimes on special
request. The purpose of the review is to provide assistance to the Routing ADs.
For more information about the Routing Directorate, please see
https://wiki.ietf.org/en/group/rtg/RtgDir Although these comments are primarily
for the use of the Routing ADs, it would be helpful if you could consider them
along with any other IETF Last Call comments that you receive, and strive to
resolve them through discussion or by updating the draft.

Document:        draft-ietf-pim-pfm-forwarding-enhancements-05
Reviewer:        Carlos Pignataro
Intended Status: Experimental

Summary: I have some minor concerns about this document that I think should be
resolved before publication.

1. T-bit semantics potential conflict with RFC 8364 — message-drop vs. TLV-drop
(Section 2.1)

The draft states: "If set to 0, a router that does not support the TLV or any
contained Sub-TLV MUST NOT forward the message." This seems to conflict with
RFC 8364 Section 3.4.2, which defines T-bit behavior at the TLV level, not the
message level. In RFC 8364, a non-transitive unsupported TLV is dropped from
the forwarded message — the message itself continues forwarding. Which one is
it?

2. Group Source Info TLV Figure confusing

The diagram shows Group Address and Source Address as single 32-bit rows, but
the text correctly notes these are variable-length (64 or 160 bits for
IPv4/IPv6 respectively). Could the Figure be updated to show variable length?

3. Could 'Relaxed-RPF' be added to the Terminology section?

4. On the IANA Section, for "PIM Flooding Mechanism Group Source Info Message
Types", could an experimental range and the value of 0 be defined?

I hope these are clear and useful.

Best,

Carlos Pignataro