[Gen-art] Gen-ART Telechat Review of draft-ietf-pcn-encoding-comparison-08

Pete McCann <mccap@petoni.org> Wed, 29 February 2012 04:33 UTC

Return-Path: <mccap@petoni.org>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 1BB9221E8039 for <gen-art@ietfa.amsl.com>; Tue, 28 Feb 2012 20:33:16 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.977
X-Spam-Level:
X-Spam-Status: No, score=-2.977 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, FM_FORGED_GMAIL=0.622, RCVD_IN_DNSWL_LOW=-1]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id iFIdskGJ36R5 for <gen-art@ietfa.amsl.com>; Tue, 28 Feb 2012 20:33:15 -0800 (PST)
Received: from mail-tul01m020-f172.google.com (mail-tul01m020-f172.google.com [209.85.214.172]) by ietfa.amsl.com (Postfix) with ESMTP id 3C2C321E8010 for <gen-art@ietf.org>; Tue, 28 Feb 2012 20:33:14 -0800 (PST)
Received: by obbeh20 with SMTP id eh20so4221765obb.31 for <gen-art@ietf.org>; Tue, 28 Feb 2012 20:33:14 -0800 (PST)
Received-SPF: pass (google.com: domain of mccap@petoni.org designates 10.182.31.50 as permitted sender) client-ip=10.182.31.50;
Authentication-Results: mr.google.com; spf=pass (google.com: domain of mccap@petoni.org designates 10.182.31.50 as permitted sender) smtp.mail=mccap@petoni.org; dkim=pass header.i=mccap@petoni.org
Received: from mr.google.com ([10.182.31.50]) by 10.182.31.50 with SMTP id x18mr7982481obh.53.1330489994402 (num_hops = 1); Tue, 28 Feb 2012 20:33:14 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=petoni.org; s=google; h=mime-version:x-originating-ip:date:message-id:subject:from:to :content-type; bh=iXv2g7zkwozl+Z7n5nIvwjqV/l38YZ+YufaFw7szdjs=; b=G6GYUZFKPZybQbL2tloXlEa0veIzZNadPqWu+aPSkDSaJniMcBX/xHRlC5v4QJph3I pC9VEivwniXiWU7W292Nrr6YqvyozMpy2hnco43OMWYXAjGr2RVSF7UmAO6CAgZMPmTo ZX2hevE3e9/5m+6B/pUUtIDIEGDZUU2eUzEUE=
MIME-Version: 1.0
Received: by 10.182.31.50 with SMTP id x18mr6964649obh.53.1330489994323; Tue, 28 Feb 2012 20:33:14 -0800 (PST)
Received: by 10.60.13.102 with HTTP; Tue, 28 Feb 2012 20:33:14 -0800 (PST)
X-Originating-IP: [68.45.157.93]
Date: Tue, 28 Feb 2012 23:33:14 -0500
Message-ID: <CACvMsLF5XNGWM16=vsnto4oFmNwLwajqE6num8U4z99FfKfn1A@mail.gmail.com>
From: Pete McCann <mccap@petoni.org>
To: gen-art@ietf.org, draft-ietf-pcn-encoding-comparison.all@tools.ietf.org
Content-Type: text/plain; charset=ISO-8859-1
X-Gm-Message-State: ALoCoQli3d+3qn5O/GU6IO4fNV/xnT0PukbDnJHA8E/6Vdl2K+jV53tJzCak5IuW204RwBB/6AOv
Subject: [Gen-art] Gen-ART Telechat Review of draft-ietf-pcn-encoding-comparison-08
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
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: <http://www.ietf.org/mail-archive/web/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: Wed, 29 Feb 2012 04:33:16 -0000

I am the assigned Gen-ART reviewer for this draft. For background on
Gen-ART, please see the FAQ at
< http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>gt;.

Please wait for direction from your document shepherd
or AD before posting a new version of the draft.

Document: draft-ietf-pcn-encoding-comparison-08
Reviewer: Peter McCann
Review Date: 2012-02-28
IETF LC End Date:
IESG Telechat date: 2012-03-01

Summary: Ready

Major issues: None

Minor issues:

Section 3.3.3.4:
   With the normal mode, the ECN field of the inner header is copied to
   the ECN field of the outer header on encapsulation (like the limited
   functionality option in Section 3.3.3.1).
The limited functionality option says to set the outer header to not-ECT.
This seems to contradict the above statement.

Nits/editorial comments:

Section 3.3.3.3:
   full-
   functionality option in Section 3.3.2.2.
I think you meant "Section 3.3.3.2".  One other place in this
paragraph needs this correction.

Section 4.2:
   The problem with 3-in-1 encoding is that the 10-codepoint does not
   survive decapsulation with the tunneling options in Section 3.3.2.1 -
   3.3.2.3.
Again, you meant 3.3.3.1 - 3.3.3.3