]> git.baikalelectronics.ru Git - kernel.git/commit
tipc: fix false detection of retransmit failures
authorTuong Lien <tuong.t.lien@dektech.com.au>
Thu, 15 Aug 2019 03:24:08 +0000 (10:24 +0700)
committerDavid S. Miller <davem@davemloft.net>
Fri, 16 Aug 2019 23:27:13 +0000 (16:27 -0700)
commit7b90db69ae62f7bb4fb1a065ea20d72194034d10
tree907be3ce9063e05a8375baa21bd9e07c951cfab1
parentea07e13bc69bdc16c66f362c0e93a2088ce3824b
tipc: fix false detection of retransmit failures

This commit eliminates the use of the link 'stale_limit' & 'prev_from'
(besides the already removed - 'stale_cnt') variables in the detection
of repeated retransmit failures as there is no proper way to initialize
them to avoid a false detection, i.e. it is not really a retransmission
failure but due to a garbage values in the variables.

Instead, a jiffies variable will be added to individual skbs (like the
way we restrict the skb retransmissions) in order to mark the first skb
retransmit time. Later on, at the next retransmissions, the timestamp
will be checked to see if the skb in the link transmq is "too stale",
that is, the link tolerance time has passed, so that a link reset will
be ordered. Note, just checking on the first skb in the queue is fine
enough since it must be the oldest one.
A counter is also added to keep track the actual skb retransmissions'
number for later checking when the failure happens.

The downside of this approach is that the skb->cb[] buffer is about to
be exhausted, however it is always able to allocate another memory area
and keep a reference to it when needed.

Fixes: 55212a0b8715 ("tipc: simplify stale link failure criteria")
Reported-by: Hoang Le <hoang.h.le@dektech.com.au>
Acked-by: Ying Xue <ying.xue@windriver.com>
Acked-by: Jon Maloy <jon.maloy@ericsson.com>
Signed-off-by: Tuong Lien <tuong.t.lien@dektech.com.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/tipc/link.c
net/tipc/msg.h