diff options
| author |   Jon Paul Maloy <jon.maloy@ericsson.com> | 2016-04-15 13:33:06 -0400 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2016-04-15 16:09:06 -0400 |
| commit | de7e07f9ee14f47d05aa43046404c2904f0247dc (patch) | |
| tree | fc4bf780d2d2b3273d7533deb3d5e08a4ddec6ba /net/tipc | |
| parent | tipc: refactor function tipc_link_timeout() (diff) | |
| download | linux-dev-de7e07f9ee14f47d05aa43046404c2904f0247dc.tar.xz linux-dev-de7e07f9ee14f47d05aa43046404c2904f0247dc.zip | |
tipc: ensure that first packets on link are sent in order
In some link establishment scenarios we see that packet #2 may be sent
out before packet #1, forcing the receiver to demand retransmission of
the missing packet. This is harmless, but may cause confusion among
people tracing the packet flow.
Since this is extremely easy to fix, we do so by adding en extra send
call to the bearer immediately after the link has come up.
Acked-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')
| -rw-r--r-- | net/tipc/node.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/net/tipc/node.c b/net/tipc/node.c index ace178fd3850..b00e12cda66c 100644 --- a/net/tipc/node.c +++ b/net/tipc/node.c @@ -581,8 +581,12 @@ static void __tipc_node_link_up(struct tipc_node *n, int bearer_id, static void tipc_node_link_up(struct tipc_node *n, int bearer_id, struct sk_buff_head *xmitq) { + struct tipc_media_addr *maddr; + tipc_node_write_lock(n); __tipc_node_link_up(n, bearer_id, xmitq); + maddr = &n->links[bearer_id].maddr; + tipc_bearer_xmit(n->net, bearer_id, xmitq, maddr); tipc_node_write_unlock(n); } |