[lisp] Tsvart telechat review of draft-ietf-lisp-rfc6830bis-19

Brian Trammell <ietf@trammell.ch> Tue, 25 September 2018 09:26 UTC

Return-Path: <ietf@trammell.ch>
X-Original-To: lisp@ietf.org
Delivered-To: lisp@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 68B64131283; Tue, 25 Sep 2018 02:26:20 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Brian Trammell <ietf@trammell.ch>
To: <tsv-art@ietf.org>
Cc: draft-ietf-lisp-rfc6830bis.all@ietf.org, ietf@ietf.org, lisp@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.84.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <153786758036.30012.5926379631854398353@ietfa.amsl.com>
Date: Tue, 25 Sep 2018 02:26:20 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/lisp/FvxScE5UCk4GucX2ihV5uxPMEMQ>
Subject: [lisp] Tsvart telechat review of draft-ietf-lisp-rfc6830bis-19
X-BeenThere: lisp@ietf.org
X-Mailman-Version: 2.1.29
List-Id: List for the discussion of the Locator/ID Separation Protocol <lisp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/lisp>, <mailto:lisp-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/lisp/>
List-Post: <mailto:lisp@ietf.org>
List-Help: <mailto:lisp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/lisp>, <mailto:lisp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 25 Sep 2018 09:26:21 -0000

Reviewer: Brian Trammell
Review result: Ready with Nits

Thanks for the revisions. Following up on my previous tsvart review, there is
one remaining nit from previous transport reviews:

- Section 5.3, the list "When doing ITR/PITR encapsulation", third item needs a
citation fix:

OLD:

      The 'Explicit Congestion Notification' (ECN) field (bits 6 and 7
      of the IPv6 'Traffic Class' field) requires special treatment in
      order to avoid discarding indications of congestion [RFC3168].

NEW:

     The 'Explicit Congestion Notification' (ECN) field (bits 6 and 7
      of the IPv6 'Traffic Class' field) requires special treatment in
      order to avoid discarding indications of congestion [RFC6040].

Thanks, cheers,

Brian