ath9k: fix a small race condition in the tx_last_beacon patch
authorFelix Fietkau <nbd@openwrt.org>
Wed, 18 May 2011 12:03:04 +0000 (12:03 +0000)
committerFelix Fietkau <nbd@openwrt.org>
Wed, 18 May 2011 12:03:04 +0000 (12:03 +0000)
SVN-Revision: 26933

package/mac80211/patches/572-ath9k_tx_last_beacon.patch

index 4f4c3b8630a014a298852a70fe894c28d9cdf6dc..f8aad94b7950bad7d6471fe0b43782de50389fe0 100644 (file)
@@ -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;
  }
  
 +              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;
  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,