diff options
author | Felix Fietkau <nbd@openwrt.org> | 2011-05-18 12:03:04 +0000 |
---|---|---|
committer | Felix Fietkau <nbd@openwrt.org> | 2011-05-18 12:03:04 +0000 |
commit | 6400498b35fe91c25172439bc1739abd4708d84c (patch) | |
tree | 12e0d950b09dbc66026770dedbb50b14b19ed166 | |
parent | e7dfadd012b0bcafce5ff4de23782d0c3d9acbdd (diff) | |
download | upstream-6400498b35fe91c25172439bc1739abd4708d84c.tar.gz upstream-6400498b35fe91c25172439bc1739abd4708d84c.tar.bz2 upstream-6400498b35fe91c25172439bc1739abd4708d84c.zip |
ath9k: fix a small race condition in the tx_last_beacon patch
SVN-Revision: 26933
-rw-r--r-- | package/mac80211/patches/572-ath9k_tx_last_beacon.patch | 15 |
1 files changed, 10 insertions, 5 deletions
diff --git a/package/mac80211/patches/572-ath9k_tx_last_beacon.patch b/package/mac80211/patches/572-ath9k_tx_last_beacon.patch index 4f4c3b8630..f8aad94b79 100644 --- a/package/mac80211/patches/572-ath9k_tx_last_beacon.patch +++ b/package/mac80211/patches/572-ath9k_tx_last_beacon.patch @@ -54,7 +54,7 @@ --- a/drivers/net/wireless/ath/ath9k/main.c +++ b/drivers/net/wireless/ath/ath9k/main.c -@@ -2314,6 +2314,43 @@ static bool ath9k_tx_frames_pending(stru +@@ -2314,6 +2314,48 @@ static bool ath9k_tx_frames_pending(stru return false; } @@ -80,16 +80,21 @@ + return 0; + + if (!sc->beacon.tx_processed) { ++ tasklet_disable(&sc->bcon_tasklet); ++ + bf = avp->av_bcbuf; + if (!bf || !bf->bf_mpdu) -+ return 0; ++ goto skip; + + status = ath9k_hw_txprocdesc(ah, bf->bf_desc, &ts); + if (status == -EINPROGRESS) -+ return 0; ++ goto skip; + + sc->beacon.tx_processed = true; -+ sc->beacon.tx_last = !(ts.ts_status & ATH9K_TXERR_FILT); ++ sc->beacon.tx_last = !(ts.ts_status & ATH9K_TXERR_MASK); ++ ++skip: ++ tasklet_enable(&sc->bcon_tasklet); + } + + return sc->beacon.tx_last; @@ -98,7 +103,7 @@ struct ieee80211_ops ath9k_ops = { .tx = ath9k_tx, .start = ath9k_start, -@@ -2338,4 +2375,5 @@ struct ieee80211_ops ath9k_ops = { +@@ -2338,4 +2380,5 @@ struct ieee80211_ops ath9k_ops = { .set_coverage_class = ath9k_set_coverage_class, .flush = ath9k_flush, .tx_frames_pending = ath9k_tx_frames_pending, |