Re: [PATCH V2] mac80211: Implementation of SIOCSIWRATE

[Larry Finger <larry.finger@lwfinger.net>]

Michael Wu wrote:
> On Thursday 07 June 2007 17:43, Larry Finger wrote:

>> +			sta->supp_rates = supp;
> This is definitely wrong. What rates we support has nothing to do with what 
> rates the AP supports.

If sta->supp_rates is not changed, how do you lock in a single rate as would be expected from an 
'iwconfig ethX rate 11M'? Won't the following code in the rate_control_rate_dec routine of 
rc80211_simple just decrease the rate if it wants to?

         while (i > 0) {
                 i--;
                 if (sta->supp_rates & BIT(i) &&
                     mode->rates[i].flags & IEEE80211_RATE_SUPPORTED) {
                         sta->txrate = i;
                         break;
                 }
         }


Larry