[Int-dir] Intdir telechat review of draft-ietf-detnet-pof-07

Pascal Thubert via Datatracker <noreply@ietf.org> Wed, 20 December 2023 09:39 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: int-dir@ietf.org
Delivered-To: int-dir@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id D757BC04B451; Wed, 20 Dec 2023 01:39:24 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Pascal Thubert via Datatracker <noreply@ietf.org>
To: int-dir@ietf.org
Cc: detnet@ietf.org, draft-ietf-detnet-pof.all@ietf.org, last-call@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 12.1.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <170306516486.55866.11706293245832555193@ietfa.amsl.com>
Reply-To: Pascal Thubert <pascal.thubert@gmail.com>
Date: Wed, 20 Dec 2023 01:39:24 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/int-dir/F1gvNenGrCgK4FIS8Hvi6ZLtIRU>
Subject: [Int-dir] Intdir telechat review of draft-ietf-detnet-pof-07
X-BeenThere: int-dir@ietf.org
X-Mailman-Version: 2.1.39
List-Id: "This list is for discussion between the members of the Internet Area directorate." <int-dir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/int-dir>, <mailto:int-dir-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/int-dir/>
List-Post: <mailto:int-dir@ietf.org>
List-Help: <mailto:int-dir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/int-dir>, <mailto:int-dir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 20 Dec 2023 09:39:24 -0000

Reviewer: Pascal Thubert
Review result: Ready

I am an assigned INT directorate reviewer for draft-ietf-detnet-pof-06. These
comments were written primarily for the benefit of the Internet Area Directors.
Document editors and shepherd(s) should treat these comments just like they
would treat comments from any other IETF contributors and resolve them along
with any other Last Call comments that have been received. For more details on
the INT Directorate, see https://datatracker.ietf.org/group/intdir/about/
<https://datatracker.ietf.org/group/intdir/about/>.

Based on my review, if I was on the IESG I would ballot this document as NO
OBJECTION.

General comments:
-----------------
I found the document to be very clear and readable. The goal appears to be
limited to an algorithm and if that's so, mission accomplished.

I regret, though, that the real implementation and deployment issues are only
lightly discussed, e.g., positioning of the function (only once on the path?
close to exit?), values of delays, or recommendations to operate the feature in
a silicon high speed hardware with many ports on multiple line cards.

Clearly, the POF function benefits from aggregation (less flows to look at) but
will reorder the aggregate even if the packets are from separate flows.

The function makes an assumption on a packet sequencing. A reference should be
provided on how this is done today. There's pending work on tagging IPv6
packets natively (as options) but that work is not progressed yet; worth
mentioning?