|Standard:||802.1AB-2009||Clause:||6.6.1, 188.8.131.52.2, 10.5.2|
|Clause title:||LLDPDU and TLV error handling|
Rationale for revision
The text for General validation rules for TLVs (184.108.40.206.2) isn't consistent with the text for LLDPDU and TLV error handling (6.6.1) The latter implies that if an LLDPDU contains a detectable error then the entire PDU and its contents are discarded, but 220.127.116.11.2 c) states that there are cases (and explicitly cites E.3.3) where it is acceptable to discard a single errored TLV and process the rest of the PDU. There have been questions from implementers regarding what is intended. Additionally, there are some inconsistent statements about how to handle the incrementing of statsFramesInErrorsTotal. The description of statsFramesInErrorsTotal in 10.5.2 is: A count of all LLDPUPDUs received at the port with one or more detectable errors 18.104.22.168). Note: The standard is missing the opening parenthesis on the reference. 22.214.171.124 agrees with the definition in 10.5.2. Therefore it should only increment once per frame. However, the TLV validation rules in 126.96.36.199.2 increment statsFramesInErrorsTotal whenever they increment statsTLVsDiscardedTotal. They could increment it multiple times per frame.
Change 6.6.1 to say something like the following: The LLDPDU is checked to ensure that it contains the correct sequence of mandatory TLVs and then each optional TLV is validated in succession. LLDPDUs that contain detectable errors in the mandatory TLVs are discarded. Optional TLVs that contain detectable errors are discarded (see 188.8.131.52.2 c)). TLVs that are not recognized, but that also contain no basic format errors, are assumed to be valid and are stored for possible later retrieval by network management (see 184.108.40.206.1 and 220.127.116.11). Change 18.104.22.168.2 Remove mention of statsFramesInErrorsTotal from a) through e) and add an additional rule at the end saying that statsFramesInErrorsTotal is incremented at the end of processing if the LLDPDU or any of the TLVs in it were discarded. Change 10.5.2 Add the missing opening parenthesis to item b) 3).
Impact on existing networks
Potentially, fewer LLDPDUs will be discarded in their entirety and only incorrect TLVs will be missing. The MIB error statistics may show fewer statsFramesInErrorsTotal. These differences are very unlikely as incorrectly formatted TLVs are rare, so the overall impact is very minimal.
|Name:||Paul Congdon, Pat Thaler||Email:||email@example.com, firstname.lastname@example.org|