Re: [GROW] I-D Action: draft-ietf-grow-bmp-tlv-ebit-04.txt

[Paolo Lucente <[e-mail]>]

Hi Luuk,

Inline:

On 1/11/23 10:45, Luuk Hendriks wrote:
> Hi Paolo, all,
> 
> Thanks for the update. I have a question about something that was actually
> already part of the previous version:
> 
> 
> Sec 3.2:
> 
> In a non-indexed enterprise-specific TLV, the Length includes the PEN, which
> I guess helps in making it backward compatible.
> For indexed TLVs, the Length does not include the Index (Sec3 in bmp-tlv-13),
> but what about indexed enterprise-specific TLVs? Does the length include the
> length of the PEN, but not the Index?

Exactly!

> If indeed the PEN is included and the Index is not, would it make sense to swap
> the Index and PEN fields? Feels more consistent to interpret/process to me, but
> perhaps I'm missing any downsides of such a reordering.

That makes a lot of sense, indeed. Will update this in the next revision 
of the document. Thank you!

Paolo