Les Ginsberg (ginsberg) <ginsb...@cisco.com> writes:
Chris -
Issue: Under both the Node and Link sub-tlv's the MSD type (1?) is not
actually mentioned, only the "MSD value", if one was pedantic it would mean
that regardless of the type the value was always the same, certainly not what
is intended. :)
[Les:] In both sections the draft says (emphasis added):
"Value: field consists of one or more pairs of a 1 octet MSD-Type
(IANA Registry) and 1 octet Value.
Why do you see this as unclear?
Let me come at this a different way:
Type -- specified: 23
Value -- specified: pairs of MSD type and MSD values.
MSD-Type -- *unspecified*
MSD-Value -- specified "the MSD value is a number ..."
One might could infer after looking at the *current* registry that the only
valid value could be:
Type: 1 -- Base MPLS Imposition MSD
However, the text also doesn't even use "Base MPLS Imposition MSD" to describe
the MSD value here (it talks about this in later sections), so I'd say it's
under-specified at this point.
Or is it in fact the case that you are saying that no matter what the MSD-Type
is the MSD value will always be as defined currently in sections 2 and 3?
That's would be surprising enough that I think it would need to be made
explicit.
FWIW, I noticed this by doing a diff so here's the OSPF text:
"MSD Type 1 (IANA Section), MSD and the Value field contains the MSD
of the originating router. Node MSD is a number in the range of
0-255. 0 represents lack of the ability to impose MSD stack of any
depth; any other value represents that of the node. This value
SHOULD represent the minimum value supported by a node."
That text looks a bit off too ("MSD and the Value field"?), but at least its saying the
MSD type is "1".
Note: this comment refers to both the Node and Link sub-TLVs.
Issue: The OSPF version adds text about what to do in the presence of
multiple instances of the same TLV. This highlighted the fact that the IS-IS
draft doesn't do this, but also doesn't talk about there only being 1 allowed.
[Les:] MSD inherits the procedures defined in
https://tools.ietf.org/html/rfc7981#section-3 .
There is therefore no need for further specification.
Well that might cover the Node TLV, but not the Link TLV, right?
The meaning of "one allowed" is not the same in IS-IS. Clearly multiple MSD
sub-TLVs are allowed since there are 255 possible MSD types and they would not all fit
into a single sub-TLV.
The main point I'm making here is that the OSPF document goes out of it's way
to be explicit about multiple values not being allowed, the IS-IS document
leaves this unspecified.
Thanks,
Chris.
_______________________________________________
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr
