From a339e58d7241876b0988fe29db2ed19ad381ae6e Mon Sep 17 00:00:00 2001 From: Felix Fietkau Date: Mon, 10 Jan 2011 00:25:17 +0000 Subject: [PATCH] ath9k: reduce the number of false positives triggered by the baseband hang check by only running it when beacon transmission is blocked SVN-Revision: 24954 --- .../patches/540-ath9k_bb_hang_check_fix.patch | 30 +++++++++++++++++++ 1 file changed, 30 insertions(+) create mode 100644 package/mac80211/patches/540-ath9k_bb_hang_check_fix.patch diff --git a/package/mac80211/patches/540-ath9k_bb_hang_check_fix.patch b/package/mac80211/patches/540-ath9k_bb_hang_check_fix.patch new file mode 100644 index 0000000000..19c6abd701 --- /dev/null +++ b/package/mac80211/patches/540-ath9k_bb_hang_check_fix.patch @@ -0,0 +1,30 @@ + +--- a/drivers/net/wireless/ath/ath9k/main.c ++++ b/drivers/net/wireless/ath/ath9k/main.c +@@ -251,6 +251,9 @@ int ath_set_channel(struct ath_softc *sc + if (!ath_stoprecv(sc)) + stopped = false; + ++ if (!ath9k_hw_check_alive(ah)) ++ stopped = false; ++ + /* XXX: do not flush receive queue here. We don't want + * to flush data frames already in queue because of + * changing channel. */ +@@ -602,7 +605,15 @@ void ath9k_tasklet(unsigned long data) + + spin_lock(&sc->sc_pcu_lock); + +- if (!ath9k_hw_check_alive(ah)) ++ /* ++ * Only run the baseband hang check if beacons stop working in AP or ++ * IBSS mode, because it has a high false positive rate. For station ++ * mode it should not be necessary, since the upper layers will detect ++ * this through a beacon miss automatically and the following channel ++ * change will trigger a hardware reset anyway ++ */ ++ if (ath9k_hw_numtxpending(ah, sc->beacon.beaconq) != 0 && ++ !ath9k_hw_check_alive(ah)) + ieee80211_queue_work(sc->hw, &sc->hw_check_work); + + if (ah->caps.hw_caps & ATH9K_HW_CAP_EDMA) -- 2.30.2