[PATCH] ath10k: fix printing of peer stats in non-AP firmware

Michal Kazior michal.kazior at tieto.com
Tue Mar 25 04:45:22 EDT 2014


On 25 March 2014 09:33, Yeoh Chun-Yeow <yeohchunyeow at gmail.com> wrote:
> Hi, Michal
>
>> I suppose there's a limit how much peer entries firmware is able to
>> pack up into a single event. The second event you handle here is still
>> going to have some sort of a limit and you'll end up dropping results
>> again. The same probably goes with vdev stats (which aren't handled
>> yet, but hey).
>
> vdev stats is not supported by firmware 999.999.0.636, am I right?

True, but once it is supported then it will most likely need the same
treatment as peer stats.


> BTW, I also saw that the same happen to latest AP firmware whenever
> the peer stats is more than 1. But since the first peer stats is self
> stats, so it is ok to overwrite it.

I disagree. Why do you think it is okay to overwrite self peer stats?

Btw. 10.1 firmware has more pdev stats which breaks parsing of peer entries...


Michał



More information about the ath10k mailing list