Re: [PATCH 2/2] mac80211: Shortcut minstrel_ht rate setup for non-MRR capable devices

[Helmut Schaa <helmut.schaa@googlemail.com>]

Am Freitag, 4. März 2011 schrieb Felix Fietkau:
> On 2011-03-04 1:31 PM, Helmut Schaa wrote:
> > Devices without multi rate retry support won't be able to use all rates
> > as specified by mintrel_ht. Hence, we can simply skip setting up further
> > rates as the devices will only use the first one.
> > 
> > Signed-off-by: Helmut Schaa <helmut.schaa@googlemail.com>
> > ---
> > 
> > has_mrr is true if the device is able to make use of at least four rates. Not
> > sure if we might want to consider further special cases for devices with for
> > example a maximum of two rates.
> I think we should add a special case for handling a maximum of two
> rates. Right now this is being used by brcm80211, so applying this patch
> might make things worse there.

John, please drop this patch then.

Felix, currently minstrel_ht uses a maximum of 3 rates as far as I can see.

In the sampling case it's
	sample_rate -> highest_tp_rate -> highest_prob_rate
and in the default case it's
	highest_tp_rate -> second_highest_tp_rate -> highest_prob_rate

For devices with just one possible rate it would be either the sample_rate
or the highest_tp_rate.

And for devices with two rates it might make sense to change the logic to
something like
	sample_rate -> highest_prob_rate
and
	highest_tp_rate -> highest_prob_rate

Does that sound reasonable?

Thanks,
Helmut