tipc: extend broadcast link initialization criteria
At first contact between two nodes, an endpoint might sometimes have time to send out a LINK_PROTOCOL/STATE packet before it has received the broadcast initialization packet from the peer, i.e., before it has received a valid broadcast packet number to add to the 'bc_ack' field of the protocol message. This means that the peer endpoint will receive a protocol packet with an invalid broadcast acknowledge value of 0. Under unlucky circumstances this may lead to the original, already received acknowledge value being overwritten, so that the whole broadcast link goes stale after a while. We fix this by delaying the setting of the link field 'bc_peer_is_up' until we know that the peer really has received our own broadcast initialization message. The latter is always sent out as the first unicast message on a link, and always with seqeunce number 1. Because of this, we only need to look for a non-zero unicast acknowledge value in the arriving STATE messages, and once that is confirmed we know we are safe and can set the mentioned field. Before this moment, we must ignore all broadcast acknowledges from the peer. 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>
This commit is contained in:
parent
34ee32c9a5
commit
2d18ac4ba7
|
@ -1559,7 +1559,12 @@ void tipc_link_bc_sync_rcv(struct tipc_link *l, struct tipc_msg *hdr,
|
||||||
if (!msg_peer_node_is_up(hdr))
|
if (!msg_peer_node_is_up(hdr))
|
||||||
return;
|
return;
|
||||||
|
|
||||||
l->bc_peer_is_up = true;
|
/* Open when peer ackowledges our bcast init msg (pkt #1) */
|
||||||
|
if (msg_ack(hdr))
|
||||||
|
l->bc_peer_is_up = true;
|
||||||
|
|
||||||
|
if (!l->bc_peer_is_up)
|
||||||
|
return;
|
||||||
|
|
||||||
/* Ignore if peers_snd_nxt goes beyond receive window */
|
/* Ignore if peers_snd_nxt goes beyond receive window */
|
||||||
if (more(peers_snd_nxt, l->rcv_nxt + l->window))
|
if (more(peers_snd_nxt, l->rcv_nxt + l->window))
|
||||||
|
|
Loading…
Reference in New Issue