Skip to main content

Early Review of draft-ietf-spring-srv6-srh-compression-16
review-ietf-spring-srv6-srh-compression-16-opsdir-early-mishra-2024-05-14-00

Request Review of draft-ietf-spring-srv6-srh-compression
Requested revision No specific revision (document currently at 23)
Type Early Review
Team Ops Directorate (opsdir)
Deadline 2024-04-19
Requested 2024-03-21
Requested by Joel M. Halpern
Authors Weiqiang Cheng , Clarence Filsfils , Zhenbin Li , Bruno Decraene , Francois Clad
I-D last updated 2024-05-14
Completed reviews Intdir Early review of -17 by Benson Muite (diff)
Rtgdir Early review of -16 by Nicolai Leymann (diff)
Opsdir Early review of -16 by Gyan Mishra (diff)
Secdir Early review of -17 by Ned Smith (diff)
Genart Last Call review of -19 by Stewart Bryant (diff)
Secdir Last Call review of -19 by Ned Smith (diff)
Comments
This draft is in working group last call, with a small number of issues still being discussed.  The content is alrgely stable, so review at this stage would be helpful.
Assignment Reviewer Gyan Mishra
State Completed
Request Early review on draft-ietf-spring-srv6-srh-compression by Ops Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/ops-dir/tNvaWMYY5Zx5Fsht28bm9TBJ1Tg
Reviewed revision 16 (document currently at 23)
Result Ready
Completed 2024-05-14
review-ietf-spring-srv6-srh-compression-16-opsdir-early-mishra-2024-05-14-00
I have reviewed the latest version 16 of the draft with Operational
Considerations section 9.3 added to address L4 checksum discussion on ML. The
draft is well written and is ready for publication.

I have been following the ML thread since it began related to L4 checksum.  The
key here for operators deploying c-SID SRv6 Compression Next SID is that
endpoints performing L4 checksum would not exist on the SRv6 network closed
domain and would exist within the customer payload which remains unaltered and
will always pass the L4 checksum.  Section 9.3 addresses the operational
consideration where the DA is not the same as the final destination and may
require enhancements to convey the final destination address.

Section 9.3
Upper layer checksums are computed by the originator of an IPv6 packet and
verified by the ultimate destination(s) as it processes the upper layer
protocol.

As specified in Section 6.5, SR source nodes originating TCP/UDP packets ensure
that the upper layer checksum is correctly calculated based on the ultimate
destination of the session, which may be different from the address placed in
the IPv6 destination address. Such SR source nodes leveraging TCP/UDP offload
engines may require enhancements to convey the ultimate destination address.
These implementation enhancements are outside the scope of this document.

Middleboxes such as packet sniffers, if deployed inside the SR domain, may fail
to verify the upper layer checksum of transit SRv6 traffic. Making these
middleboxes SRv6 aware in general or C-SID aware in particular is out of the
scope of this document.

Major issues:
None

Minor issues:
None

Nits:
None