From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [net 1/1] tipc: initialize broadcast link stale counter correctly Date: Mon, 15 Oct 2018 22:04:39 -0700 (PDT) Message-ID: <20181015.220439.2254090460096435575.davem@davemloft.net> References: <1539288149-24122-1-git-send-email-jon.maloy@ericsson.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, gordan.mihaljevic@dektech.com.au, tung.q.nguyen@dektech.com.au, hoang.h.le@dektech.com.au, canh.d.luu@dektech.com.au, ying.xue@windriver.com, tipc-discussion@lists.sourceforge.net To: jon.maloy@ericsson.com Return-path: Received: from shards.monkeyblade.net ([23.128.96.9]:43366 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727008AbeJPMxP (ORCPT ); Tue, 16 Oct 2018 08:53:15 -0400 In-Reply-To: <1539288149-24122-1-git-send-email-jon.maloy@ericsson.com> Sender: netdev-owner@vger.kernel.org List-ID: From: Jon Maloy Date: Thu, 11 Oct 2018 22:02:29 +0200 > In the commit referred to below we added link tolerance as an additional > criteria for declaring broadcast transmission "stale" and resetting the > unicast links to the affected node. > > Unfortunately, this 'improvement' introduced two bugs, which each and > one alone cause only limited problems, but combined lead to seemingly > stochastic unicast link resets, depending on the amount of broadcast > traffic transmitted. > > The first issue, a missing initialization of the 'tolerance' field of > the receiver broadcast link, was recently fixed by commit 047491ea334a > ("tipc: set link tolerance correctly in broadcast link"). > > Ths second issue, where we omit to reset the 'stale_cnt' field of > the same link after a 'stale' period is over, leads to this counter > accumulating over time, and in the absence of the 'tolerance' criteria > leads to the above described symptoms. This commit adds the missing > initialization. > > Fixes: a4dc70d46cf1 ("tipc: extend link reset criteria for stale packet > retransmission") > > Signed-off-by: Jon Maloy Applied, thanks Jon. In the future, please always make your Fixes: tag one single line no matter how long it is. And also do not separate it from other tags like the signoff with empty lines. All of the tags should be collected together without any empty lines in between. Thank you.