Hi Paulo, Yunan,
I have a few comments on draft-ietf-grow-bmp-tlv.
1. It would help to introduce the format of Group TLV in Section 4.2.1; just
like how the format of the indexed TLV is introduced in Section 3. Currently,
group_index is learnt only from the example in section 4.2.1.1.
2. In a group TLV, ‘index’ and ‘group_index’ are 2 different fields. ‘index’ is
redundant (always set to 0). Index=0 means that the TLV applies to all NLRIs
which is not true in case of Group TLV.
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
| type=TBD2 | length=0x000a |
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
| index=0 | group_index=0x000c |
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
| value={0x0004, 0x0005, |
| 0x0006} |
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
This could be avoided by using the index space itself for group_index, but
differentiated using a bit called ‘G bit’. This bit, when set, would mean that
the index is a group index.
Format:
0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
| Type (2 octets) | Length (2 octets) |
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
|G| Index (15 bits) |
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
~ Value (variable) ~
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
Example:
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
| type=TBD2 | length=0x000a |
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
|1| index=0x000c |
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
| value={0x0004, 0x0005, |
| 0x0006} |
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
3. Section 4.2.1 says “One or more 2 bytes indexes ..”. Shouldn’t this be “Two
or more 2 byte indexes ..”? If the TLV pertains to a single NLRI, Group TLV is
not necessary.
--
Regards,
Dhananjay
_______________________________________________
GROW mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/grow
