ath9k_hw: Fix exceed transmission burst-time of 5GHz
authorRajkumar Manoharan <rmanohar@qca.qualcomm.com>
Sat, 13 Aug 2011 04:58:08 +0000 (10:28 +0530)
committerJohn W. Linville <linville@tuxdriver.com>
Wed, 24 Aug 2011 17:59:44 +0000 (13:59 -0400)
The WAR which adds extra delimiters when using RTS/CTS
with aggregation and non-enterprise AR9003 chips.
This extra padding is done after doing all the 4ms limit
checks and hence the total aggregate sizes are exceeding
the allowed duration. This patch limits the aggregate
sizes appropriately after including these extra delimiters.

Signed-off-by: Rajkumar Manoharan <rmanohar@qca.qualcomm.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ath/ath9k/ar9003_mac.c
drivers/net/wireless/ath/ath9k/xmit.c

index 1aadc4757e672a1cf9b93eb502c4b6a527c0fb76..81ccce1faff538d2e32d6a0743dd16dfbd028d27 100644 (file)
@@ -415,36 +415,12 @@ static void ar9003_hw_set11n_ratescenario(struct ath_hw *ah, void *ds,
 static void ar9003_hw_set11n_aggr_first(struct ath_hw *ah, void *ds,
                                        u32 aggrLen)
 {
-#define FIRST_DESC_NDELIMS 60
        struct ar9003_txc *ads = (struct ar9003_txc *) ds;
 
        ads->ctl12 |= (AR_IsAggr | AR_MoreAggr);
 
-       if (ah->ent_mode & AR_ENT_OTP_MPSD) {
-               u32 ctl17, ndelim;
-               /*
-                * Add delimiter when using RTS/CTS with aggregation
-                * and non enterprise AR9003 card
-                */
-               ctl17 = ads->ctl17;
-               ndelim = MS(ctl17, AR_PadDelim);
-
-               if (ndelim < FIRST_DESC_NDELIMS) {
-                       aggrLen += (FIRST_DESC_NDELIMS - ndelim) * 4;
-                       ndelim = FIRST_DESC_NDELIMS;
-               }
-
-               ctl17 &= ~AR_AggrLen;
-               ctl17 |= SM(aggrLen, AR_AggrLen);
-
-               ctl17 &= ~AR_PadDelim;
-               ctl17 |= SM(ndelim, AR_PadDelim);
-
-               ads->ctl17 = ctl17;
-       } else {
-               ads->ctl17 &= ~AR_AggrLen;
-               ads->ctl17 |= SM(aggrLen, AR_AggrLen);
-       }
+       ads->ctl17 &= ~AR_AggrLen;
+       ads->ctl17 |= SM(aggrLen, AR_AggrLen);
 }
 
 static void ar9003_hw_set11n_aggr_middle(struct ath_hw *ah, void *ds,
index e1d1e903229b69c994ecc881d0632542b56d6ad5..feef0135515c0e0fa3ef4e14790816462ba7cd98 100644 (file)
@@ -644,8 +644,10 @@ static u32 ath_lookup_rate(struct ath_softc *sc, struct ath_buf *bf,
  * meet the minimum required mpdudensity.
  */
 static int ath_compute_num_delims(struct ath_softc *sc, struct ath_atx_tid *tid,
-                                 struct ath_buf *bf, u16 frmlen)
+                                 struct ath_buf *bf, u16 frmlen,
+                                 bool first_subfrm)
 {
+#define FIRST_DESC_NDELIMS 60
        struct sk_buff *skb = bf->bf_mpdu;
        struct ieee80211_tx_info *tx_info = IEEE80211_SKB_CB(skb);
        u32 nsymbits, nsymbols;
@@ -667,6 +669,13 @@ static int ath_compute_num_delims(struct ath_softc *sc, struct ath_atx_tid *tid,
            !(sc->sc_ah->caps.hw_caps & ATH9K_HW_CAP_EDMA))
                ndelim += ATH_AGGR_ENCRYPTDELIM;
 
+       /*
+        * Add delimiter when using RTS/CTS with aggregation
+        * and non enterprise AR9003 card
+        */
+       if (first_subfrm)
+               ndelim = max(ndelim, FIRST_DESC_NDELIMS);
+
        /*
         * Convert desired mpdu density from microeconds to bytes based
         * on highest rate in rate series (i.e. first rate) to determine
@@ -756,7 +765,6 @@ static enum ATH_AGGR_STATUS ath_tx_form_aggr(struct ath_softc *sc,
                        status = ATH_AGGR_LIMITED;
                        break;
                }
-               nframes++;
 
                /* add padding for previous frame to aggregation length */
                al += bpad + al_delta;
@@ -765,9 +773,11 @@ static enum ATH_AGGR_STATUS ath_tx_form_aggr(struct ath_softc *sc,
                 * Get the delimiters needed to meet the MPDU
                 * density for this node.
                 */
-               ndelim = ath_compute_num_delims(sc, tid, bf_first, fi->framelen);
+               ndelim = ath_compute_num_delims(sc, tid, bf_first, fi->framelen,
+                                               !nframes);
                bpad = PADBYTES(al_delta) + (ndelim << 2);
 
+               nframes++;
                bf->bf_next = NULL;
                ath9k_hw_set_desc_link(sc->sc_ah, bf->bf_desc, 0);