From: Ivan Vecera <c...@cera.cz> Date: Fri, 19 May 2017 19:30:43 +0200
> Current bridge code incorrectly handles starting/stopping of hello and > hold timers during STP enable/disable. > > 1. Timers are stopped in br_stp_start() during NO_STP->USER_STP > transition. The timers are already stopped in NO_STP state so > this is confusing no-op. > > 2. During USER_STP->NO_STP transition the timers are started. This > does not make sense and is confusion because the timer should not be > active in NO_STP state. > > Cc: da...@davemloft.net > Cc: sas...@cumulusnetworks.com > Cc: step...@networkplumber.org > Cc: bri...@lists.linux-foundation.org > Cc: lucien....@gmail.com > Cc: niko...@cumulusnetworks.com > Signed-off-by: Ivan Vecera <c...@cera.cz> Applied.
