intel / openlldp

Other
54 stars 42 forks source link

Persistent willing2 #13

Closed imanti closed 6 years ago

imanti commented 6 years ago

This merge contains only changes to honor willing value from config file; The default willing value (which is enabled) has not been changed.

There are two commits in this merge:

189f63a - 8021qaz: Honor willing value from config file Read the willing value of pfc and ets variables directly from the config file before calling set_ieee_hw. In case such value exists and its value is false, then we don't set any pfc/ets data to write to hw.

59c9a54 - 8021qaz: Remove unused variable Previous commit had introduced an unused variable.

imanti commented 6 years ago

@dmertman

As requested in PR #12, this PR does not modify the default willing value.

dmertman commented 6 years ago

Ron, Could you do a quick validation on this for me? Thanks

imanti commented 6 years ago

@dmertman Any news with the validation?

praeluceo commented 6 years ago

Validation has confirmed that this PR builds fine and is functionally the same as what we were testing previously. Looks good.