[Pals] Mirja Kühlewind's Discuss on draft-ietf-pals-mpls-tp-pw-over-bidir-lsp-08: (with DISCUSS and COMMENT)
"Mirja Kuehlewind" <ietf@kuehlewind.net> Tue, 05 July 2016 16:38 UTC
Return-Path: <ietf@kuehlewind.net>
X-Original-To: pals@ietf.org
Delivered-To: pals@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 2A8D012D094; Tue, 5 Jul 2016 09:38:46 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Mirja Kuehlewind <ietf@kuehlewind.net>
To: The IESG <iesg@ietf.org>
X-Test-IDTracker: no
X-IETF-IDTracker: 6.25.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <20160705163846.22350.79584.idtracker@ietfa.amsl.com>
Date: Tue, 05 Jul 2016 09:38:46 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/pals/xPKXW4Wt8YZvQXiang6akpLJjVA>
Cc: stewart.bryant@gmail.com, pals-chairs@ietf.org, draft-ietf-pals-mpls-tp-pw-over-bidir-lsp@ietf.org, pals@ietf.org
Subject: [Pals] Mirja Kühlewind's Discuss on draft-ietf-pals-mpls-tp-pw-over-bidir-lsp-08: (with DISCUSS and COMMENT)
X-BeenThere: pals@ietf.org
X-Mailman-Version: 2.1.17
List-Id: "Pseudowire And LDP-enabled Services dicussion list." <pals.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/pals>, <mailto:pals-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/pals/>
List-Post: <mailto:pals@ietf.org>
List-Help: <mailto:pals-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/pals>, <mailto:pals-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 05 Jul 2016 16:38:46 -0000
Mirja Kühlewind has entered the following ballot position for draft-ietf-pals-mpls-tp-pw-over-bidir-lsp-08: 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-pals-mpls-tp-pw-over-bidir-lsp/ ---------------------------------------------------------------------- DISCUSS: ---------------------------------------------------------------------- I think the protocol specification is not complete: - What happens if none of the two S and C bits are set? - What happens if more the one sub-TLV is present? Actually I think that the protocol design is more complicated than needed and simplifying it would resolve these error cases. But as that's not a DISCUSS reason, please see more detailed comments below! ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- A few questions/comments: 1) The shepherd writeup says: "An IPR discolusure has been filed and the WG has not remarked on this." Does this mean the wg is not aware of it or it is aware and nobody commented? 2) Acronym: I found a few acronyms that where not spelled out, e.g. FEC as well as LDP and TE in the abstract. One can argue if that's needed but usually I think it helps. Further there are quite a few acronyms that are introduced somewhere and then never used again (e.g. Point-to-Point (P2P); that makes it actually harder to read, so I would recommend to remove them. 3) I think I don't understand the following MUST: "PSN Tunnel Binding TLV is an optional TLV and MUST be carried in the Label Mapping message [RFC5036] if PW to LSP binding is required." and again later "To perform PSN binding, the Label Mapping messages MUST carry a PSN Tunnel Binding TLV". Maybe just remove it? 4) Why is there a "MUST be zero" and a "Reserved" field? Shouldn't this all just be a large "Reserved" field? And the "Reserve" field is not discussed. 5) On the T (Tunnel Representation) bit: Why is that an extra bit? Usually you can simply use the length field here and then either have the LSP Number fields in the sub-TLV or not. 6) Why are there two bits for C and S if they are mutual exclusive and one of them has to be set? In this case you should use one bit and say e.g. 0 means S(trict) and 1 means C(o-routed path). This would also avoid the error case when both are set! (Otherwise if it should be possible to not set both flags, this case is not described in the draft and must be added.) 7) Sub-TLV: I'm not an LDP expert but I assume having sub-TLV is not a general concept that is widely used this way but something introduced in this document. If that's the case, I have a couple of questions/remarks: - Given that the PSN Tunnel Binding TLV already has a length field that includes the sub-TLV, why do you need another length field in the sub-TLV? - Why is there another 'Reserved' field in the sub_TLV (that is also not discussed)? - If you already have different types of sub-TLVs here, why don't you simply define some with or without the the LSP Number fields instead of the T bit? - And finally do you really need these sub-TLVs or would a flag be enough to indicate IPv4 or IPv6 (or again just use the length field to distinguish)? - And this should be normative "Each PSN Tunnel Binding TLV can only have one such sub-TLV." -> "Each PSN Tunnel Binding TLV MUST have only one such sub-TLV." and you must specify what happen if more then one is presented (if you actually want to use sub-TLVs). 8) There are a couple of MUSTs basically twice in the document (once in section 2 and another time in sections 4 and 5). I would recommend to avoid these duplications and subsequent redundancy and really state it only once to avoid any future confusions.
- Re: [Pals] Mirja Kühlewind's Discuss on draft-iet… Mach Chen
- Re: [Pals] Mirja Kühlewind's Discuss on draft-iet… Gmail
- Re: [Pals] Mirja Kühlewind's Discuss on draft-iet… Mirja Kuehlewind (IETF)
- Re: [Pals] Mirja Kühlewind's Discuss on draft-iet… BRUNGARD, DEBORAH A
- Re: [Pals] Mirja Kühlewind's Discuss on draft-iet… Stewart Bryant
- Re: [Pals] Mirja Kühlewind's Discuss on draft-iet… Stewart Bryant
- Re: [Pals] Mirja Kühlewind's Discuss on draft-iet… Mach Chen
- Re: [Pals] Mirja Kühlewind's Discuss on draft-iet… Mirja Kuehlewind (IETF)
- Re: [Pals] Mirja Kühlewind's Discuss on draft-iet… Mach Chen
- Re: [Pals] Mirja Kühlewind's Discuss on draft-iet… Mirja Kuehlewind (IETF)
- Re: [Pals] Mirja Kühlewind's Discuss on draft-iet… Mach Chen
- [Pals] Mirja Kühlewind's Discuss on draft-ietf-pa… Mirja Kuehlewind
- Re: [Pals] Mirja Kühlewind's Discuss on draft-iet… Mach Chen
- Re: [Pals] Mirja Kühlewind's Discuss on draft-iet… Mirja Kuehlewind (IETF)
- Re: [Pals] Mirja Kühlewind's Discuss on draft-iet… Mach Chen