diff options
author | Felix Fietkau <nbd@openwrt.org> | 2009-12-05 18:08:05 +0100 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2009-12-07 22:54:59 +0100 |
commit | 815833e7ecf0b9a017315cae6aef4d7cd9517681 (patch) | |
tree | 68cc8441a6735b9b8917c1ea67cdd97b4702665a /drivers | |
parent | mac80211: Fix bug in computing crc over dynamic IEs in beacon (diff) | |
download | linux-815833e7ecf0b9a017315cae6aef4d7cd9517681.tar.xz linux-815833e7ecf0b9a017315cae6aef4d7cd9517681.zip |
ath9k: fix tx status reporting
This patch fixes a bug in ath9k's tx status check, which
caused mac80211 to consider regularly transmitted unicast frames
as un-acked.
When checking the ts_status field for errors, it needs to be masked
with ATH9K_TXERR_FILT, because this field also contains other fields
like ATH9K_TX_ACKED.
Without this patch, AP mode is pretty much unusable, as hostapd
checks the ACK status for the frames that it injects.
Signed-off-by: Felix Fietkau <nbd@openwrt.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/net/wireless/ath/ath9k/xmit.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/ath/ath9k/xmit.c b/drivers/net/wireless/ath/ath9k/xmit.c index 564c6cb1c2b4..2a11cc57ceea 100644 --- a/drivers/net/wireless/ath/ath9k/xmit.c +++ b/drivers/net/wireless/ath/ath9k/xmit.c @@ -2078,7 +2078,7 @@ static void ath_tx_processq(struct ath_softc *sc, struct ath_txq *txq) &txq->axq_q, lastbf->list.prev); txq->axq_depth--; - txok = (ds->ds_txstat.ts_status == 0); + txok = !(ds->ds_txstat.ts_status & ATH9K_TXERR_FILT); txq->axq_tx_inprogress = false; spin_unlock_bh(&txq->axq_lock); |