On Thu, Dec 31, 2015 at 12:07:12PM +0100, Linus Lüssing wrote:
Suggestion: Change "If the OGMv2 sequence number is not newer or equal compared to the last received OGMV2 (from any neighbor for this originator) and if the throughput is not better, drop the OGMv2" to: "[...] (from the currently selected router for this originator) [...]"
(-> is another sequence number check here redundant? see "Age check above"? maybe some merging+reordering here?)
Another issue I just noticed with the according sentence we have in the spec right now:
Bad news never travells:
We would ignore updates from our selected router if the path throughput got worse - even if it's a new sequence number.
* Add a branch: If seqno is new + from currently selected router: -> accept (even if path throughput is lower than with the previous seqno)