Search Linux Wireless

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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxxxx>
> > ---
> > 
> > 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
--
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


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux