[Gen-art] Genart last call review of draft-ietf-intarea-frag-fragile-13

Pete Resnick via Datatracker <noreply@ietf.org> Thu, 04 July 2019 20:47 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: gen-art@ietf.org
Delivered-To: gen-art@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 0B890120247; Thu, 4 Jul 2019 13:47:52 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Pete Resnick via Datatracker <noreply@ietf.org>
To: <gen-art@ietf.org>
Cc: draft-ietf-intarea-frag-fragile.all@ietf.org, int-area@ietf.org, ietf@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.98.2
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Pete Resnick <resnick@episteme.net>
Message-ID: <156227327196.12217.1279652184062580039@ietfa.amsl.com>
Date: Thu, 04 Jul 2019 13:47:52 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/4_SyBcEA3uWxCfVpkv8UecOV7YI>
Subject: [Gen-art] Genart last call review of draft-ietf-intarea-frag-fragile-13
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 04 Jul 2019 20:47:52 -0000

Reviewer: Pete Resnick
Review result: Ready with Nits

I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at

<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>;.

Document: draft-ietf-intarea-frag-fragile-13
Reviewer: Pete Resnick
Review Date: 2019-07-04
IETF LC End Date: 2019-07-04
IESG Telechat date: Not scheduled for a telechat

Summary: A document so well-written that even this application-layer idiot
could follow along easily. Thanks for that. I see no significant issues with
this document going forward as a BCP; just a few editorial bits.

Major issues: None

Minor issues: None

Nits/editorial comments:

In Section 2.1 and 2.2, Instead of "set to one" and "set to zero", it would
read easier with "set to (1)" and "set to (0)", or some similar construction.

Section 3 is in an odd place. I'd say either move it up to the top, or put it
down in section 7.

4.2 mentions virtual reassembly. Virtual reassembly applies to 4.1, 4.3, and
4.4 as well. Perhaps moving the discussion of virtual reassembly up to the top
of 4 would make more sense.

In 4.5, insert "duplicate IDs resulting in" after prevent. It took me a bit to
figure out what this was referring to. Also, change "are not easily
reproducible" to "do not occur as frequently"; I think it reads better.

And just to yell into the wind: Section 7.3 seemed a little wimpy to me, but I
can't for the life of me figure out how to make it any stronger or more likely
to be listened to. End of pointless yelling.