* [mac80211] minstrel_ht rate population
@ 2010-08-17 17:37 Michael Griepentrog
2010-08-17 18:22 ` Felix Fietkau
0 siblings, 1 reply; 2+ messages in thread
From: Michael Griepentrog @ 2010-08-17 17:37 UTC (permalink / raw)
To: linux-wireless
Hi,
I'm looking at minstrel_ht_get_rate() from the latest git, and I
noticed that the rates are getting set into info->status.rates. Is
this correct? My impression was that array gets populated by the
driver after a successful transmission. ath9k's rate control algorithm
uses the expected tx_info->control.rates. Minstrel_ht appears to still
work, am I missing something?
Thanks,
Michael Griepentrog
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [mac80211] minstrel_ht rate population
2010-08-17 17:37 [mac80211] minstrel_ht rate population Michael Griepentrog
@ 2010-08-17 18:22 ` Felix Fietkau
0 siblings, 0 replies; 2+ messages in thread
From: Felix Fietkau @ 2010-08-17 18:22 UTC (permalink / raw)
To: Michael Griepentrog; +Cc: linux-wireless
On 2010-08-17 7:37 PM, Michael Griepentrog wrote:
> Hi,
>
> I'm looking at minstrel_ht_get_rate() from the latest git, and I
> noticed that the rates are getting set into info->status.rates. Is
> this correct? My impression was that array gets populated by the
> driver after a successful transmission. ath9k's rate control algorithm
> uses the expected tx_info->control.rates. Minstrel_ht appears to still
> work, am I missing something?
That's technically a bug (I'll send a fix for this soon), but it doesn't
matter in practice, as info->control.rates always points to the same
location as info->status.rates.
- Felix
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2010-08-17 18:22 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-08-17 17:37 [mac80211] minstrel_ht rate population Michael Griepentrog
2010-08-17 18:22 ` Felix Fietkau
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).