Re: [PATCH 1/4] mac80211: Clean up rate selection

[Mattias Nissler <mattias.nissler@gmx.de>]

On Sat, 2007-12-08 at 12:33 +0100, Mattias Nissler wrote:
> > 
> > > +	/* If we haven't found the rate yet, ask the rate control algo. */
> > > +	if (!sel->rate)
> > > +		ref->ops->get_rate(ref->priv, dev, mode, skb, sel);
> > 
> > > +	/* Select a non-ERP backup rate. */
> > > +	if (!sel->nonerp) {
> > > +		for (i = 0; i < mode->num_rates - 1; i++) {
> > > +			struct ieee80211_rate *rate = &mode->rates[i];
> > > +			if (sel->rate->rate < rate->rate)
> > > +				break;
> > > +
> > > +			if (rate_supported(sta, mode, i) &&
> > > +			    !(rate->flags & IEEE80211_RATE_ERP))
> > > +				sel->nonerp = rate;
> > > +		}
> > > +	}
> > 
> > That I'm not sure about. This is the fallback rate?
> 
> Yes. It's after the get_rate() call so the algorithm can override it.
> But if it doesn't we compute it ourselves.

To be a little more specific, it's the non-extended-rate-set rate. So
the code falls back to this one e.g. if it's an AP has 80211.g but
stations that are only 80211.b

Mattias