[trill] Alissa Cooper's Discuss on draft-ietf-trill-smart-endnodes-10: (with DISCUSS and COMMENT)
Alissa Cooper <alissa@cooperw.in> Tue, 06 March 2018 20:45 UTC
Return-Path: <alissa@cooperw.in>
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 A0FD91270B4; Tue, 6 Mar 2018 12:45:00 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Alissa Cooper <alissa@cooperw.in>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-trill-smart-endnodes@ietf.org, trill-chairs@ietf.org, shares@ndzh.com, trill@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.74.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <152036910061.28346.1223384759329179194.idtracker@ietfa.amsl.com>
Date: Tue, 06 Mar 2018 12:45:00 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/trill/g40p63SLqq1YpRdLcEt-dy6ZtjM>
Subject: [trill] Alissa Cooper's Discuss on draft-ietf-trill-smart-endnodes-10: (with DISCUSS and COMMENT)
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: Tue, 06 Mar 2018 20:45:01 -0000
Alissa Cooper has entered the following ballot position for draft-ietf-trill-smart-endnodes-10: Discuss When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html for more information about IESG DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-trill-smart-endnodes/ ---------------------------------------------------------------------- DISCUSS: ---------------------------------------------------------------------- This should hopefully be easy to fix and was pointed out by the Gen-ART reviewer: All of section 4.3 is confusing as to what the length of the TLV really is. Row 3 in the diagram says 2 bytes or 4 bytes, but the number of bits called out in bullets 4 and 5 below it don't seem to add up to those things. Maybe it would be better to draw a diagram with F=0 and a separate diagram with F=1. Please make it clear both in the diagram and in the text what the expected lengths of the fields are -- I find it particularly confusing that the number of bits pictured doesn't align with the number of bits specified in the text per field. ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- Please also look at the Gen-ART reviewer's other comments.
- [trill] Alissa Cooper's Discuss on draft-ietf-tri… Alissa Cooper
- Re: [trill] Alissa Cooper's Discuss on draft-ietf… hu.fangwei
- Re: [trill] Alissa Cooper's Discuss on draft-ietf… Alissa Cooper
- Re: [trill] Alissa Cooper's Discuss on draft-ietf… Donald Eastlake
- Re: [trill] Alissa Cooper's Discuss on draft-ietf… Donald Eastlake