Commit e00cfce0 authored by Jouni Malinen's avatar Jouni Malinen Committed by John W. Linville

mac80211: Select lowest rate based on basic rate set in AP mode

If the basic rate set is configured to not include the lowest rate
(e.g., basic rate set = 6, 12, 24 Mbps in IEEE 802.11g mode), the AP
should not send out broadcast frames at 1 Mbps. This type of
configuration can be used to optimize channel usage in cases where
there is no need for backwards compatibility with IEEE 802.11b-only
devices.

In AP mode, mac80211 was unconditionally using the lowest rate for
Beacon frames and similarly, with all rate control algorithms that use
rate_control_send_low(), the lowest rate ended up being used for all
broadcast frames (and all unicast frames that are sent before
association). Change this to take into account the basic rate
configuration in AP mode, i.e., use the lowest rate in the basic rate
set instead of the lowest supported rate when selecting the rate.
Signed-off-by: default avatarJouni Malinen <jouni.malinen@atheros.com>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 3dc1de0b
...@@ -2299,6 +2299,7 @@ enum rate_control_changed { ...@@ -2299,6 +2299,7 @@ enum rate_control_changed {
* @max_rate_idx: user-requested maximum rate (not MCS for now) * @max_rate_idx: user-requested maximum rate (not MCS for now)
* @skb: the skb that will be transmitted, the control information in it needs * @skb: the skb that will be transmitted, the control information in it needs
* to be filled in * to be filled in
* @ap: whether this frame is sent out in AP mode
*/ */
struct ieee80211_tx_rate_control { struct ieee80211_tx_rate_control {
struct ieee80211_hw *hw; struct ieee80211_hw *hw;
...@@ -2308,6 +2309,7 @@ struct ieee80211_tx_rate_control { ...@@ -2308,6 +2309,7 @@ struct ieee80211_tx_rate_control {
struct ieee80211_tx_rate reported_rate; struct ieee80211_tx_rate reported_rate;
bool rts, short_preamble; bool rts, short_preamble;
u8 max_rate_idx; u8 max_rate_idx;
bool ap;
}; };
struct rate_control_ops { struct rate_control_ops {
......
...@@ -207,6 +207,27 @@ static bool rc_no_data_or_no_ack(struct ieee80211_tx_rate_control *txrc) ...@@ -207,6 +207,27 @@ static bool rc_no_data_or_no_ack(struct ieee80211_tx_rate_control *txrc)
return ((info->flags & IEEE80211_TX_CTL_NO_ACK) || !ieee80211_is_data(fc)); return ((info->flags & IEEE80211_TX_CTL_NO_ACK) || !ieee80211_is_data(fc));
} }
static void rc_send_low_broadcast(s8 *idx, u32 basic_rates, u8 max_rate_idx)
{
u8 i;
if (basic_rates == 0)
return; /* assume basic rates unknown and accept rate */
if (*idx < 0)
return;
if (basic_rates & (1 << *idx))
return; /* selected rate is a basic rate */
for (i = *idx + 1; i <= max_rate_idx; i++) {
if (basic_rates & (1 << i)) {
*idx = i;
return;
}
}
/* could not find a basic rate; use original selection */
}
bool rate_control_send_low(struct ieee80211_sta *sta, bool rate_control_send_low(struct ieee80211_sta *sta,
void *priv_sta, void *priv_sta,
struct ieee80211_tx_rate_control *txrc) struct ieee80211_tx_rate_control *txrc)
...@@ -218,6 +239,10 @@ bool rate_control_send_low(struct ieee80211_sta *sta, ...@@ -218,6 +239,10 @@ bool rate_control_send_low(struct ieee80211_sta *sta,
info->control.rates[0].count = info->control.rates[0].count =
(info->flags & IEEE80211_TX_CTL_NO_ACK) ? (info->flags & IEEE80211_TX_CTL_NO_ACK) ?
1 : txrc->hw->max_rate_tries; 1 : txrc->hw->max_rate_tries;
if (!sta && txrc->ap)
rc_send_low_broadcast(&info->control.rates[0].idx,
txrc->bss_conf->basic_rates,
txrc->sband->n_bitrates);
return true; return true;
} }
return false; return false;
......
...@@ -520,6 +520,7 @@ ieee80211_tx_h_rate_ctrl(struct ieee80211_tx_data *tx) ...@@ -520,6 +520,7 @@ ieee80211_tx_h_rate_ctrl(struct ieee80211_tx_data *tx)
txrc.skb = tx->skb; txrc.skb = tx->skb;
txrc.reported_rate.idx = -1; txrc.reported_rate.idx = -1;
txrc.max_rate_idx = tx->sdata->max_ratectrl_rateidx; txrc.max_rate_idx = tx->sdata->max_ratectrl_rateidx;
txrc.ap = tx->sdata->vif.type == NL80211_IFTYPE_AP;
/* set up RTS protection if desired */ /* set up RTS protection if desired */
if (len > tx->local->hw.wiphy->rts_threshold) { if (len > tx->local->hw.wiphy->rts_threshold) {
...@@ -2060,6 +2061,7 @@ struct sk_buff *ieee80211_beacon_get_tim(struct ieee80211_hw *hw, ...@@ -2060,6 +2061,7 @@ struct sk_buff *ieee80211_beacon_get_tim(struct ieee80211_hw *hw,
struct beacon_data *beacon; struct beacon_data *beacon;
struct ieee80211_supported_band *sband; struct ieee80211_supported_band *sband;
enum ieee80211_band band = local->hw.conf.channel->band; enum ieee80211_band band = local->hw.conf.channel->band;
struct ieee80211_tx_rate_control txrc;
sband = local->hw.wiphy->bands[band]; sband = local->hw.wiphy->bands[band];
...@@ -2167,21 +2169,21 @@ struct sk_buff *ieee80211_beacon_get_tim(struct ieee80211_hw *hw, ...@@ -2167,21 +2169,21 @@ struct sk_buff *ieee80211_beacon_get_tim(struct ieee80211_hw *hw,
info = IEEE80211_SKB_CB(skb); info = IEEE80211_SKB_CB(skb);
info->flags |= IEEE80211_TX_INTFL_DONT_ENCRYPT; info->flags |= IEEE80211_TX_INTFL_DONT_ENCRYPT;
info->flags |= IEEE80211_TX_CTL_NO_ACK;
info->band = band; info->band = band;
/*
* XXX: For now, always use the lowest rate memset(&txrc, 0, sizeof(txrc));
*/ txrc.hw = hw;
info->control.rates[0].idx = 0; txrc.sband = sband;
info->control.rates[0].count = 1; txrc.bss_conf = &sdata->vif.bss_conf;
info->control.rates[1].idx = -1; txrc.skb = skb;
info->control.rates[2].idx = -1; txrc.reported_rate.idx = -1;
info->control.rates[3].idx = -1; txrc.max_rate_idx = sdata->max_ratectrl_rateidx;
info->control.rates[4].idx = -1; txrc.ap = true;
BUILD_BUG_ON(IEEE80211_TX_MAX_RATES != 5); rate_control_get_rate(sdata, NULL, &txrc);
info->control.vif = vif; info->control.vif = vif;
info->flags |= IEEE80211_TX_CTL_NO_ACK;
info->flags |= IEEE80211_TX_CTL_CLEAR_PS_FILT; info->flags |= IEEE80211_TX_CTL_CLEAR_PS_FILT;
info->flags |= IEEE80211_TX_CTL_ASSIGN_SEQ; info->flags |= IEEE80211_TX_CTL_ASSIGN_SEQ;
out: out:
......
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