Commit c7317e41 authored by Felix Fietkau's avatar Felix Fietkau Committed by John W. Linville

mac80211: minstrel_ht - reduce the overhead of rate sampling

- reduce the number of retransmission attempts for sample rates
- sample lower rates less often
- do not use RTS/CTS for sampling frames
- increase the time between sampling attempts
Signed-off-by: default avatarFelix Fietkau <nbd@openwrt.org>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent ff32d9cd
...@@ -407,8 +407,8 @@ minstrel_ht_tx_status(void *priv, struct ieee80211_supported_band *sband, ...@@ -407,8 +407,8 @@ minstrel_ht_tx_status(void *priv, struct ieee80211_supported_band *sband,
mi->ampdu_len += info->status.ampdu_len; mi->ampdu_len += info->status.ampdu_len;
if (!mi->sample_wait && !mi->sample_tries && mi->sample_count > 0) { if (!mi->sample_wait && !mi->sample_tries && mi->sample_count > 0) {
mi->sample_wait = 4 + 2 * MINSTREL_TRUNC(mi->avg_ampdu_len); mi->sample_wait = 16 + 2 * MINSTREL_TRUNC(mi->avg_ampdu_len);
mi->sample_tries = 3; mi->sample_tries = 2;
mi->sample_count--; mi->sample_count--;
} }
...@@ -506,7 +506,9 @@ minstrel_ht_set_rate(struct minstrel_priv *mp, struct minstrel_ht_sta *mi, ...@@ -506,7 +506,9 @@ minstrel_ht_set_rate(struct minstrel_priv *mp, struct minstrel_ht_sta *mi,
if (!mr->retry_updated) if (!mr->retry_updated)
minstrel_calc_retransmit(mp, mi, index); minstrel_calc_retransmit(mp, mi, index);
if (mr->probability < MINSTREL_FRAC(20, 100)) if (sample)
rate->count = 1;
else if (mr->probability < MINSTREL_FRAC(20, 100))
rate->count = 2; rate->count = 2;
else if (rtscts) else if (rtscts)
rate->count = mr->retry_count_rtscts; rate->count = mr->retry_count_rtscts;
...@@ -562,7 +564,7 @@ minstrel_get_sample_rate(struct minstrel_priv *mp, struct minstrel_ht_sta *mi) ...@@ -562,7 +564,7 @@ minstrel_get_sample_rate(struct minstrel_priv *mp, struct minstrel_ht_sta *mi)
*/ */
if (minstrel_get_duration(sample_idx) > if (minstrel_get_duration(sample_idx) >
minstrel_get_duration(mi->max_tp_rate)) { minstrel_get_duration(mi->max_tp_rate)) {
if (mr->sample_skipped < 10) if (mr->sample_skipped < 20)
goto next; goto next;
if (mi->sample_slow++ > 2) if (mi->sample_slow++ > 2)
...@@ -586,6 +588,7 @@ minstrel_ht_get_rate(void *priv, struct ieee80211_sta *sta, void *priv_sta, ...@@ -586,6 +588,7 @@ minstrel_ht_get_rate(void *priv, struct ieee80211_sta *sta, void *priv_sta,
struct minstrel_ht_sta *mi = &msp->ht; struct minstrel_ht_sta *mi = &msp->ht;
struct minstrel_priv *mp = priv; struct minstrel_priv *mp = priv;
int sample_idx; int sample_idx;
bool sample = false;
if (rate_control_send_low(sta, priv_sta, txrc)) if (rate_control_send_low(sta, priv_sta, txrc))
return; return;
...@@ -596,10 +599,11 @@ minstrel_ht_get_rate(void *priv, struct ieee80211_sta *sta, void *priv_sta, ...@@ -596,10 +599,11 @@ minstrel_ht_get_rate(void *priv, struct ieee80211_sta *sta, void *priv_sta,
info->flags |= mi->tx_flags; info->flags |= mi->tx_flags;
sample_idx = minstrel_get_sample_rate(mp, mi); sample_idx = minstrel_get_sample_rate(mp, mi);
if (sample_idx >= 0) { if (sample_idx >= 0) {
sample = true;
minstrel_ht_set_rate(mp, mi, &ar[0], sample_idx, minstrel_ht_set_rate(mp, mi, &ar[0], sample_idx,
txrc, true, false); txrc, true, false);
minstrel_ht_set_rate(mp, mi, &ar[1], mi->max_tp_rate, minstrel_ht_set_rate(mp, mi, &ar[1], mi->max_tp_rate,
txrc, false, true); txrc, false, false);
info->flags |= IEEE80211_TX_CTL_RATE_CTRL_PROBE; info->flags |= IEEE80211_TX_CTL_RATE_CTRL_PROBE;
} else { } else {
minstrel_ht_set_rate(mp, mi, &ar[0], mi->max_tp_rate, minstrel_ht_set_rate(mp, mi, &ar[0], mi->max_tp_rate,
...@@ -607,7 +611,7 @@ minstrel_ht_get_rate(void *priv, struct ieee80211_sta *sta, void *priv_sta, ...@@ -607,7 +611,7 @@ minstrel_ht_get_rate(void *priv, struct ieee80211_sta *sta, void *priv_sta,
minstrel_ht_set_rate(mp, mi, &ar[1], mi->max_tp_rate2, minstrel_ht_set_rate(mp, mi, &ar[1], mi->max_tp_rate2,
txrc, false, true); txrc, false, true);
} }
minstrel_ht_set_rate(mp, mi, &ar[2], mi->max_prob_rate, txrc, false, true); minstrel_ht_set_rate(mp, mi, &ar[2], mi->max_prob_rate, txrc, false, !sample);
ar[3].count = 0; ar[3].count = 0;
ar[3].idx = -1; ar[3].idx = -1;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment