Re: Clarification needed on Last Tx Rate statistic update

[Saravana <saravanad@posedge.com>]

Hi Johannes,

Thanks for the clarification. So it seems the in the other case that you 
had mentioned, we can confirm that last_tx_rate is updated with 
inappropriate value.
So probably we could add another variable for the highest 
performing/probability rate and update it with the  txrc.reported rate 
value.
The Last_tx_rate update with the highest performing/probability rate can 
be removed.

As far as Last Tx Rate is concerned, should we need to check for the flag?
I think the Last Tx Rate can be updated irrespective of the 
IEEE80211_HW_HAS_RATE_CONTROL set or not.

Please let me know your inputs on this.

Thanks,
Saravana
On 11/07/2012 01:35 PM, Johannes Berg wrote:
> On Wed, 2012-11-07 at 13:17 +0530, Saravana wrote:
>
>> Could somebody clarify why the statistics update happens only when the
>> rate control is done by the hardware(flag IEEE80211_HW_HAS_RATE_CONTROL
>> is set)?
> In the other case, it's set to txrc.reported_rate upon transmission, so
> it's not the *last* rate, but more accurately the highest
> performing/probability rate.
>
> johannes
>
>