On 3/13/17 5:10 AM, Robert Shearman wrote: > Doesn't this leave the problem that if the device's link goes down and > then the device gets deleted the alive count will be decremented twice > for the same path?
yes. and it exposes another bug in multipath selection. > > Perhaps it would be better to change the condition for decrementing the > alive count to be "!(nh->nh_flags & (RTNH_F_LINKDOWN | RTNH_F_DEAD))"? or maybe the logic in mpls_ifup is the way to go -- reset the alive counter based on the sum of each nexhop's status. I'll send more patches soon.