i checked your patch on 10.4 based chipsets with 9984. the values are
now looking bogus and wrong at all. busy and active time time in ms does
increase in hours each second
the problem seem to be that your patch is 10.2.4 only related.
ath_clean_survey does not trigger on 10.4 so the values double itself
each time the event raises since you add the full values and not just a
delta on top
Sebastian
Am 18.09.2019 um 14:42 schrieb Sven Eckelmann:
From: Sven Eckelmann <seckelmann@xxxxxxxxx>
Hi,
it was observed that ath9k provides accumulated survey counters but ath10k
neither provides deltas nor accumulated counters. Instead it returns
some value which was returned at some point from the firmware.
But as it turns out, this data is not reliable. To make it more useful,
ath10k has to:
* retrieve counters rather frequently for hardware which is known to use
firmware versions with low number counter bits (for only 14-30s)
* clean up received counter values
* accumulate counters from firmware
A comparison of the resulting output with these fixes can be seen under
https://stats.freifunk-vogtland.net/d/ffv_node/nodeinfo?orgId=1&var-node=ac86749f4d60&fullscreen&panelId=5&from=1568782046974&to=1568807068706
The left side of the graph shows the output before the patches were applied
and the right side the output with the patches applied. Just as reference, an
ath9k device in the same building is
https://stats.freifunk-vogtland.net/d/ffv_node/nodeinfo?orgId=1&var-node=ac86740037e0&fullscreen&panelId=5&from=1568782046974&to=1568807068706
Kind regards,
Sven
Sven Eckelmann (2):
ath10k: report survey info as accumulated values
ath10k: regularly fetch survey counters
drivers/net/wireless/ath/ath10k/core.c | 8 ++++
drivers/net/wireless/ath/ath10k/core.h | 3 ++
drivers/net/wireless/ath/ath10k/hw.c | 13 +++--
drivers/net/wireless/ath/ath10k/mac.c | 52 ++++++++++++++++++++
drivers/net/wireless/ath/ath10k/mac.h | 3 ++
drivers/net/wireless/ath/ath10k/wmi.c | 66 ++++++++++++++++++++++----
6 files changed, 130 insertions(+), 15 deletions(-)