madwifi: clean up tx status mrr handling and fix mrr parsing for 5416

SVN-Revision: 15757
This commit is contained in:
Felix Fietkau 2009-05-10 19:05:15 +00:00
parent f0eecf38aa
commit 09f06bbfb0

View File

@ -0,0 +1,507 @@
--- a/net80211/ieee80211_rate.h
+++ b/net80211/ieee80211_rate.h
@@ -81,6 +81,8 @@ struct ieee80211vap;
/* Multi-rare retry: 3 additional rate/retry pairs */
struct ieee80211_mrr {
+ int rate0;
+ int retries0;
int rate1;
int retries1;
int rate2;
@@ -142,7 +144,7 @@ struct ieee80211_rate_ops {
* for packets that were successfully sent and for those that
* failed (consult the descriptor for details). */
void (*tx_complete)(struct ath_softc *sc, struct ath_node *an,
- const struct ath_buf *bf);
+ const struct ath_buf *bf, const struct ieee80211_mrr *mrr);
};
struct ath_ratectrl {
--- a/ath/if_ath.c
+++ b/ath/if_ath.c
@@ -8634,6 +8634,8 @@ ath_tx_processq(struct ath_softc *sc, st
ni = bf->bf_node;
if (ni != NULL) {
+ struct ieee80211_mrr mrr;
+
an = ATH_NODE(ni);
if (ts->ts_status == 0) {
u_int8_t txant = ts->ts_antenna;
@@ -8686,15 +8688,43 @@ ath_tx_processq(struct ath_softc *sc, st
lr = ts->ts_longretry;
sc->sc_stats.ast_tx_shortretry += sr;
sc->sc_stats.ast_tx_longretry += lr;
+ memset(&mrr, 0, sizeof(mrr));
+
+ switch(ah->ah_macType) {
+ case 5210:
+ case 5211:
+ goto skip_mrr;
+
+ case 5212:
+ mrr.rate0 = sc->sc_hwmap[MS(ds->ds_ctl3, AR_XmitRate0)].ieeerate;
+ mrr.rate1 = sc->sc_hwmap[MS(ds->ds_ctl3, AR_XmitRate1)].ieeerate;
+ mrr.rate2 = sc->sc_hwmap[MS(ds->ds_ctl3, AR_XmitRate2)].ieeerate;
+ mrr.rate3 = sc->sc_hwmap[MS(ds->ds_ctl3, AR_XmitRate3)].ieeerate;
+ break;
+
+ case 5416:
+ mrr.rate0 = sc->sc_hwmap[MS(ds->ds_ctl3, AR5416_XmitRate0)].ieeerate;
+ mrr.rate1 = sc->sc_hwmap[MS(ds->ds_ctl3, AR5416_XmitRate1)].ieeerate;
+ mrr.rate2 = sc->sc_hwmap[MS(ds->ds_ctl3, AR5416_XmitRate2)].ieeerate;
+ mrr.rate3 = sc->sc_hwmap[MS(ds->ds_ctl3, AR5416_XmitRate3)].ieeerate;
+ break;
+ }
+
+ mrr.retries0 = MS(ds->ds_ctl2, AR_XmitDataTries0);
+ mrr.retries1 = MS(ds->ds_ctl2, AR_XmitDataTries1);
+ mrr.retries2 = MS(ds->ds_ctl2, AR_XmitDataTries2);
+ mrr.retries3 = MS(ds->ds_ctl2, AR_XmitDataTries3);
+
/*
* Hand the descriptor to the rate control algorithm
* if the frame wasn't dropped for filtering or sent
* w/o waiting for an ack. In those cases the rssi
* and retry counts will be meaningless.
*/
+skip_mrr:
if ((ts->ts_status & HAL_TXERR_FILT) == 0 &&
(bf->bf_flags & HAL_TXDESC_NOACK) == 0)
- sc->sc_rc->ops->tx_complete(sc, an, bf);
+ sc->sc_rc->ops->tx_complete(sc, an, bf, &mrr);
}
bus_unmap_single(sc->sc_bdev, bf->bf_skbaddr,
--- a/ath/if_athvar.h
+++ b/ath/if_athvar.h
@@ -595,6 +595,46 @@ struct ath_vap {
(_tqs)->axq_link = NULL; \
} while (0)
+/*
+ * Definitions for pulling the rate and trie counts from
+ * a 5212 h/w descriptor. These Don't belong here; the
+ * driver should record this information so the rate control
+ * code doesn't go groveling around in the descriptor bits.
+ */
+#define ds_ctl2 ds_hw[0]
+#define ds_ctl3 ds_hw[1]
+
+/* TX ds_ctl3 */
+#define AR_XmitDataTries0 0x000f0000 /* series 0 max attempts */
+#define AR_XmitDataTries0_S 16
+#define AR_XmitDataTries1 0x00f00000 /* series 1 max attempts */
+#define AR_XmitDataTries1_S 20
+#define AR_XmitDataTries2 0x0f000000 /* series 2 max attempts */
+#define AR_XmitDataTries2_S 24
+#define AR_XmitDataTries3 0xf0000000 /* series 3 max attempts */
+#define AR_XmitDataTries3_S 28
+
+/* TX ds_ctl3 */
+#define AR_XmitRate0 0x0000001f /* series 0 tx rate */
+#define AR_XmitRate0_S 0
+#define AR_XmitRate1 0x000003e0 /* series 1 tx rate */
+#define AR_XmitRate1_S 5
+#define AR_XmitRate2 0x00007c00 /* series 2 tx rate */
+#define AR_XmitRate2_S 10
+#define AR_XmitRate3 0x000f8000 /* series 3 tx rate */
+#define AR_XmitRate3_S 15
+
+#define AR5416_XmitRate0 0x000000ff
+#define AR5416_XmitRate0_S 0
+#define AR5416_XmitRate1 0x0000ff00
+#define AR5416_XmitRate1_S 8
+#define AR5416_XmitRate2 0x00ff0000
+#define AR5416_XmitRate2_S 16
+#define AR5416_XmitRate3 0xff000000
+#define AR5416_XmitRate3_S 24
+
+#define MS(_v, _f) (((_v) & (_f)) >> _f##_S)
+
/*
* concat buffers from one queue to other
*/
--- a/ath_rate/amrr/amrr.c
+++ b/ath_rate/amrr/amrr.c
@@ -123,7 +123,8 @@ ath_rate_get_mrr(struct ath_softc *sc, s
static void
ath_rate_tx_complete(struct ath_softc *sc,
- struct ath_node *an, const struct ath_buf *bf)
+ struct ath_node *an, const struct ath_buf *bf,
+ const struct ieee80211_mrr *mrr)
{
struct amrr_node *amn = ATH_NODE_AMRR(an);
const struct ath_tx_status *ts = &bf->bf_dsstatus.ds_txstat;
--- a/ath_rate/minstrel/minstrel.c
+++ b/ath_rate/minstrel/minstrel.c
@@ -333,7 +333,8 @@ ath_rate_get_mrr(struct ath_softc *sc, s
static void
ath_rate_tx_complete(struct ath_softc *sc,
- struct ath_node *an, const struct ath_buf *bf)
+ struct ath_node *an, const struct ath_buf *bf,
+ const struct ieee80211_mrr *mrr)
{
struct minstrel_node *sn = ATH_NODE_MINSTREL(an);
struct ieee80211com *ic = &sc->sc_ic;
@@ -341,12 +342,9 @@ ath_rate_tx_complete(struct ath_softc *s
const struct ath_desc *ds = &bf->bf_desc[0];
int final_rate = 0;
int tries = 0;
- int mrr;
+ int use_mrr;
int final_ndx;
- int rate0, tries0, ndx0;
- int rate1, tries1, ndx1;
- int rate2, tries2, ndx2;
- int rate3, tries3, ndx3;
+ int ndx0, ndx1, ndx2, ndx3;
/* This is the index in the retry chain we finish at.
* With no retransmits, it is always 0.
@@ -376,9 +374,9 @@ ath_rate_tx_complete(struct ath_softc *s
if (!ts->ts_status) /* Success when sending a packet*/
sn->rs_ratesuccess[final_ndx]++;
- mrr = sc->sc_mrretry && !(ic->ic_flags & IEEE80211_F_USEPROT) && ENABLE_MRR;
+ use_mrr = sc->sc_mrretry && !(ic->ic_flags & IEEE80211_F_USEPROT) && ENABLE_MRR;
- if (!mrr) {
+ if (!use_mrr) {
if ((0 <= final_ndx) && (final_ndx < sn->num_rates)) {
sn->rs_rateattempts[final_ndx] += tries; /* only one rate was used */
}
@@ -388,47 +386,36 @@ ath_rate_tx_complete(struct ath_softc *s
/* Now, query the hal/hardware to find out the contents of the multirate retry chain.
* If we have it set to 6,3,2,2, this call will always return 6,3,2,2. For some packets, we can
* get a mrr of 0, -1, -1, -1, which indicates there is no chain installed for that packet */
- rate0 = sc->sc_hwmap[MS(ds->ds_ctl3, AR_XmitRate0)].ieeerate;
- tries0 = MS(ds->ds_ctl2, AR_XmitDataTries0);
- ndx0 = rate_to_ndx(sn, rate0);
+ ndx0 = rate_to_ndx(sn, mrr->rate0);
+ ndx1 = rate_to_ndx(sn, mrr->rate1);
+ ndx2 = rate_to_ndx(sn, mrr->rate2);
+ ndx3 = rate_to_ndx(sn, mrr->rate3);
- rate1 = sc->sc_hwmap[MS(ds->ds_ctl3, AR_XmitRate1)].ieeerate;
- tries1 = MS(ds->ds_ctl2, AR_XmitDataTries1);
- ndx1 = rate_to_ndx(sn, rate1);
-
- rate2 = sc->sc_hwmap[MS(ds->ds_ctl3, AR_XmitRate2)].ieeerate;
- tries2 = MS(ds->ds_ctl2, AR_XmitDataTries2);
- ndx2 = rate_to_ndx(sn, rate2);
-
- rate3 = sc->sc_hwmap[MS(ds->ds_ctl3, AR_XmitRate3)].ieeerate;
- tries3 = MS(ds->ds_ctl2, AR_XmitDataTries3);
- ndx3 = rate_to_ndx(sn, rate3);
-
- sn->rs_rateattempts[ndx0] += MIN(tries, tries0);
- if (tries <= tries0)
+ sn->rs_rateattempts[ndx0] += MIN(tries, mrr->retries0);
+ if (tries <= mrr->retries0)
return;
- if (tries1 < 0)
+ if (mrr->retries1 < 0)
return;
- tries = tries - tries0;
- sn->rs_rateattempts[ndx1] += MIN(tries, tries1);
- if (tries <= tries1)
+ tries = tries - mrr->retries0;
+ sn->rs_rateattempts[ndx1] += MIN(tries, mrr->retries1);
+ if (tries <= mrr->retries1)
return;
if (bf->rcflags)
sn->sample_count++;
- if (tries2 < 0)
+ if (mrr->retries2 < 0)
return;
- tries = tries - tries1;
- sn->rs_rateattempts[ndx2] += MIN(tries, tries2);
- if (tries <= tries2)
+ tries = tries - mrr->retries1;
+ sn->rs_rateattempts[ndx2] += MIN(tries, mrr->retries2);
+ if (tries <= mrr->retries2)
return;
- if (tries3 < 0)
+ if (mrr->retries3 < 0)
return;
- tries = tries - tries2;
- sn->rs_rateattempts[ndx3] += MIN(tries, tries3);
+ tries = tries - mrr->retries2;
+ sn->rs_rateattempts[ndx3] += MIN(tries, mrr->retries3);
}
static void
--- a/ath_rate/minstrel/minstrel.h
+++ b/ath_rate/minstrel/minstrel.h
@@ -171,36 +171,6 @@ struct minstrel_node {
#define ATH_NODE_MINSTREL(an) ((struct minstrel_node *)&an[1])
-/*
- * Definitions for pulling the rate and trie counts from
- * a 5212 h/w descriptor. These Don't belong here; the
- * driver should record this information so the rate control
- * code doesn't go groveling around in the descriptor bits.
- */
-#define ds_ctl2 ds_hw[0]
-#define ds_ctl3 ds_hw[1]
-
-/* TX ds_ctl3 */
-#define AR_XmitDataTries0 0x000f0000 /* series 0 max attempts */
-#define AR_XmitDataTries0_S 16
-#define AR_XmitDataTries1 0x00f00000 /* series 1 max attempts */
-#define AR_XmitDataTries1_S 20
-#define AR_XmitDataTries2 0x0f000000 /* series 2 max attempts */
-#define AR_XmitDataTries2_S 24
-#define AR_XmitDataTries3 0xf0000000 /* series 3 max attempts */
-#define AR_XmitDataTries3_S 28
-
-/* TX ds_ctl3 */
-#define AR_XmitRate0 0x0000001f /* series 0 tx rate */
-#define AR_XmitRate0_S 0
-#define AR_XmitRate1 0x000003e0 /* series 1 tx rate */
-#define AR_XmitRate1_S 5
-#define AR_XmitRate2 0x00007c00 /* series 2 tx rate */
-#define AR_XmitRate2_S 10
-#define AR_XmitRate3 0x000f8000 /* series 3 tx rate */
-#define AR_XmitRate3_S 15
-
-#define MS(_v, _f) (((_v) & (_f)) >> _f##_S)
#endif /* _DEV_ATH_RATE_MINSTEL_H */
/* The comment below is magic for those who use emacs to edit this file. */
--- a/ath_rate/onoe/onoe.c
+++ b/ath_rate/onoe/onoe.c
@@ -137,7 +137,8 @@ ath_rate_get_mrr(struct ath_softc *sc, s
static void
ath_rate_tx_complete(struct ath_softc *sc,
- struct ath_node *an, const struct ath_buf *bf)
+ struct ath_node *an, const struct ath_buf *bf,
+ const struct ieee80211_mrr *mrr)
{
struct onoe_node *on = ATH_NODE_ONOE(an);
const struct ath_tx_status *ts = &bf->bf_dsstatus.ds_txstat;
--- a/ath_rate/sample/sample.c
+++ b/ath_rate/sample/sample.c
@@ -178,10 +178,6 @@ static __inline int best_rate_ndx(struct
!sn->stats[size_bin][x].packets_acked))
continue;
- /* 9 megabits never works better than 12 */
- if (sn->rates[x].rate == 18)
- continue;
-
/* don't use a bit-rate that has been failing */
if (sn->stats[size_bin][x].successive_failures > 3)
continue;
@@ -234,10 +230,6 @@ pick_sample_ndx(struct sample_node *sn,
if (sn->rates[ndx].rate > 22 && ndx > current_ndx + 2)
continue;
- /* 9 megabits never works better than 12 */
- if (sn->rates[ndx].rate == 18)
- continue;
-
/* if we're using 11 megabits, only sample up to 12 megabits
*/
if (sn->rates[current_ndx].rate == 22 && ndx > current_ndx + 1)
@@ -531,7 +523,8 @@ update_stats(struct ath_softc *sc, struc
static void
ath_rate_tx_complete(struct ath_softc *sc,
- struct ath_node *an, const struct ath_buf *bf)
+ struct ath_node *an, const struct ath_buf *bf,
+ const struct ieee80211_mrr *mrr)
{
struct sample_node *sn = ATH_NODE_SAMPLE(an);
struct ieee80211com *ic = &sc->sc_ic;
@@ -541,7 +534,7 @@ ath_rate_tx_complete(struct ath_softc *s
unsigned int short_tries;
unsigned int long_tries;
unsigned int frame_size;
- unsigned int mrr;
+ unsigned int use_mrr;
final_rate = sc->sc_hwmap[ts->ts_rate &~ HAL_TXSTAT_ALTRATE].ieeerate;
short_tries = ts->ts_shortretry + 1;
@@ -557,7 +550,7 @@ ath_rate_tx_complete(struct ath_softc *s
return;
}
- mrr = sc->sc_mrretry && !(ic->ic_flags & IEEE80211_F_USEPROT) && ENABLE_MRR;
+ use_mrr = sc->sc_mrretry && !(ic->ic_flags & IEEE80211_F_USEPROT) && ENABLE_MRR;
if (sc->sc_mrretry && ts->ts_status) {
@@ -566,22 +559,15 @@ ath_rate_tx_complete(struct ath_softc *s
dev_info,
MAC_ADDR(an->an_node.ni_macaddr),
bin_to_size(size_to_bin(frame_size)),
- sc->sc_hwmap[MS(ds->ds_ctl3, AR_XmitRate0)].ieeerate,
- MS(ds->ds_ctl2, AR_XmitDataTries0),
- sc->sc_hwmap[MS(ds->ds_ctl3, AR_XmitRate1)].ieeerate,
- MS(ds->ds_ctl2, AR_XmitDataTries1),
- sc->sc_hwmap[MS(ds->ds_ctl3, AR_XmitRate2)].ieeerate,
- MS(ds->ds_ctl2, AR_XmitDataTries2),
- sc->sc_hwmap[MS(ds->ds_ctl3, AR_XmitRate3)].ieeerate,
- MS(ds->ds_ctl2, AR_XmitDataTries3),
+ mrr->rate0,
+ mrr->rate1,
+ mrr->rate2,
+ mrr->rate3,
ts->ts_status ? "FAIL" : "OK",
short_tries, long_tries);
}
- mrr = sc->sc_mrretry && !(ic->ic_flags & IEEE80211_F_USEPROT) && ENABLE_MRR;
-
-
- if (!mrr || !(ts->ts_rate & HAL_TXSTAT_ALTRATE)) {
+ if (!use_mrr || !(ts->ts_rate & HAL_TXSTAT_ALTRATE)) {
/* only one rate was used */
int ndx = rate_to_ndx(sn, final_rate);
if ((ndx >= 0) && (ndx < sn->num_rates)) {
@@ -593,7 +579,6 @@ ath_rate_tx_complete(struct ath_softc *s
short_tries, long_tries, ts->ts_status);
}
} else {
- unsigned int rate[4], tries[4];
int ndx[4];
int finalTSIdx = ts->ts_finaltsi;
@@ -601,21 +586,10 @@ ath_rate_tx_complete(struct ath_softc *s
* Process intermediate rates that failed.
*/
- rate[0] = sc->sc_hwmap[MS(ds->ds_ctl3, AR_XmitRate0)].ieeerate;
- tries[0] = MS(ds->ds_ctl2, AR_XmitDataTries0);
- ndx[0] = rate_to_ndx(sn, rate[0]);
-
- rate[1] = sc->sc_hwmap[MS(ds->ds_ctl3, AR_XmitRate1)].ieeerate;
- tries[1] = MS(ds->ds_ctl2, AR_XmitDataTries1);
- ndx[1] = rate_to_ndx(sn, rate[1]);
-
- rate[2] = sc->sc_hwmap[MS(ds->ds_ctl3, AR_XmitRate2)].ieeerate;
- tries[2] = MS(ds->ds_ctl2, AR_XmitDataTries2);
- ndx[2] = rate_to_ndx(sn, rate[2]);
-
- rate[3] = sc->sc_hwmap[MS(ds->ds_ctl3, AR_XmitRate3)].ieeerate;
- tries[3] = MS(ds->ds_ctl2, AR_XmitDataTries3);
- ndx[3] = rate_to_ndx(sn, rate[3]);
+ ndx[0] = rate_to_ndx(sn, mrr->rate0);
+ ndx[1] = rate_to_ndx(sn, mrr->rate1);
+ ndx[2] = rate_to_ndx(sn, mrr->rate2);
+ ndx[3] = rate_to_ndx(sn, mrr->rate3);
#if 0
DPRINTF(sc, ATH_DEBUG_RATE, "%s: " MAC_FMT " size %u finaltsidx %u tries %u status %u rate/try %u/%u %u/%u %u/%u %u/%u\n",
@@ -636,43 +610,43 @@ ath_rate_tx_complete(struct ath_softc *s
* sample higher rates 1 try at a time doing so
* may unfairly penalize them.
*/
- if (tries[0] && ndx[0] >= 0) {
+ if (mrr->retries0 && ndx[0] >= 0) {
update_stats(sc, an, frame_size,
- ndx[0], tries[0],
- ndx[1], tries[1],
- ndx[2], tries[2],
- ndx[3], tries[3],
+ ndx[0], mrr->retries0,
+ ndx[1], mrr->retries1,
+ ndx[2], mrr->retries2,
+ ndx[3], mrr->retries3,
short_tries, long_tries,
- long_tries > tries[0]);
- long_tries -= tries[0];
+ long_tries > mrr->retries0);
+ long_tries -= mrr->retries0;
}
- if (tries[1] && ndx[1] >= 0 && finalTSIdx > 0) {
+ if (mrr->retries1 && ndx[1] >= 0 && finalTSIdx > 0) {
update_stats(sc, an, frame_size,
- ndx[1], tries[1],
- ndx[2], tries[2],
- ndx[3], tries[3],
+ ndx[1], mrr->retries1,
+ ndx[2], mrr->retries2,
+ ndx[3], mrr->retries3,
0, 0,
short_tries, long_tries,
ts->ts_status);
- long_tries -= tries[1];
+ long_tries -= mrr->retries1;
}
- if (tries[2] && ndx[2] >= 0 && finalTSIdx > 1) {
+ if (mrr->retries2 && ndx[2] >= 0 && finalTSIdx > 1) {
update_stats(sc, an, frame_size,
- ndx[2], tries[2],
- ndx[3], tries[3],
+ ndx[2], mrr->retries2,
+ ndx[3], mrr->retries3,
0, 0,
0, 0,
short_tries, long_tries,
ts->ts_status);
- long_tries -= tries[2];
+ long_tries -= mrr->retries2;
}
- if (tries[3] && ndx[3] >= 0 && finalTSIdx > 2) {
+ if (mrr->retries3 && ndx[3] >= 0 && finalTSIdx > 2) {
update_stats(sc, an, frame_size,
- ndx[3], tries[3],
+ ndx[3], mrr->retries3,
0, 0,
0, 0,
0, 0,
--- a/ath_rate/sample/sample.h
+++ b/ath_rate/sample/sample.h
@@ -98,36 +98,4 @@ struct sample_node {
};
#define ATH_NODE_SAMPLE(an) ((struct sample_node *)&an[1])
-
-/*
- * Definitions for pulling the rate and trie counts from
- * a 5212 h/w descriptor. These Don't belong here; the
- * driver should record this information so the rate control
- * code doesn't go groveling around in the descriptor bits.
- */
-#define ds_ctl2 ds_hw[0]
-#define ds_ctl3 ds_hw[1]
-
-/* TX ds_ctl3 */
-#define AR_XmitDataTries0 0x000f0000 /* series 0 max attempts */
-#define AR_XmitDataTries0_S 16
-#define AR_XmitDataTries1 0x00f00000 /* series 1 max attempts */
-#define AR_XmitDataTries1_S 20
-#define AR_XmitDataTries2 0x0f000000 /* series 2 max attempts */
-#define AR_XmitDataTries2_S 24
-#define AR_XmitDataTries3 0xf0000000 /* series 3 max attempts */
-#define AR_XmitDataTries3_S 28
-
-/* TX ds_ctl3 */
-#define AR_XmitRate0 0x0000001f /* series 0 tx rate */
-#define AR_XmitRate0_S 0
-#define AR_XmitRate1 0x000003e0 /* series 1 tx rate */
-#define AR_XmitRate1_S 5
-#define AR_XmitRate2 0x00007c00 /* series 2 tx rate */
-#define AR_XmitRate2_S 10
-#define AR_XmitRate3 0x000f8000 /* series 3 tx rate */
-#define AR_XmitRate3_S 15
-
-#define MS(_v, _f) (((_v) & (_f)) >> _f##_S)
-
#endif /* _DEV_ATH_RATE_SAMPLE_H */