tipc: correct settings of broadcast link state
authorJon Paul Maloy <jon.maloy@ericsson.com>
Thu, 19 Nov 2015 19:12:50 +0000 (14:12 -0500)
committerDavid S. Miller <davem@davemloft.net>
Fri, 20 Nov 2015 19:08:51 +0000 (14:08 -0500)
commit9a65083827da0da5fadf9dea5bca25192073b112
treebe156bf8710ba8e8a701ddffaf1e06de86bb4ade
parent52dfc8301248f5008d64a680e832e2f99c55ec9a
tipc: correct settings of broadcast link state

Since commit 5266698661401afc5e ("tipc: let broadcast packet
reception use new link receive function") the broadcast send
link state was meant to always be set to LINK_ESTABLISHED, since
we don't need this link to follow the regular link FSM rules. It
was also the intention that this state anyway shouldn't impact
the run-time working state of the link, since the latter in
reality is controlled by the number of registered peers.

We have now discovered that this assumption is not quite correct.
If the broadcast link is reset because of too many retransmissions,
its state will inadvertently go to LINK_RESETTING, and never go
back to LINK_ESTABLISHED, because the LINK_FAILURE event was not
anticipated. This will work well once, but if it happens a second
time, the reset on a link in LINK_RESETTING has has no effect, and
neither the broadcast link nor the unicast links will go down as
they should.

Furthermore, it is confusing that the management tool shows that
this link is in UP state when that obviously isn't the case.

We now ensure that this state strictly follows the true working
state of the link. The state is set to LINK_ESTABLISHED when
the number of peers is non-zero, and to LINK_RESET otherwise.

Signed-off-by: Jon Maloy <jon.maloy@ericsson.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/tipc/link.c