diff options
author | Felix Fietkau <nbd@openwrt.org> | 2009-12-05 12:08:05 -0500 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2009-12-07 16:54:59 -0500 |
commit | 815833e7ecf0b9a017315cae6aef4d7cd9517681 (patch) | |
tree | 68cc8441a6735b9b8917c1ea67cdd97b4702665a | |
parent | 1814077fd12a9cdf478c10076e9c42094e9d9250 (diff) |
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>
-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) | |||
2078 | &txq->axq_q, lastbf->list.prev); | 2078 | &txq->axq_q, lastbf->list.prev); |
2079 | 2079 | ||
2080 | txq->axq_depth--; | 2080 | txq->axq_depth--; |
2081 | txok = (ds->ds_txstat.ts_status == 0); | 2081 | txok = !(ds->ds_txstat.ts_status & ATH9K_TXERR_FILT); |
2082 | txq->axq_tx_inprogress = false; | 2082 | txq->axq_tx_inprogress = false; |
2083 | spin_unlock_bh(&txq->axq_lock); | 2083 | spin_unlock_bh(&txq->axq_lock); |
2084 | 2084 | ||