[Tsv-art] draft-ietf-ippm-asymmetrical-pkts-07 early Tsvart review

Lars Eggert via Datatracker <noreply@ietf.org> Tue, 03 June 2025 11:30 UTC

MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Lars Eggert via Datatracker <noreply@ietf.org>
To: tsv-art@ietf.org
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <174895024824.2740627.17855291019432436704@dt-datatracker-59b84fc74f-84jsl>
Date: Tue, 03 Jun 2025 04:30:48 -0700
Message-ID-Hash: FSN74ZMQANAS6HSG5FNXRVLJXGSNG4CH
CC: draft-ietf-ippm-asymmetrical-pkts.all@ietf.org, ippm@ietf.org
Reply-To: Lars Eggert <lars@eggert.org>
Subject: [Tsv-art] draft-ietf-ippm-asymmetrical-pkts-07 early Tsvart review
Archived-At: <https://mailarchive.ietf.org/arch/msg/tsv-art/0HE3aXmSMk598YCbTUMvCyh63mA>

Document: draft-ietf-ippm-asymmetrical-pkts
Title: Performance Measurement with Asymmetrical Traffic Using STAMP
Reviewer: Lars Eggert
Review result: On the Right Track

# tsvart-early review of draft-ietf-ippm-asymmetrical-pkts-07

CC @larseggert

This document has been reviewed as part of the transport
area review team's ongoing effort to review key IETF
documents. These comments were written primarily for the
transport area directors, but are copied to the
document's authors and WG to allow them to address any
issues raised and also to the IETF discussion list for
information.

When done at the time of IETF Last Call, the authors
should consider this review as part of the last-call
comments they receive. Please always CC tsv-art@ietf.org
if you reply to or forward this review.

## Comments

This is a readable document in decent shape. I have no particular concerns.

### DOWNREFs

DOWNREF `[RFC7497]` from this Proposed Standard to Informational `RFC7497`.

## Nits

All comments below are about very minor potential issues that you may choose to
address in some way - or ignore - as you see fit. Some were flagged by
automated tools (via https://github.com/larseggert/ietf-reviewtool) so there
will likely be some false positives. There is no need to let me know what you
did with these suggestions.

### Typos

#### Section 3.3, paragraph 3
```
-    multicast network with monitoring the onsistency of Differentiated
+    multicast network with monitoring the consistency of Differentiated
+                                          +
```

### Outdated references

Document references `draft-ietf-ippm-capacity-protocol-15`, but `-18` is the
latest available revision.

### Grammar/style

#### Section 3.3, paragraph 2
```
 test packet reflection, e.g., extremely small intervals and/or too many conc
                               ^^^^^^^^^^^^^^^
```
Consider using an extreme adjective for "small".

#### Section 4, paragraph 3
```
A of [RFC9097] offers an example pseudo-code for a UDP load rate adjustment a
                                 ^^^^^^^^^^^
```
This word is normally spelled as one.

## Notes

This review is in the ["IETF Comments" Markdown format][ICMF]. You can use the
[`ietf-comments` tool][ICT] to automatically convert this review into
individual GitHub issues. Review generated by the [`ietf-reviewtool`][IRT].

[ICMF]: https://github.com/mnot/ietf-comments/blob/main/format.md
[ICT]: https://github.com/mnot/ietf-comments
[IRT]: https://github.com/larseggert/ietf-reviewtool

[Tsv-art] draft-ietf-ippm-asymmetrical-pkts-07 ea… Lars Eggert via Datatracker
[Tsv-art] Re: draft-ietf-ippm-asymmetrical-pkts-0… Greg Mirsky