Changeset 39751


Ignore:
Timestamp:
2014-02-25T11:27:02+01:00 (4 years ago)
Author:
nbd
Message:

ath9k: add fixes for rx processing and baseband hang detection

Signed-off-by: Felix Fietkau <nbd@…>

Location:
trunk/package/kernel/mac80211/patches
Files:
5 edited

Legend:

Unmodified
Added
Removed
  • trunk/package/kernel/mac80211/patches/300-pending_work.patch

    r39691 r39751  
     1commit 98d1a6c5b14688ed030e81b889f607be308e0df9 
     2Author: Felix Fietkau <nbd@openwrt.org> 
     3Date:   Mon Feb 24 22:20:32 2014 +0100 
     4 
     5    ath9k: fix invalid descriptor discarding 
     6     
     7    Only set sc->rx.discard_next to rx_stats->rs_more when actually 
     8    discarding the current descriptor. 
     9     
     10    Also, fix a detection of broken descriptors: 
     11    First the code checks if the current descriptor is not done. 
     12    Then it checks if the next descriptor is done. 
     13    Add a check that afterwards checks the first descriptor again, because 
     14    it might have been completed in the mean time. 
     15     
     16    This fixes a regression introduced in 
     17    commit 723e711356b5a8a95728a890e254e8b0d47b55cf 
     18    "ath9k: fix handling of broken descriptors" 
     19     
     20    Cc: stable@vger.kernel.org 
     21    Reported-by: Marco André Dinis <marcoandredinis@gmail.com> 
     22    Signed-off-by: Felix Fietkau <nbd@openwrt.org> 
     23 
     24commit 52a46300e782fe6994466523eb2b0b59091ea59f 
     25Author: Felix Fietkau <nbd@openwrt.org> 
     26Date:   Mon Feb 24 11:43:50 2014 +0100 
     27 
     28    ath9k: reduce baseband hang detection false positive rate 
     29     
     30    Check if the baseband state remains stable, and add a small delay 
     31    between register reads. 
     32     
     33    Signed-off-by: Felix Fietkau <nbd@openwrt.org> 
     34 
    135commit 118945bb12082e9d4edddc868d88143164e0f440 
    236Author: Felix Fietkau <nbd@openwrt.org> 
     
    10541088                udelay(100); 
    10551089  
    1056 @@ -2051,9 +2051,8 @@ static bool ath9k_hw_set_power_awake(str 
     1090@@ -1534,7 +1534,7 @@ EXPORT_SYMBOL(ath9k_hw_check_nav); 
     1091 bool ath9k_hw_check_alive(struct ath_hw *ah) 
     1092 { 
     1093        int count = 50; 
     1094-       u32 reg; 
     1095+       u32 reg, last_val; 
     1096  
     1097        if (AR_SREV_9300(ah)) 
     1098                return !ath9k_hw_detect_mac_hang(ah); 
     1099@@ -1542,9 +1542,13 @@ bool ath9k_hw_check_alive(struct ath_hw  
     1100        if (AR_SREV_9285_12_OR_LATER(ah)) 
     1101                return true; 
     1102  
     1103+       last_val = REG_READ(ah, AR_OBS_BUS_1); 
     1104        do { 
     1105                reg = REG_READ(ah, AR_OBS_BUS_1); 
     1106+               if (reg != last_val) 
     1107+                       return true; 
     1108  
     1109+               last_val = reg; 
     1110                if ((reg & 0x7E7FFFEF) == 0x00702400) 
     1111                        continue; 
     1112  
     1113@@ -1556,6 +1560,8 @@ bool ath9k_hw_check_alive(struct ath_hw  
     1114                default: 
     1115                        return true; 
     1116                } 
     1117+ 
     1118+               udelay(1); 
     1119        } while (count-- > 0); 
     1120  
     1121        return false; 
     1122@@ -2051,9 +2057,8 @@ static bool ath9k_hw_set_power_awake(str 
    10571123  
    10581124        REG_SET_BIT(ah, AR_RTC_FORCE_WAKE, 
     
    33883454                        SURVEY_INFO_CHANNEL_TIME_BUSY | 
    33893455                        SURVEY_INFO_CHANNEL_TIME_RX | 
     3456--- a/drivers/net/wireless/ath/ath9k/recv.c 
     3457+++ b/drivers/net/wireless/ath/ath9k/recv.c 
     3458@@ -732,11 +732,18 @@ static struct ath_rxbuf *ath_get_next_rx 
     3459                        return NULL; 
     3460  
     3461                /* 
     3462-                * mark descriptor as zero-length and set the 'more' 
     3463-                * flag to ensure that both buffers get discarded 
     3464+                * Re-check previous descriptor, in case it has been filled 
     3465+                * in the mean time. 
     3466                 */ 
     3467-               rs->rs_datalen = 0; 
     3468-               rs->rs_more = true; 
     3469+               ret = ath9k_hw_rxprocdesc(ah, ds, rs); 
     3470+               if (ret == -EINPROGRESS) { 
     3471+                       /* 
     3472+                        * mark descriptor as zero-length and set the 'more' 
     3473+                        * flag to ensure that both buffers get discarded 
     3474+                        */ 
     3475+                       rs->rs_datalen = 0; 
     3476+                       rs->rs_more = true; 
     3477+               } 
     3478        } 
     3479  
     3480        list_del(&bf->list); 
     3481@@ -985,32 +992,32 @@ static int ath9k_rx_skb_preprocess(struc 
     3482        struct ath_common *common = ath9k_hw_common(ah); 
     3483        struct ieee80211_hdr *hdr; 
     3484        bool discard_current = sc->rx.discard_next; 
     3485-       int ret = 0; 
     3486  
     3487        /* 
     3488         * Discard corrupt descriptors which are marked in 
     3489         * ath_get_next_rx_buf(). 
     3490         */ 
     3491-       sc->rx.discard_next = rx_stats->rs_more; 
     3492        if (discard_current) 
     3493-               return -EINVAL; 
     3494+               goto corrupt; 
     3495+ 
     3496+       sc->rx.discard_next = false; 
     3497  
     3498        /* 
     3499         * Discard zero-length packets. 
     3500         */ 
     3501        if (!rx_stats->rs_datalen) { 
     3502                RX_STAT_INC(rx_len_err); 
     3503-               return -EINVAL; 
     3504+               goto corrupt; 
     3505        } 
     3506  
     3507-        /* 
     3508-         * rs_status follows rs_datalen so if rs_datalen is too large 
     3509-         * we can take a hint that hardware corrupted it, so ignore 
     3510-         * those frames. 
     3511-         */ 
     3512+       /* 
     3513+        * rs_status follows rs_datalen so if rs_datalen is too large 
     3514+        * we can take a hint that hardware corrupted it, so ignore 
     3515+        * those frames. 
     3516+        */ 
     3517        if (rx_stats->rs_datalen > (common->rx_bufsize - ah->caps.rx_status_len)) { 
     3518                RX_STAT_INC(rx_len_err); 
     3519-               return -EINVAL; 
     3520+               goto corrupt; 
     3521        } 
     3522  
     3523        /* Only use status info from the last fragment */ 
     3524@@ -1024,10 +1031,8 @@ static int ath9k_rx_skb_preprocess(struc 
     3525         * This is different from the other corrupt descriptor 
     3526         * condition handled above. 
     3527         */ 
     3528-       if (rx_stats->rs_status & ATH9K_RXERR_CORRUPT_DESC) { 
     3529-               ret = -EINVAL; 
     3530-               goto exit; 
     3531-       } 
     3532+       if (rx_stats->rs_status & ATH9K_RXERR_CORRUPT_DESC) 
     3533+               goto corrupt; 
     3534  
     3535        hdr = (struct ieee80211_hdr *) (skb->data + ah->caps.rx_status_len); 
     3536  
     3537@@ -1043,18 +1048,15 @@ static int ath9k_rx_skb_preprocess(struc 
     3538                if (ath_process_fft(sc, hdr, rx_stats, rx_status->mactime)) 
     3539                        RX_STAT_INC(rx_spectral); 
     3540  
     3541-               ret = -EINVAL; 
     3542-               goto exit; 
     3543+               return -EINVAL; 
     3544        } 
     3545  
     3546        /* 
     3547         * everything but the rate is checked here, the rate check is done 
     3548         * separately to avoid doing two lookups for a rate for each frame. 
     3549         */ 
     3550-       if (!ath9k_rx_accept(common, hdr, rx_status, rx_stats, decrypt_error)) { 
     3551-               ret = -EINVAL; 
     3552-               goto exit; 
     3553-       } 
     3554+       if (!ath9k_rx_accept(common, hdr, rx_status, rx_stats, decrypt_error)) 
     3555+               return -EINVAL; 
     3556  
     3557        if (ath_is_mybeacon(common, hdr)) { 
     3558                RX_STAT_INC(rx_beacons); 
     3559@@ -1064,15 +1066,11 @@ static int ath9k_rx_skb_preprocess(struc 
     3560        /* 
     3561         * This shouldn't happen, but have a safety check anyway. 
     3562         */ 
     3563-       if (WARN_ON(!ah->curchan)) { 
     3564-               ret = -EINVAL; 
     3565-               goto exit; 
     3566-       } 
     3567+       if (WARN_ON(!ah->curchan)) 
     3568+               return -EINVAL; 
     3569  
     3570-       if (ath9k_process_rate(common, hw, rx_stats, rx_status)) { 
     3571-               ret =-EINVAL; 
     3572-               goto exit; 
     3573-       } 
     3574+       if (ath9k_process_rate(common, hw, rx_stats, rx_status)) 
     3575+               return -EINVAL; 
     3576  
     3577        ath9k_process_rssi(common, hw, rx_stats, rx_status); 
     3578  
     3579@@ -1087,9 +1085,11 @@ static int ath9k_rx_skb_preprocess(struc 
     3580                sc->rx.num_pkts++; 
     3581 #endif 
     3582  
     3583-exit: 
     3584-       sc->rx.discard_next = false; 
     3585-       return ret; 
     3586+       return 0; 
     3587+ 
     3588+corrupt: 
     3589+       sc->rx.discard_next = rx_stats->rs_more; 
     3590+       return -EINVAL; 
     3591 } 
     3592  
     3593 static void ath9k_rx_skb_postprocess(struct ath_common *common, 
  • trunk/package/kernel/mac80211/patches/523-ath9k_use_configured_antenna_gain.patch

    r39427 r39751  
    1111--- a/drivers/net/wireless/ath/ath9k/hw.c 
    1212+++ b/drivers/net/wireless/ath/ath9k/hw.c 
    13 @@ -2716,7 +2716,7 @@ void ath9k_hw_apply_txpower(struct ath_h 
     13@@ -2722,7 +2722,7 @@ void ath9k_hw_apply_txpower(struct ath_h 
    1414        channel = chan->chan; 
    1515        chan_pwr = min_t(int, channel->max_power * 2, MAX_RATE_POWER); 
  • trunk/package/kernel/mac80211/patches/542-ath9k_debugfs_diag.patch

    r39687 r39751  
    9595--- a/drivers/net/wireless/ath/ath9k/hw.c 
    9696+++ b/drivers/net/wireless/ath/ath9k/hw.c 
    97 @@ -1730,6 +1730,20 @@ fail: 
     97@@ -1736,6 +1736,20 @@ fail: 
    9898        return -EINVAL; 
    9999 } 
     
    116116                   struct ath9k_hw_cal_data *caldata, bool fastcc) 
    117117 { 
    118 @@ -1935,6 +1949,7 @@ int ath9k_hw_reset(struct ath_hw *ah, st 
     118@@ -1941,6 +1955,7 @@ int ath9k_hw_reset(struct ath_hw *ah, st 
    119119                ar9003_hw_disable_phy_restart(ah); 
    120120  
  • trunk/package/kernel/mac80211/patches/543-ath9k-allow-to-disable-bands-via-platform-data.patch

    r39427 r39751  
    1212--- a/drivers/net/wireless/ath/ath9k/hw.c 
    1313+++ b/drivers/net/wireless/ath/ath9k/hw.c 
    14 @@ -2323,17 +2323,25 @@ int ath9k_hw_fill_cap_info(struct ath_hw 
     14@@ -2329,17 +2329,25 @@ int ath9k_hw_fill_cap_info(struct ath_hw 
    1515        } 
    1616  
  • trunk/package/kernel/mac80211/patches/566-ath9k-ar933x-usb-hang-workaround.patch

    r39427 r39751  
    4141 } 
    4242  
    43 @@ -1725,8 +1744,14 @@ static int ath9k_hw_do_fastcc(struct ath 
     43@@ -1731,8 +1750,14 @@ static int ath9k_hw_do_fastcc(struct ath 
    4444        if (AR_SREV_9271(ah)) 
    4545                ar9002_hw_load_ani_reg(ah, chan); 
     
    5656 } 
    5757  
    58 @@ -1954,6 +1979,9 @@ int ath9k_hw_reset(struct ath_hw *ah, st 
     58@@ -1960,6 +1985,9 @@ int ath9k_hw_reset(struct ath_hw *ah, st 
    5959        if (AR_SREV_9565(ah) && common->bt_ant_diversity) 
    6060                REG_SET_BIT(ah, AR_BTCOEX_WL_LNADIV, AR_BTCOEX_WL_LNADIV_FORCE_ON); 
Note: See TracChangeset for help on using the changeset viewer.