ath9k: add fixes for rx processing and baseband hang detection
Signed-off-by: Felix Fietkau <nbd@openwrt.org> SVN-Revision: 39751
This commit is contained in:
parent
de750029f4
commit
ccccc104b3
5 changed files with 211 additions and 7 deletions
|
@ -1,3 +1,37 @@
|
||||||
|
commit 98d1a6c5b14688ed030e81b889f607be308e0df9
|
||||||
|
Author: Felix Fietkau <nbd@openwrt.org>
|
||||||
|
Date: Mon Feb 24 22:20:32 2014 +0100
|
||||||
|
|
||||||
|
ath9k: fix invalid descriptor discarding
|
||||||
|
|
||||||
|
Only set sc->rx.discard_next to rx_stats->rs_more when actually
|
||||||
|
discarding the current descriptor.
|
||||||
|
|
||||||
|
Also, fix a detection of broken descriptors:
|
||||||
|
First the code checks if the current descriptor is not done.
|
||||||
|
Then it checks if the next descriptor is done.
|
||||||
|
Add a check that afterwards checks the first descriptor again, because
|
||||||
|
it might have been completed in the mean time.
|
||||||
|
|
||||||
|
This fixes a regression introduced in
|
||||||
|
commit 723e711356b5a8a95728a890e254e8b0d47b55cf
|
||||||
|
"ath9k: fix handling of broken descriptors"
|
||||||
|
|
||||||
|
Cc: stable@vger.kernel.org
|
||||||
|
Reported-by: Marco André Dinis <marcoandredinis@gmail.com>
|
||||||
|
Signed-off-by: Felix Fietkau <nbd@openwrt.org>
|
||||||
|
|
||||||
|
commit 52a46300e782fe6994466523eb2b0b59091ea59f
|
||||||
|
Author: Felix Fietkau <nbd@openwrt.org>
|
||||||
|
Date: Mon Feb 24 11:43:50 2014 +0100
|
||||||
|
|
||||||
|
ath9k: reduce baseband hang detection false positive rate
|
||||||
|
|
||||||
|
Check if the baseband state remains stable, and add a small delay
|
||||||
|
between register reads.
|
||||||
|
|
||||||
|
Signed-off-by: Felix Fietkau <nbd@openwrt.org>
|
||||||
|
|
||||||
commit 118945bb12082e9d4edddc868d88143164e0f440
|
commit 118945bb12082e9d4edddc868d88143164e0f440
|
||||||
Author: Felix Fietkau <nbd@openwrt.org>
|
Author: Felix Fietkau <nbd@openwrt.org>
|
||||||
Date: Sat Feb 22 14:55:23 2014 +0100
|
Date: Sat Feb 22 14:55:23 2014 +0100
|
||||||
|
@ -1053,7 +1087,39 @@ Date: Thu Jan 23 20:06:34 2014 +0100
|
||||||
else
|
else
|
||||||
udelay(100);
|
udelay(100);
|
||||||
|
|
||||||
@@ -2051,9 +2051,8 @@ static bool ath9k_hw_set_power_awake(str
|
@@ -1534,7 +1534,7 @@ EXPORT_SYMBOL(ath9k_hw_check_nav);
|
||||||
|
bool ath9k_hw_check_alive(struct ath_hw *ah)
|
||||||
|
{
|
||||||
|
int count = 50;
|
||||||
|
- u32 reg;
|
||||||
|
+ u32 reg, last_val;
|
||||||
|
|
||||||
|
if (AR_SREV_9300(ah))
|
||||||
|
return !ath9k_hw_detect_mac_hang(ah);
|
||||||
|
@@ -1542,9 +1542,13 @@ bool ath9k_hw_check_alive(struct ath_hw
|
||||||
|
if (AR_SREV_9285_12_OR_LATER(ah))
|
||||||
|
return true;
|
||||||
|
|
||||||
|
+ last_val = REG_READ(ah, AR_OBS_BUS_1);
|
||||||
|
do {
|
||||||
|
reg = REG_READ(ah, AR_OBS_BUS_1);
|
||||||
|
+ if (reg != last_val)
|
||||||
|
+ return true;
|
||||||
|
|
||||||
|
+ last_val = reg;
|
||||||
|
if ((reg & 0x7E7FFFEF) == 0x00702400)
|
||||||
|
continue;
|
||||||
|
|
||||||
|
@@ -1556,6 +1560,8 @@ bool ath9k_hw_check_alive(struct ath_hw
|
||||||
|
default:
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
+
|
||||||
|
+ udelay(1);
|
||||||
|
} while (count-- > 0);
|
||||||
|
|
||||||
|
return false;
|
||||||
|
@@ -2051,9 +2057,8 @@ static bool ath9k_hw_set_power_awake(str
|
||||||
|
|
||||||
REG_SET_BIT(ah, AR_RTC_FORCE_WAKE,
|
REG_SET_BIT(ah, AR_RTC_FORCE_WAKE,
|
||||||
AR_RTC_FORCE_WAKE_EN);
|
AR_RTC_FORCE_WAKE_EN);
|
||||||
|
@ -3387,3 +3453,141 @@ Date: Thu Jan 23 20:06:34 2014 +0100
|
||||||
SURVEY_INFO_CHANNEL_TIME |
|
SURVEY_INFO_CHANNEL_TIME |
|
||||||
SURVEY_INFO_CHANNEL_TIME_BUSY |
|
SURVEY_INFO_CHANNEL_TIME_BUSY |
|
||||||
SURVEY_INFO_CHANNEL_TIME_RX |
|
SURVEY_INFO_CHANNEL_TIME_RX |
|
||||||
|
--- a/drivers/net/wireless/ath/ath9k/recv.c
|
||||||
|
+++ b/drivers/net/wireless/ath/ath9k/recv.c
|
||||||
|
@@ -732,11 +732,18 @@ static struct ath_rxbuf *ath_get_next_rx
|
||||||
|
return NULL;
|
||||||
|
|
||||||
|
/*
|
||||||
|
- * mark descriptor as zero-length and set the 'more'
|
||||||
|
- * flag to ensure that both buffers get discarded
|
||||||
|
+ * Re-check previous descriptor, in case it has been filled
|
||||||
|
+ * in the mean time.
|
||||||
|
*/
|
||||||
|
- rs->rs_datalen = 0;
|
||||||
|
- rs->rs_more = true;
|
||||||
|
+ ret = ath9k_hw_rxprocdesc(ah, ds, rs);
|
||||||
|
+ if (ret == -EINPROGRESS) {
|
||||||
|
+ /*
|
||||||
|
+ * mark descriptor as zero-length and set the 'more'
|
||||||
|
+ * flag to ensure that both buffers get discarded
|
||||||
|
+ */
|
||||||
|
+ rs->rs_datalen = 0;
|
||||||
|
+ rs->rs_more = true;
|
||||||
|
+ }
|
||||||
|
}
|
||||||
|
|
||||||
|
list_del(&bf->list);
|
||||||
|
@@ -985,32 +992,32 @@ static int ath9k_rx_skb_preprocess(struc
|
||||||
|
struct ath_common *common = ath9k_hw_common(ah);
|
||||||
|
struct ieee80211_hdr *hdr;
|
||||||
|
bool discard_current = sc->rx.discard_next;
|
||||||
|
- int ret = 0;
|
||||||
|
|
||||||
|
/*
|
||||||
|
* Discard corrupt descriptors which are marked in
|
||||||
|
* ath_get_next_rx_buf().
|
||||||
|
*/
|
||||||
|
- sc->rx.discard_next = rx_stats->rs_more;
|
||||||
|
if (discard_current)
|
||||||
|
- return -EINVAL;
|
||||||
|
+ goto corrupt;
|
||||||
|
+
|
||||||
|
+ sc->rx.discard_next = false;
|
||||||
|
|
||||||
|
/*
|
||||||
|
* Discard zero-length packets.
|
||||||
|
*/
|
||||||
|
if (!rx_stats->rs_datalen) {
|
||||||
|
RX_STAT_INC(rx_len_err);
|
||||||
|
- return -EINVAL;
|
||||||
|
+ goto corrupt;
|
||||||
|
}
|
||||||
|
|
||||||
|
- /*
|
||||||
|
- * rs_status follows rs_datalen so if rs_datalen is too large
|
||||||
|
- * we can take a hint that hardware corrupted it, so ignore
|
||||||
|
- * those frames.
|
||||||
|
- */
|
||||||
|
+ /*
|
||||||
|
+ * rs_status follows rs_datalen so if rs_datalen is too large
|
||||||
|
+ * we can take a hint that hardware corrupted it, so ignore
|
||||||
|
+ * those frames.
|
||||||
|
+ */
|
||||||
|
if (rx_stats->rs_datalen > (common->rx_bufsize - ah->caps.rx_status_len)) {
|
||||||
|
RX_STAT_INC(rx_len_err);
|
||||||
|
- return -EINVAL;
|
||||||
|
+ goto corrupt;
|
||||||
|
}
|
||||||
|
|
||||||
|
/* Only use status info from the last fragment */
|
||||||
|
@@ -1024,10 +1031,8 @@ static int ath9k_rx_skb_preprocess(struc
|
||||||
|
* This is different from the other corrupt descriptor
|
||||||
|
* condition handled above.
|
||||||
|
*/
|
||||||
|
- if (rx_stats->rs_status & ATH9K_RXERR_CORRUPT_DESC) {
|
||||||
|
- ret = -EINVAL;
|
||||||
|
- goto exit;
|
||||||
|
- }
|
||||||
|
+ if (rx_stats->rs_status & ATH9K_RXERR_CORRUPT_DESC)
|
||||||
|
+ goto corrupt;
|
||||||
|
|
||||||
|
hdr = (struct ieee80211_hdr *) (skb->data + ah->caps.rx_status_len);
|
||||||
|
|
||||||
|
@@ -1043,18 +1048,15 @@ static int ath9k_rx_skb_preprocess(struc
|
||||||
|
if (ath_process_fft(sc, hdr, rx_stats, rx_status->mactime))
|
||||||
|
RX_STAT_INC(rx_spectral);
|
||||||
|
|
||||||
|
- ret = -EINVAL;
|
||||||
|
- goto exit;
|
||||||
|
+ return -EINVAL;
|
||||||
|
}
|
||||||
|
|
||||||
|
/*
|
||||||
|
* everything but the rate is checked here, the rate check is done
|
||||||
|
* separately to avoid doing two lookups for a rate for each frame.
|
||||||
|
*/
|
||||||
|
- if (!ath9k_rx_accept(common, hdr, rx_status, rx_stats, decrypt_error)) {
|
||||||
|
- ret = -EINVAL;
|
||||||
|
- goto exit;
|
||||||
|
- }
|
||||||
|
+ if (!ath9k_rx_accept(common, hdr, rx_status, rx_stats, decrypt_error))
|
||||||
|
+ return -EINVAL;
|
||||||
|
|
||||||
|
if (ath_is_mybeacon(common, hdr)) {
|
||||||
|
RX_STAT_INC(rx_beacons);
|
||||||
|
@@ -1064,15 +1066,11 @@ static int ath9k_rx_skb_preprocess(struc
|
||||||
|
/*
|
||||||
|
* This shouldn't happen, but have a safety check anyway.
|
||||||
|
*/
|
||||||
|
- if (WARN_ON(!ah->curchan)) {
|
||||||
|
- ret = -EINVAL;
|
||||||
|
- goto exit;
|
||||||
|
- }
|
||||||
|
+ if (WARN_ON(!ah->curchan))
|
||||||
|
+ return -EINVAL;
|
||||||
|
|
||||||
|
- if (ath9k_process_rate(common, hw, rx_stats, rx_status)) {
|
||||||
|
- ret =-EINVAL;
|
||||||
|
- goto exit;
|
||||||
|
- }
|
||||||
|
+ if (ath9k_process_rate(common, hw, rx_stats, rx_status))
|
||||||
|
+ return -EINVAL;
|
||||||
|
|
||||||
|
ath9k_process_rssi(common, hw, rx_stats, rx_status);
|
||||||
|
|
||||||
|
@@ -1087,9 +1085,11 @@ static int ath9k_rx_skb_preprocess(struc
|
||||||
|
sc->rx.num_pkts++;
|
||||||
|
#endif
|
||||||
|
|
||||||
|
-exit:
|
||||||
|
- sc->rx.discard_next = false;
|
||||||
|
- return ret;
|
||||||
|
+ return 0;
|
||||||
|
+
|
||||||
|
+corrupt:
|
||||||
|
+ sc->rx.discard_next = rx_stats->rs_more;
|
||||||
|
+ return -EINVAL;
|
||||||
|
}
|
||||||
|
|
||||||
|
static void ath9k_rx_skb_postprocess(struct ath_common *common,
|
||||||
|
|
|
@ -10,7 +10,7 @@
|
||||||
|
|
||||||
--- a/drivers/net/wireless/ath/ath9k/hw.c
|
--- a/drivers/net/wireless/ath/ath9k/hw.c
|
||||||
+++ b/drivers/net/wireless/ath/ath9k/hw.c
|
+++ b/drivers/net/wireless/ath/ath9k/hw.c
|
||||||
@@ -2716,7 +2716,7 @@ void ath9k_hw_apply_txpower(struct ath_h
|
@@ -2722,7 +2722,7 @@ void ath9k_hw_apply_txpower(struct ath_h
|
||||||
channel = chan->chan;
|
channel = chan->chan;
|
||||||
chan_pwr = min_t(int, channel->max_power * 2, MAX_RATE_POWER);
|
chan_pwr = min_t(int, channel->max_power * 2, MAX_RATE_POWER);
|
||||||
new_pwr = min_t(int, chan_pwr, reg->power_limit);
|
new_pwr = min_t(int, chan_pwr, reg->power_limit);
|
||||||
|
|
|
@ -94,7 +94,7 @@
|
||||||
struct ath_gen_timer *ath_gen_timer_alloc(struct ath_hw *ah,
|
struct ath_gen_timer *ath_gen_timer_alloc(struct ath_hw *ah,
|
||||||
--- a/drivers/net/wireless/ath/ath9k/hw.c
|
--- a/drivers/net/wireless/ath/ath9k/hw.c
|
||||||
+++ b/drivers/net/wireless/ath/ath9k/hw.c
|
+++ b/drivers/net/wireless/ath/ath9k/hw.c
|
||||||
@@ -1730,6 +1730,20 @@ fail:
|
@@ -1736,6 +1736,20 @@ fail:
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -115,7 +115,7 @@
|
||||||
int ath9k_hw_reset(struct ath_hw *ah, struct ath9k_channel *chan,
|
int ath9k_hw_reset(struct ath_hw *ah, struct ath9k_channel *chan,
|
||||||
struct ath9k_hw_cal_data *caldata, bool fastcc)
|
struct ath9k_hw_cal_data *caldata, bool fastcc)
|
||||||
{
|
{
|
||||||
@@ -1935,6 +1949,7 @@ int ath9k_hw_reset(struct ath_hw *ah, st
|
@@ -1941,6 +1955,7 @@ int ath9k_hw_reset(struct ath_hw *ah, st
|
||||||
ar9003_hw_disable_phy_restart(ah);
|
ar9003_hw_disable_phy_restart(ah);
|
||||||
|
|
||||||
ath9k_hw_apply_gpio_override(ah);
|
ath9k_hw_apply_gpio_override(ah);
|
||||||
|
|
|
@ -11,7 +11,7 @@
|
||||||
int (*external_reset)(void);
|
int (*external_reset)(void);
|
||||||
--- a/drivers/net/wireless/ath/ath9k/hw.c
|
--- a/drivers/net/wireless/ath/ath9k/hw.c
|
||||||
+++ b/drivers/net/wireless/ath/ath9k/hw.c
|
+++ b/drivers/net/wireless/ath/ath9k/hw.c
|
||||||
@@ -2323,17 +2323,25 @@ int ath9k_hw_fill_cap_info(struct ath_hw
|
@@ -2329,17 +2329,25 @@ int ath9k_hw_fill_cap_info(struct ath_hw
|
||||||
}
|
}
|
||||||
|
|
||||||
eeval = ah->eep_ops->get_eeprom(ah, EEP_OP_MODE);
|
eeval = ah->eep_ops->get_eeprom(ah, EEP_OP_MODE);
|
||||||
|
|
|
@ -40,7 +40,7 @@
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -1725,8 +1744,14 @@ static int ath9k_hw_do_fastcc(struct ath
|
@@ -1731,8 +1750,14 @@ static int ath9k_hw_do_fastcc(struct ath
|
||||||
if (AR_SREV_9271(ah))
|
if (AR_SREV_9271(ah))
|
||||||
ar9002_hw_load_ani_reg(ah, chan);
|
ar9002_hw_load_ani_reg(ah, chan);
|
||||||
|
|
||||||
|
@ -55,7 +55,7 @@
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -1954,6 +1979,9 @@ int ath9k_hw_reset(struct ath_hw *ah, st
|
@@ -1960,6 +1985,9 @@ int ath9k_hw_reset(struct ath_hw *ah, st
|
||||||
if (AR_SREV_9565(ah) && common->bt_ant_diversity)
|
if (AR_SREV_9565(ah) && common->bt_ant_diversity)
|
||||||
REG_SET_BIT(ah, AR_BTCOEX_WL_LNADIV, AR_BTCOEX_WL_LNADIV_FORCE_ON);
|
REG_SET_BIT(ah, AR_BTCOEX_WL_LNADIV, AR_BTCOEX_WL_LNADIV_FORCE_ON);
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue