[tsvwg] Opsdir last call review of draft-ietf-tsvwg-sctp-zero-checksum-11

Victor Kuarsingh via Datatracker <noreply@ietf.org> Sat, 22 June 2024 18:24 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: tsvwg@ietf.org
Delivered-To: tsvwg@ietfa.amsl.com
Received: from [10.244.2.12] (unknown [104.131.183.230]) by ietfa.amsl.com (Postfix) with ESMTP id DC1E6C14F6A7; Sat, 22 Jun 2024 11:24:15 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Victor Kuarsingh via Datatracker <noreply@ietf.org>
To: ops-dir@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 12.16.1
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <171908065555.146684.4738949602355307984@dt-datatracker-b7bc58d5f-jsj9g>
Date: Sat, 22 Jun 2024 11:24:15 -0700
Message-ID-Hash: 4FYCVS4IZIDO5NZ5XJGGOQLFRYPXFGQ5
X-Message-ID-Hash: 4FYCVS4IZIDO5NZ5XJGGOQLFRYPXFGQ5
X-MailFrom: noreply@ietf.org
X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; header-match-tsvwg.ietf.org-0; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header
CC: draft-ietf-tsvwg-sctp-zero-checksum.all@ietf.org, last-call@ietf.org, tsvwg@ietf.org
X-Mailman-Version: 3.3.9rc4
Reply-To: Victor Kuarsingh <victor@jvknet.com>
Subject: [tsvwg] Opsdir last call review of draft-ietf-tsvwg-sctp-zero-checksum-11
List-Id: Transport Area Working Group <tsvwg.ietf.org>
Archived-At: <https://mailarchive.ietf.org/arch/msg/tsvwg/jDDttbEGPbfJadveU156JaoGTG4>
List-Archive: <https://mailarchive.ietf.org/arch/browse/tsvwg>
List-Help: <mailto:tsvwg-request@ietf.org?subject=help>
List-Owner: <mailto:tsvwg-owner@ietf.org>
List-Post: <mailto:tsvwg@ietf.org>
List-Subscribe: <mailto:tsvwg-join@ietf.org>
List-Unsubscribe: <mailto:tsvwg-leave@ietf.org>

Reviewer: Victor Kuarsingh
Review result: Ready

Reviewer: Victor Kuarsingh
Review Result: Ready

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 with the intent of improving the operational aspects of
the IETF drafts. Comments that are not addressed in last call may be included
in AD reviews during the IESG review.  Document editors and WG chairs should
treat these comments just like any other last call comments.

As part of this ops review, I did not find anything that would restrict this
document's readiness, but have operationally oriented questions/comments for
consideration.

(1) The document did not share how much (% of overhead, cycle time, etc) impact
calculating the checksum has which means that, in production networks, the true
impact is unknown.  The overhead is likely to change, so how do participating
nodes know if they should even care about the overhead?  Changing code
implementation that affects functionality should be considered in relation to
such benefit.  In short, exposing stable implementations to changes to save
what could be a trivial amount of overhead may or may not be worth it.  Again,
this is a general question and does not impact publishing IMO.

(2) What is the assumption about how the higher layer integrity checks occur? 
What about a use case where the upper layer checks can be turned off
mid-session?  would that consideration need to be considered?  I don't know if
such implementations actually occur in practical terms, but it's a potential
failure case.  this is theoretical in nature.

outside of those question, no specific changes are noted to be made as part of
this review.