Hi Pedro, Thanks for the quick reply. Please find comments inline.
On 11-06-07 07:15 PM, Pedro Marques wrote:
Suresh, Thank you for the review. Regarding Section 5: The Length can be either 1 or 2 octets. 4271 defines Attr Flags. Bit 0 is the Optional bit; Bit 1 is the Transitive bit; Bit 3 is the Extended length bit. This document specified that bits 0 and 1 must be set to 1 (Optional + transitive). The Extended length bit must be set according to the attribute payload as per 4271. The convention is the same as for any other BGP attribute. I'll add a clarification to this effect.
Perfect. I did read up on Section 4.3 of RFC4271 before I sent my review but it was not clear whether setting the bit was per attribute type code or it was runtime. I think a clarification would be good here.
Regarding security considerations: - The resource management considerations exist regarding routes as well as attributes, with or without this specification. I don't mind highlighting it, if you believe that improves the specification.
I do believe this is worth mentioning, but I leave it up to your discretion on whether you want to add any text.
Thanks Suresh _______________________________________________ Gen-art mailing list [email protected] https://www.ietf.org/mailman/listinfo/gen-art
