On Tue, Nov 20, 2012 at 10:13 AM, Luciano Coelho <coelho@xxxxxx> wrote:
> 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... :(
>
yeah, we have a total mess with all the configured/supported rates.
we'll have to reorganize it at some point.
Eliad.
--
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
