[trill] Review of draft-ietf-trill-p2mp-bfd-04

Carlos Pignataro <cpignata@cisco.com> Thu, 23 March 2017 19:20 UTC

Return-Path: <cpignata@cisco.com>
X-Original-To: trill@ietf.org
Delivered-To: trill@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 635E7129C0B; Thu, 23 Mar 2017 12:20:29 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Carlos Pignataro <cpignata@cisco.com>
To: <rtg-dir@ietf.org>
Cc: draft-ietf-trill-p2mp-bfd.all@ietf.org, ietf@ietf.org, trill@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.48.1
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <149029682927.22415.11722241596169339182@ietfa.amsl.com>
Date: Thu, 23 Mar 2017 12:20:29 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/trill/Jj8OrBZWnzMmoJCAAXtzVrFCGtg>
Subject: [trill] Review of draft-ietf-trill-p2mp-bfd-04
X-BeenThere: trill@ietf.org
X-Mailman-Version: 2.1.22
List-Id: "Developing a hybrid router/bridge." <trill.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/trill>, <mailto:trill-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/trill/>
List-Post: <mailto:trill@ietf.org>
List-Help: <mailto:trill-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/trill>, <mailto:trill-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 23 Mar 2017 19:20:29 -0000

Reviewer: Carlos Pignataro
Review result: Has Nits

Hi,

This is a short yet well written document. Looks really good. Just a
couple of small questions/comments:

1. It's not clear exactly what from RFC 7175 is being updated by
Section 4. This is based on the last paragraph of the Intro, yet
Section 5 also seems relevant to that update.
2. In the IANA COnsiderations, it would be useful to explicitly list
that the "RBridge Channel Protocols" registry is part of the
"Transparent Interconnection of Lots of Links (TRILL) Parameters" 

Thanks,

Carlos Pignataro.