On Mon, 2012-11-19 at 18:39 +0200, Eliad Peller wrote:
> When first configuring the rate policy, before auth,
> we still don't have the correct rates that were
> agreed during association.
>
> Reconfigure the rate policy on association in order
> to update them.
>
> Signed-off-by: Eliad Peller <eliad@xxxxxxxxxx>
> ---
> drivers/net/wireless/ti/wlcore/main.c | 14 +++++++++++++-
> 1 files changed, 13 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/net/wireless/ti/wlcore/main.c b/drivers/net/wireless/ti/wlcore/main.c
> index 0dd0ec1..a1ad326 100644
> --- a/drivers/net/wireless/ti/wlcore/main.c
> +++ b/drivers/net/wireless/ti/wlcore/main.c
> @@ -3771,7 +3771,8 @@ static void wl1271_bss_info_changed_sta(struct wl1271 *wl,
> wlvif->rssi_thold = bss_conf->cqm_rssi_thold;
> }
>
> - if (changed & (BSS_CHANGED_BSSID | BSS_CHANGED_HT)) {
> + if (changed & (BSS_CHANGED_BSSID | BSS_CHANGED_HT |
> + BSS_CHANGED_ASSOC)) {
> rcu_read_lock();
> sta = ieee80211_find_sta(vif, bss_conf->bssid);
> if (!sta)
> @@ -3958,6 +3959,17 @@ sta_not_found:
> if (ret < 0)
> goto out;
>
> + if (sta_rate_set) {
> + wlvif->rate_set =
> + wl1271_tx_enabled_rates_get(wl,
> + sta_rate_set,
> + wlvif->band);
> +
> + ret = wl1271_acx_sta_rate_policies(wl, wlvif);
> + if (ret < 0)
> + goto out;
> + }
> +
> if (test_bit(WLVIF_FLAG_STA_AUTHORIZED, &wlvif->flags))
> wl12xx_set_authorized(wl, wlvif);
> }
Yes, it's messy that we use the same value for vif-supported rates and
current association negotiated rates... :(
--
Luca.
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
