Early Review of draft-ietf-ippm-asymmetrical-pkts-07
review-ietf-ippm-asymmetrical-pkts-07-tsvart-early-eggert-2025-06-03-00

Request Review of draft-ietf-ippm-asymmetrical-pkts
Requested revision No specific revision (document currently at 08)
Type Early Review
Team Transport Area Review Team (tsvart)
Deadline 2025-06-13
Requested 2025-05-16
Requested by Marcus Ihlar
Authors Greg Mirsky , Ernesto Ruffini , Henrik Nydell , Richard "Footer" Foote , Will Hawkins
I-D last updated 2025-09-02 (Latest revision 2025-06-28)
Completed reviews Tsvart Early review of -07 by Lars Eggert (diff)
Comments 
The document defines a new optional TLV for STAMP test packets that lets a Session‑Sender ask the Session‑Reflector to return “asymmetrical” traffic: a controllable number of reflected packets, of a specified length, spaced at a specified nanosecond interval. This allows for use cases such as access rate measurements. It also provides guidelines for measurements in multicast topologies.

The document does acknowledge UDP considerations by explicitly citing RFC 8085 and RFC 9097 and by mandating rate/volume limits and congestion‑aware behaviour.

We would like to request a Transport Area review to understand whether the provided guidance around congestion control and the associated references are sufficient.
Assignment Reviewer Lars Eggert
State Completed
Request Early review on draft-ietf-ippm-asymmetrical-pkts by Transport Area Review Team Assigned
Posted at https://mailarchive.ietf.org/arch/msg/tsv-art/0HE3aXmSMk598YCbTUMvCyh63mA
Reviewed revision 07 (document currently at 08)
Result On the right track
Completed 2025-06-03
review-ietf-ippm-asymmetrical-pkts-07-tsvart-early-eggert-2025-06-03-00
# 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