diff options
author | Jon Paul Maloy <jon.maloy@ericsson.com> | 2015-07-31 00:24:15 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-07-31 02:25:13 +0200 |
commit | cbeb83ca68dcedf69b336fd1c5263658cbe5b51e (patch) | |
tree | b22597a2b2c77c52c320210316433a939cc699b7 /net/tipc/bearer.c | |
parent | Merge branch 'for-upstream' of git://git.kernel.org/pub/scm/linux/kernel/git/... (diff) | |
download | linux-cbeb83ca68dcedf69b336fd1c5263658cbe5b51e.tar.xz linux-cbeb83ca68dcedf69b336fd1c5263658cbe5b51e.zip |
tipc: eliminate function tipc_link_activate()
The function tipc_link_activate() is redundant, since it mostly performs
settings that have already been done in a preceding tipc_link_reset().
There are three exceptions to this:
- The actual state change to TIPC_LINK_WORKING. This should anyway be done
in the FSM, and not in a separate function.
- Registration of the link with the bearer. This should be done by the
node, since we don't want the link to have any knowledge about its
specific bearer.
- Call to tipc_node_link_up() for user access registration. With the new
role distribution between link aggregation and link level this becomes
the wrong call order; tipc_node_link_up() should instead be called
directly as a result of a TIPC_LINK_UP event, hence by the node itself.
This commit implements those changes.
Tested-by: Ying Xue <ying.xue@windriver.com>
Signed-off-by: Jon Maloy <jon.maloy@ericsson.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/tipc/bearer.c')
0 files changed, 0 insertions, 0 deletions