Document: draft-ietf-6man-sidlist-clarification
Title: Clarifying SRv6 SID List Processing
Reviewer: Peter Yee
Review result: Has Nits

This draft clarifies that Segment Identifier (SID) entries in the Segment Routing Header are not necessarily routable IPv6 addresses but may require mapping to routable and usable IPv6 addresses. From a security perspective, I don’t have great concerns, although the mapping process (not specified in this draft) could potentially cause packets to be misrouted if it makes an incorrect mapping. The document has a few piddling nits that could be fixed.

Major concerns: None

Minor concerns: None

Nits:

Page 2, section 1, 2nd paragraph: append a comma after “[RFC8754]”.

Page 3, 2nd paragraph, 2nd sentence: delete the comma after “addresses)”.

Page 4, paragraph after figure: change “128 bit” to “128-bit”.

Page 5, figure: in both cases, change “addresses to address”. Each entry is presumably mapped to a single address.

Page 5, 2nd paragraph: change “128 bit” to “128-bit”.