* [PATCH] ath10k: fix printing of peer stats in non-AP firmware
@ 2014-03-25 7:55 ` Chun-Yeow Yeoh
0 siblings, 0 replies; 22+ messages in thread
From: Chun-Yeow Yeoh @ 2014-03-25 7:55 UTC (permalink / raw)
To: ath10k; +Cc: kvalo, linux-wireless, Chun-Yeow Yeoh
This patch is intended to fix the problem if we use the
firmware 999.999.0.636 to get peer stats when the number
of peer is more than 3. The WMI_UPDATE_STATS_EVENTID may
trigger more than 1 time if the number of peers is more
than 3. So this patch allows us to do the checking on this
and make sure that we print the peer stats correctly.
Signed-off-by: Chun-Yeow Yeoh <yeohchunyeow@gmail.com>
---
drivers/net/wireless/ath/ath10k/debug.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/drivers/net/wireless/ath/ath10k/debug.c b/drivers/net/wireless/ath/ath10k/debug.c
index 7be284c..4a4072e 100644
--- a/drivers/net/wireless/ath/ath10k/debug.c
+++ b/drivers/net/wireless/ath/ath10k/debug.c
@@ -245,10 +245,17 @@ void ath10k_debug_read_target_stats(struct ath10k *ar,
if (num_peer_stats) {
struct wmi_peer_stats_10x *peer_stats;
struct ath10k_peer_stat *s;
+ int j = 0;
+
+ if (!test_bit(ATH10K_FW_FEATURE_WMI_10X, ar->fw_features) &&
+ !num_pdev_stats) {
+ j = 3;
+ num_peer_stats += 3;
+ }
stats->peers = num_peer_stats;
- for (i = 0; i < num_peer_stats; i++) {
+ for (i = j; i < num_peer_stats; i++) {
peer_stats = (struct wmi_peer_stats_10x *)tmp;
s = &stats->peer_stat[i];
--
1.7.9.5
_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH] ath10k: fix printing of peer stats in non-AP firmware
@ 2014-03-25 7:55 ` Chun-Yeow Yeoh
0 siblings, 0 replies; 22+ messages in thread
From: Chun-Yeow Yeoh @ 2014-03-25 7:55 UTC (permalink / raw)
To: ath10k; +Cc: linux-wireless, kvalo, Chun-Yeow Yeoh
This patch is intended to fix the problem if we use the
firmware 999.999.0.636 to get peer stats when the number
of peer is more than 3. The WMI_UPDATE_STATS_EVENTID may
trigger more than 1 time if the number of peers is more
than 3. So this patch allows us to do the checking on this
and make sure that we print the peer stats correctly.
Signed-off-by: Chun-Yeow Yeoh <yeohchunyeow@gmail.com>
---
drivers/net/wireless/ath/ath10k/debug.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/drivers/net/wireless/ath/ath10k/debug.c b/drivers/net/wireless/ath/ath10k/debug.c
index 7be284c..4a4072e 100644
--- a/drivers/net/wireless/ath/ath10k/debug.c
+++ b/drivers/net/wireless/ath/ath10k/debug.c
@@ -245,10 +245,17 @@ void ath10k_debug_read_target_stats(struct ath10k *ar,
if (num_peer_stats) {
struct wmi_peer_stats_10x *peer_stats;
struct ath10k_peer_stat *s;
+ int j = 0;
+
+ if (!test_bit(ATH10K_FW_FEATURE_WMI_10X, ar->fw_features) &&
+ !num_pdev_stats) {
+ j = 3;
+ num_peer_stats += 3;
+ }
stats->peers = num_peer_stats;
- for (i = 0; i < num_peer_stats; i++) {
+ for (i = j; i < num_peer_stats; i++) {
peer_stats = (struct wmi_peer_stats_10x *)tmp;
s = &stats->peer_stat[i];
--
1.7.9.5
^ permalink raw reply related [flat|nested] 22+ messages in thread* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
2014-03-25 7:55 ` Chun-Yeow Yeoh
@ 2014-03-25 8:11 ` Michal Kazior
-1 siblings, 0 replies; 22+ messages in thread
From: Michal Kazior @ 2014-03-25 8:11 UTC (permalink / raw)
To: Chun-Yeow Yeoh; +Cc: Kalle Valo, linux-wireless, ath10k@lists.infradead.org
On 25 March 2014 08:55, Chun-Yeow Yeoh <yeohchunyeow@gmail.com> wrote:
> This patch is intended to fix the problem if we use the
> firmware 999.999.0.636 to get peer stats when the number
> of peer is more than 3. The WMI_UPDATE_STATS_EVENTID may
> trigger more than 1 time if the number of peers is more
> than 3. So this patch allows us to do the checking on this
> and make sure that we print the peer stats correctly.
>
> Signed-off-by: Chun-Yeow Yeoh <yeohchunyeow@gmail.com>
> ---
> drivers/net/wireless/ath/ath10k/debug.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/wireless/ath/ath10k/debug.c b/drivers/net/wireless/ath/ath10k/debug.c
> index 7be284c..4a4072e 100644
> --- a/drivers/net/wireless/ath/ath10k/debug.c
> +++ b/drivers/net/wireless/ath/ath10k/debug.c
> @@ -245,10 +245,17 @@ void ath10k_debug_read_target_stats(struct ath10k *ar,
> if (num_peer_stats) {
> struct wmi_peer_stats_10x *peer_stats;
> struct ath10k_peer_stat *s;
> + int j = 0;
> +
> + if (!test_bit(ATH10K_FW_FEATURE_WMI_10X, ar->fw_features) &&
> + !num_pdev_stats) {
> + j = 3;
> + num_peer_stats += 3;
> + }
>
> stats->peers = num_peer_stats;
>
> - for (i = 0; i < num_peer_stats; i++) {
> + for (i = j; i < num_peer_stats; i++) {
> peer_stats = (struct wmi_peer_stats_10x *)tmp;
> s = &stats->peer_stat[i];
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).
I think a better approach would be to make read_target_stats *append*
data to the stats structure and make ath10k wait for more possible
events instead of being satisfied with just one.
The way you do it now seems like you end up with the extra results
being shown upon *next* stat request-response.. i.e. you see old
results.
Michał
_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
@ 2014-03-25 8:11 ` Michal Kazior
0 siblings, 0 replies; 22+ messages in thread
From: Michal Kazior @ 2014-03-25 8:11 UTC (permalink / raw)
To: Chun-Yeow Yeoh; +Cc: ath10k@lists.infradead.org, linux-wireless, Kalle Valo
On 25 March 2014 08:55, Chun-Yeow Yeoh <yeohchunyeow@gmail.com> wrote:
> This patch is intended to fix the problem if we use the
> firmware 999.999.0.636 to get peer stats when the number
> of peer is more than 3. The WMI_UPDATE_STATS_EVENTID may
> trigger more than 1 time if the number of peers is more
> than 3. So this patch allows us to do the checking on this
> and make sure that we print the peer stats correctly.
>
> Signed-off-by: Chun-Yeow Yeoh <yeohchunyeow@gmail.com>
> ---
> drivers/net/wireless/ath/ath10k/debug.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/wireless/ath/ath10k/debug.c b/drivers/net/wireless/ath/ath10k/debug.c
> index 7be284c..4a4072e 100644
> --- a/drivers/net/wireless/ath/ath10k/debug.c
> +++ b/drivers/net/wireless/ath/ath10k/debug.c
> @@ -245,10 +245,17 @@ void ath10k_debug_read_target_stats(struct ath10k *ar,
> if (num_peer_stats) {
> struct wmi_peer_stats_10x *peer_stats;
> struct ath10k_peer_stat *s;
> + int j = 0;
> +
> + if (!test_bit(ATH10K_FW_FEATURE_WMI_10X, ar->fw_features) &&
> + !num_pdev_stats) {
> + j = 3;
> + num_peer_stats += 3;
> + }
>
> stats->peers = num_peer_stats;
>
> - for (i = 0; i < num_peer_stats; i++) {
> + for (i = j; i < num_peer_stats; i++) {
> peer_stats = (struct wmi_peer_stats_10x *)tmp;
> s = &stats->peer_stat[i];
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).
I think a better approach would be to make read_target_stats *append*
data to the stats structure and make ath10k wait for more possible
events instead of being satisfied with just one.
The way you do it now seems like you end up with the extra results
being shown upon *next* stat request-response.. i.e. you see old
results.
Michał
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
2014-03-25 8:11 ` Michal Kazior
@ 2014-03-25 8:33 ` Yeoh Chun-Yeow
-1 siblings, 0 replies; 22+ messages in thread
From: Yeoh Chun-Yeow @ 2014-03-25 8:33 UTC (permalink / raw)
To: Michal Kazior; +Cc: Kalle Valo, linux-wireless, ath10k@lists.infradead.org
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?
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 think a better approach would be to make read_target_stats *append*
> data to the stats structure and make ath10k wait for more possible
> events instead of being satisfied with just one.
>
> The way you do it now seems like you end up with the extra results
> being shown upon *next* stat request-response.. i.e. you see old
> results.
Will take a look on this.
----
Chun-Yeow
_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
@ 2014-03-25 8:33 ` Yeoh Chun-Yeow
0 siblings, 0 replies; 22+ messages in thread
From: Yeoh Chun-Yeow @ 2014-03-25 8:33 UTC (permalink / raw)
To: Michal Kazior; +Cc: ath10k@lists.infradead.org, linux-wireless, Kalle Valo
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?
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 think a better approach would be to make read_target_stats *append*
> data to the stats structure and make ath10k wait for more possible
> events instead of being satisfied with just one.
>
> The way you do it now seems like you end up with the extra results
> being shown upon *next* stat request-response.. i.e. you see old
> results.
Will take a look on this.
----
Chun-Yeow
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
2014-03-25 8:33 ` Yeoh Chun-Yeow
@ 2014-03-25 8:45 ` Michal Kazior
-1 siblings, 0 replies; 22+ messages in thread
From: Michal Kazior @ 2014-03-25 8:45 UTC (permalink / raw)
To: Yeoh Chun-Yeow; +Cc: Kalle Valo, linux-wireless, ath10k@lists.infradead.org
On 25 March 2014 09:33, Yeoh Chun-Yeow <yeohchunyeow@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ł
_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
@ 2014-03-25 8:45 ` Michal Kazior
0 siblings, 0 replies; 22+ messages in thread
From: Michal Kazior @ 2014-03-25 8:45 UTC (permalink / raw)
To: Yeoh Chun-Yeow; +Cc: ath10k@lists.infradead.org, linux-wireless, Kalle Valo
On 25 March 2014 09:33, Yeoh Chun-Yeow <yeohchunyeow@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ł
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
2014-03-25 8:45 ` Michal Kazior
@ 2014-03-25 8:54 ` Yeoh Chun-Yeow
-1 siblings, 0 replies; 22+ messages in thread
From: Yeoh Chun-Yeow @ 2014-03-25 8:54 UTC (permalink / raw)
To: Michal Kazior; +Cc: Kalle Valo, linux-wireless, ath10k@lists.infradead.org
>
>> 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?
>
Because TxRate, RxRate, RSSI are all no value, just the MAC address.
> Btw. 10.1 firmware has more pdev stats which breaks parsing of peer entries...
Yes, let me investigate further.
-----
Chun-Yeow
_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
@ 2014-03-25 8:54 ` Yeoh Chun-Yeow
0 siblings, 0 replies; 22+ messages in thread
From: Yeoh Chun-Yeow @ 2014-03-25 8:54 UTC (permalink / raw)
To: Michal Kazior; +Cc: ath10k@lists.infradead.org, linux-wireless, Kalle Valo
>
>> 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?
>
Because TxRate, RxRate, RSSI are all no value, just the MAC address.
> Btw. 10.1 firmware has more pdev stats which breaks parsing of peer entries...
Yes, let me investigate further.
-----
Chun-Yeow
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
2014-03-25 8:54 ` Yeoh Chun-Yeow
@ 2014-03-25 8:57 ` Michal Kazior
-1 siblings, 0 replies; 22+ messages in thread
From: Michal Kazior @ 2014-03-25 8:57 UTC (permalink / raw)
To: Yeoh Chun-Yeow; +Cc: Kalle Valo, linux-wireless, ath10k@lists.infradead.org
On 25 March 2014 09:54, Yeoh Chun-Yeow <yeohchunyeow@gmail.com> wrote:
>>
>>> 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?
>>
> Because TxRate, RxRate, RSSI are all no value, just the MAC address.
I don't think that's true for 10.1 firmware and AP interface types.
Michał
_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
@ 2014-03-25 8:57 ` Michal Kazior
0 siblings, 0 replies; 22+ messages in thread
From: Michal Kazior @ 2014-03-25 8:57 UTC (permalink / raw)
To: Yeoh Chun-Yeow; +Cc: ath10k@lists.infradead.org, linux-wireless, Kalle Valo
On 25 March 2014 09:54, Yeoh Chun-Yeow <yeohchunyeow@gmail.com> wrote:
>>
>>> 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?
>>
> Because TxRate, RxRate, RSSI are all no value, just the MAC address.
I don't think that's true for 10.1 firmware and AP interface types.
Michał
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
2014-03-25 8:57 ` Michal Kazior
@ 2014-03-25 9:38 ` Yeoh Chun-Yeow
-1 siblings, 0 replies; 22+ messages in thread
From: Yeoh Chun-Yeow @ 2014-03-25 9:38 UTC (permalink / raw)
To: Michal Kazior; +Cc: Kalle Valo, linux-wireless, ath10k@lists.infradead.org
> I don't think that's true for 10.1 firmware and AP interface types.
I try to play around with 10.1 firmware as AP (3 connected STAs) and I
get the following:
The first WMI_UPDATE_STATS_EVENTID as follow:
[ 338.290000] ath10k: pdev 0 vdev 0 peer 3
[ 338.300000] ath10k: MAC 04:f0:21:0c:a5:44
[ 338.300000] ath10k: RSSI 60
[ 338.300000] ath10k: Tx 1170000
[ 338.310000] ath10k: Rx 1170000
[ 338.310000] ath10k: MAC 04:f0:21:0c:a5:19
[ 338.320000] ath10k: RSSI 64
[ 338.320000] ath10k: Tx 1300000
[ 338.320000] ath10k: Rx 975000
[ 338.320000] ath10k: MAC 04:f0:21:0c:a5:1c
[ 338.330000] ath10k: RSSI 71
[ 338.330000] ath10k: Tx 975000
[ 338.330000] ath10k: Rx 1300000
I think that above is correct.
The second WMI_UPDATE_STATS_EVENTID as follow:
[ 338.340000] ath10k: pdev 1 vdev 0 peer 1
[ 338.350000] ath10k: MAC a8:02:00:00:00:00
[ 338.350000] ath10k: RSSI 0
[ 338.350000] ath10k: Tx 565
[ 338.360000] ath10k: Rx 0
Although indicating 1 peer, but the data seems to be not the peer
stats, not even self STA stats. Any idea?
---
Chun-Yeow
_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
@ 2014-03-25 9:38 ` Yeoh Chun-Yeow
0 siblings, 0 replies; 22+ messages in thread
From: Yeoh Chun-Yeow @ 2014-03-25 9:38 UTC (permalink / raw)
To: Michal Kazior; +Cc: ath10k@lists.infradead.org, linux-wireless, Kalle Valo
> I don't think that's true for 10.1 firmware and AP interface types.
I try to play around with 10.1 firmware as AP (3 connected STAs) and I
get the following:
The first WMI_UPDATE_STATS_EVENTID as follow:
[ 338.290000] ath10k: pdev 0 vdev 0 peer 3
[ 338.300000] ath10k: MAC 04:f0:21:0c:a5:44
[ 338.300000] ath10k: RSSI 60
[ 338.300000] ath10k: Tx 1170000
[ 338.310000] ath10k: Rx 1170000
[ 338.310000] ath10k: MAC 04:f0:21:0c:a5:19
[ 338.320000] ath10k: RSSI 64
[ 338.320000] ath10k: Tx 1300000
[ 338.320000] ath10k: Rx 975000
[ 338.320000] ath10k: MAC 04:f0:21:0c:a5:1c
[ 338.330000] ath10k: RSSI 71
[ 338.330000] ath10k: Tx 975000
[ 338.330000] ath10k: Rx 1300000
I think that above is correct.
The second WMI_UPDATE_STATS_EVENTID as follow:
[ 338.340000] ath10k: pdev 1 vdev 0 peer 1
[ 338.350000] ath10k: MAC a8:02:00:00:00:00
[ 338.350000] ath10k: RSSI 0
[ 338.350000] ath10k: Tx 565
[ 338.360000] ath10k: Rx 0
Although indicating 1 peer, but the data seems to be not the peer
stats, not even self STA stats. Any idea?
---
Chun-Yeow
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
2014-03-25 9:38 ` Yeoh Chun-Yeow
@ 2014-03-25 9:47 ` Michal Kazior
-1 siblings, 0 replies; 22+ messages in thread
From: Michal Kazior @ 2014-03-25 9:47 UTC (permalink / raw)
To: Yeoh Chun-Yeow; +Cc: Kalle Valo, linux-wireless, ath10k@lists.infradead.org
On 25 March 2014 10:38, Yeoh Chun-Yeow <yeohchunyeow@gmail.com> wrote:
>> I don't think that's true for 10.1 firmware and AP interface types.
>
> I try to play around with 10.1 firmware as AP (3 connected STAs) and I
> get the following:
>
> The first WMI_UPDATE_STATS_EVENTID as follow:
> [ 338.290000] ath10k: pdev 0 vdev 0 peer 3
> [ 338.300000] ath10k: MAC 04:f0:21:0c:a5:44
> [ 338.300000] ath10k: RSSI 60
> [ 338.300000] ath10k: Tx 1170000
> [ 338.310000] ath10k: Rx 1170000
> [ 338.310000] ath10k: MAC 04:f0:21:0c:a5:19
> [ 338.320000] ath10k: RSSI 64
> [ 338.320000] ath10k: Tx 1300000
> [ 338.320000] ath10k: Rx 975000
> [ 338.320000] ath10k: MAC 04:f0:21:0c:a5:1c
> [ 338.330000] ath10k: RSSI 71
> [ 338.330000] ath10k: Tx 975000
> [ 338.330000] ath10k: Rx 1300000
>
> I think that above is correct.
>
> The second WMI_UPDATE_STATS_EVENTID as follow:
> [ 338.340000] ath10k: pdev 1 vdev 0 peer 1
> [ 338.350000] ath10k: MAC a8:02:00:00:00:00
> [ 338.350000] ath10k: RSSI 0
> [ 338.350000] ath10k: Tx 565
> [ 338.360000] ath10k: Rx 0
>
> Although indicating 1 peer, but the data seems to be not the peer
> stats, not even self STA stats. Any idea?
Yes.
The second stats event has pdev stats. As per my other mail, 10.1 has
extra pdev stats. Since ath10k doesn't account that it reads peer
stats too early from the buffer. What you see is a tail of pdev stats
for 10.1.
You can try the following to test *10.1*:
--- a/drivers/net/wireless/ath/ath10k/wmi.h
+++ b/drivers/net/wireless/ath/ath10k/wmi.h
@@ -2804,6 +2804,12 @@ struct wmi_pdev_stats {
__le32 phy_err_count; /* Phy error count */
__le32 chan_tx_pwr; /* channel tx power */
struct wal_dbg_stats wal; /* WAL dbg stats */
+ __le32 ack_rx_bad;
+ __le32 rts_bad;
+ __le32 rts_good;
+ __le32 fcs_bad;
+ __le32 no_beacons;
+ __le32 mib_int_count;
} __packed;
Michał
_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
@ 2014-03-25 9:47 ` Michal Kazior
0 siblings, 0 replies; 22+ messages in thread
From: Michal Kazior @ 2014-03-25 9:47 UTC (permalink / raw)
To: Yeoh Chun-Yeow; +Cc: ath10k@lists.infradead.org, linux-wireless, Kalle Valo
On 25 March 2014 10:38, Yeoh Chun-Yeow <yeohchunyeow@gmail.com> wrote:
>> I don't think that's true for 10.1 firmware and AP interface types.
>
> I try to play around with 10.1 firmware as AP (3 connected STAs) and I
> get the following:
>
> The first WMI_UPDATE_STATS_EVENTID as follow:
> [ 338.290000] ath10k: pdev 0 vdev 0 peer 3
> [ 338.300000] ath10k: MAC 04:f0:21:0c:a5:44
> [ 338.300000] ath10k: RSSI 60
> [ 338.300000] ath10k: Tx 1170000
> [ 338.310000] ath10k: Rx 1170000
> [ 338.310000] ath10k: MAC 04:f0:21:0c:a5:19
> [ 338.320000] ath10k: RSSI 64
> [ 338.320000] ath10k: Tx 1300000
> [ 338.320000] ath10k: Rx 975000
> [ 338.320000] ath10k: MAC 04:f0:21:0c:a5:1c
> [ 338.330000] ath10k: RSSI 71
> [ 338.330000] ath10k: Tx 975000
> [ 338.330000] ath10k: Rx 1300000
>
> I think that above is correct.
>
> The second WMI_UPDATE_STATS_EVENTID as follow:
> [ 338.340000] ath10k: pdev 1 vdev 0 peer 1
> [ 338.350000] ath10k: MAC a8:02:00:00:00:00
> [ 338.350000] ath10k: RSSI 0
> [ 338.350000] ath10k: Tx 565
> [ 338.360000] ath10k: Rx 0
>
> Although indicating 1 peer, but the data seems to be not the peer
> stats, not even self STA stats. Any idea?
Yes.
The second stats event has pdev stats. As per my other mail, 10.1 has
extra pdev stats. Since ath10k doesn't account that it reads peer
stats too early from the buffer. What you see is a tail of pdev stats
for 10.1.
You can try the following to test *10.1*:
--- a/drivers/net/wireless/ath/ath10k/wmi.h
+++ b/drivers/net/wireless/ath/ath10k/wmi.h
@@ -2804,6 +2804,12 @@ struct wmi_pdev_stats {
__le32 phy_err_count; /* Phy error count */
__le32 chan_tx_pwr; /* channel tx power */
struct wal_dbg_stats wal; /* WAL dbg stats */
+ __le32 ack_rx_bad;
+ __le32 rts_bad;
+ __le32 rts_good;
+ __le32 fcs_bad;
+ __le32 no_beacons;
+ __le32 mib_int_count;
} __packed;
Michał
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
2014-03-25 9:47 ` Michal Kazior
@ 2014-03-25 10:31 ` Yeoh Chun-Yeow
-1 siblings, 0 replies; 22+ messages in thread
From: Yeoh Chun-Yeow @ 2014-03-25 10:31 UTC (permalink / raw)
To: Michal Kazior; +Cc: Kalle Valo, linux-wireless, ath10k@lists.infradead.org
>
> The second stats event has pdev stats. As per my other mail, 10.1 has
> extra pdev stats. Since ath10k doesn't account that it reads peer
> stats too early from the buffer. What you see is a tail of pdev stats
> for 10.1.
>
> You can try the following to test *10.1*:
>
> --- a/drivers/net/wireless/ath/ath10k/wmi.h
> +++ b/drivers/net/wireless/ath/ath10k/wmi.h
> @@ -2804,6 +2804,12 @@ struct wmi_pdev_stats {
> __le32 phy_err_count; /* Phy error count */
> __le32 chan_tx_pwr; /* channel tx power */
> struct wal_dbg_stats wal; /* WAL dbg stats */
> + __le32 ack_rx_bad;
> + __le32 rts_bad;
> + __le32 rts_good;
> + __le32 fcs_bad;
> + __le32 no_beacons;
> + __le32 mib_int_count;
> } __packed;
>
Ok. Probably can come out with a patch extending the pdev stats on this.
I get the self STA stats as follow on 10.1 firmware:
AP:
[ 754.280000] ath10k: pdev 1 vdev 0 peer 1
[ 754.280000] ath10k: MAC 04:f0:21:0c:a5:43
[ 754.280000] ath10k: RSSI 0
[ 754.290000] ath10k: Tx 6000
[ 754.290000] ath10k: Rx 0
STA:
[ 880.060000] ath10k: pdev 1 vdev 0 peer 1
[ 880.060000] ath10k: MAC 00:00:00:00:00:01
[ 880.070000] ath10k: RSSI 0
[ 880.070000] ath10k: Tx 6000
[ 880.070000] ath10k: Rx 0
Similar to 636. But on 636, I don't find any tx rate, only rssi.
---
Chun-Yeow
_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
@ 2014-03-25 10:31 ` Yeoh Chun-Yeow
0 siblings, 0 replies; 22+ messages in thread
From: Yeoh Chun-Yeow @ 2014-03-25 10:31 UTC (permalink / raw)
To: Michal Kazior; +Cc: ath10k@lists.infradead.org, linux-wireless, Kalle Valo
>
> The second stats event has pdev stats. As per my other mail, 10.1 has
> extra pdev stats. Since ath10k doesn't account that it reads peer
> stats too early from the buffer. What you see is a tail of pdev stats
> for 10.1.
>
> You can try the following to test *10.1*:
>
> --- a/drivers/net/wireless/ath/ath10k/wmi.h
> +++ b/drivers/net/wireless/ath/ath10k/wmi.h
> @@ -2804,6 +2804,12 @@ struct wmi_pdev_stats {
> __le32 phy_err_count; /* Phy error count */
> __le32 chan_tx_pwr; /* channel tx power */
> struct wal_dbg_stats wal; /* WAL dbg stats */
> + __le32 ack_rx_bad;
> + __le32 rts_bad;
> + __le32 rts_good;
> + __le32 fcs_bad;
> + __le32 no_beacons;
> + __le32 mib_int_count;
> } __packed;
>
Ok. Probably can come out with a patch extending the pdev stats on this.
I get the self STA stats as follow on 10.1 firmware:
AP:
[ 754.280000] ath10k: pdev 1 vdev 0 peer 1
[ 754.280000] ath10k: MAC 04:f0:21:0c:a5:43
[ 754.280000] ath10k: RSSI 0
[ 754.290000] ath10k: Tx 6000
[ 754.290000] ath10k: Rx 0
STA:
[ 880.060000] ath10k: pdev 1 vdev 0 peer 1
[ 880.060000] ath10k: MAC 00:00:00:00:00:01
[ 880.070000] ath10k: RSSI 0
[ 880.070000] ath10k: Tx 6000
[ 880.070000] ath10k: Rx 0
Similar to 636. But on 636, I don't find any tx rate, only rssi.
---
Chun-Yeow
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
2014-03-25 8:11 ` Michal Kazior
@ 2014-03-25 16:09 ` Ben Greear
-1 siblings, 0 replies; 22+ messages in thread
From: Ben Greear @ 2014-03-25 16:09 UTC (permalink / raw)
To: Michal Kazior
Cc: Chun-Yeow Yeoh, linux-wireless, Kalle Valo,
ath10k@lists.infradead.org
On 03/25/2014 01:11 AM, Michal Kazior wrote:
> On 25 March 2014 08:55, Chun-Yeow Yeoh <yeohchunyeow@gmail.com> wrote:
>> This patch is intended to fix the problem if we use the
>> firmware 999.999.0.636 to get peer stats when the number
>> of peer is more than 3. The WMI_UPDATE_STATS_EVENTID may
>> trigger more than 1 time if the number of peers is more
>> than 3. So this patch allows us to do the checking on this
>> and make sure that we print the peer stats correctly.
>>
>> Signed-off-by: Chun-Yeow Yeoh <yeohchunyeow@gmail.com>
>> ---
>> drivers/net/wireless/ath/ath10k/debug.c | 9 ++++++++-
>> 1 file changed, 8 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/net/wireless/ath/ath10k/debug.c b/drivers/net/wireless/ath/ath10k/debug.c
>> index 7be284c..4a4072e 100644
>> --- a/drivers/net/wireless/ath/ath10k/debug.c
>> +++ b/drivers/net/wireless/ath/ath10k/debug.c
>> @@ -245,10 +245,17 @@ void ath10k_debug_read_target_stats(struct ath10k *ar,
>> if (num_peer_stats) {
>> struct wmi_peer_stats_10x *peer_stats;
>> struct ath10k_peer_stat *s;
>> + int j = 0;
>> +
>> + if (!test_bit(ATH10K_FW_FEATURE_WMI_10X, ar->fw_features) &&
>> + !num_pdev_stats) {
>> + j = 3;
>> + num_peer_stats += 3;
>> + }
>>
>> stats->peers = num_peer_stats;
>>
>> - for (i = 0; i < num_peer_stats; i++) {
>> + for (i = j; i < num_peer_stats; i++) {
>> peer_stats = (struct wmi_peer_stats_10x *)tmp;
>> s = &stats->peer_stat[i];
>
> 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).
At least in my 10.x 389 firmware, the limit is 11 peers. There is some
round-robin logic that means next request of stats gets the next set of peer
stats, so calling get-stats multiple times would get the entire list of peers.
I'm not sure how to deal with this in the driver. In my case, my multiple
stations mostly connect to the same AP, so peer-mac is often the same.
This makes it difficult for me to know if the round-robin logic is working
properly, but I think we might as well assume it does until proven otherwise.
I may try to poke a station identifier into the 16 unused bits of the mac-addr
struct eventually, but unless upstream firmware does the same, that doesn't
help any general issue...
Thanks,
Ben
--
Ben Greear <greearb@candelatech.com>
Candela Technologies Inc http://www.candelatech.com
_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
@ 2014-03-25 16:09 ` Ben Greear
0 siblings, 0 replies; 22+ messages in thread
From: Ben Greear @ 2014-03-25 16:09 UTC (permalink / raw)
To: Michal Kazior
Cc: Chun-Yeow Yeoh, ath10k@lists.infradead.org, linux-wireless,
Kalle Valo
On 03/25/2014 01:11 AM, Michal Kazior wrote:
> On 25 March 2014 08:55, Chun-Yeow Yeoh <yeohchunyeow@gmail.com> wrote:
>> This patch is intended to fix the problem if we use the
>> firmware 999.999.0.636 to get peer stats when the number
>> of peer is more than 3. The WMI_UPDATE_STATS_EVENTID may
>> trigger more than 1 time if the number of peers is more
>> than 3. So this patch allows us to do the checking on this
>> and make sure that we print the peer stats correctly.
>>
>> Signed-off-by: Chun-Yeow Yeoh <yeohchunyeow@gmail.com>
>> ---
>> drivers/net/wireless/ath/ath10k/debug.c | 9 ++++++++-
>> 1 file changed, 8 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/net/wireless/ath/ath10k/debug.c b/drivers/net/wireless/ath/ath10k/debug.c
>> index 7be284c..4a4072e 100644
>> --- a/drivers/net/wireless/ath/ath10k/debug.c
>> +++ b/drivers/net/wireless/ath/ath10k/debug.c
>> @@ -245,10 +245,17 @@ void ath10k_debug_read_target_stats(struct ath10k *ar,
>> if (num_peer_stats) {
>> struct wmi_peer_stats_10x *peer_stats;
>> struct ath10k_peer_stat *s;
>> + int j = 0;
>> +
>> + if (!test_bit(ATH10K_FW_FEATURE_WMI_10X, ar->fw_features) &&
>> + !num_pdev_stats) {
>> + j = 3;
>> + num_peer_stats += 3;
>> + }
>>
>> stats->peers = num_peer_stats;
>>
>> - for (i = 0; i < num_peer_stats; i++) {
>> + for (i = j; i < num_peer_stats; i++) {
>> peer_stats = (struct wmi_peer_stats_10x *)tmp;
>> s = &stats->peer_stat[i];
>
> 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).
At least in my 10.x 389 firmware, the limit is 11 peers. There is some
round-robin logic that means next request of stats gets the next set of peer
stats, so calling get-stats multiple times would get the entire list of peers.
I'm not sure how to deal with this in the driver. In my case, my multiple
stations mostly connect to the same AP, so peer-mac is often the same.
This makes it difficult for me to know if the round-robin logic is working
properly, but I think we might as well assume it does until proven otherwise.
I may try to poke a station identifier into the 16 unused bits of the mac-addr
struct eventually, but unless upstream firmware does the same, that doesn't
help any general issue...
Thanks,
Ben
--
Ben Greear <greearb@candelatech.com>
Candela Technologies Inc http://www.candelatech.com
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
2014-03-25 7:55 ` Chun-Yeow Yeoh
@ 2014-03-28 12:47 ` Kalle Valo
-1 siblings, 0 replies; 22+ messages in thread
From: Kalle Valo @ 2014-03-28 12:47 UTC (permalink / raw)
To: Chun-Yeow Yeoh; +Cc: linux-wireless, ath10k
Chun-Yeow Yeoh <yeohchunyeow@gmail.com> writes:
> This patch is intended to fix the problem if we use the
> firmware 999.999.0.636 to get peer stats when the number
> of peer is more than 3. The WMI_UPDATE_STATS_EVENTID may
> trigger more than 1 time if the number of peers is more
> than 3. So this patch allows us to do the checking on this
> and make sure that we print the peer stats correctly.
>
> Signed-off-by: Chun-Yeow Yeoh <yeohchunyeow@gmail.com>
Based on Michal's comments I'm going to drop this one, we need to find a
better way to solve this.
--
Kalle Valo
_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] ath10k: fix printing of peer stats in non-AP firmware
@ 2014-03-28 12:47 ` Kalle Valo
0 siblings, 0 replies; 22+ messages in thread
From: Kalle Valo @ 2014-03-28 12:47 UTC (permalink / raw)
To: Chun-Yeow Yeoh; +Cc: ath10k, linux-wireless
Chun-Yeow Yeoh <yeohchunyeow@gmail.com> writes:
> This patch is intended to fix the problem if we use the
> firmware 999.999.0.636 to get peer stats when the number
> of peer is more than 3. The WMI_UPDATE_STATS_EVENTID may
> trigger more than 1 time if the number of peers is more
> than 3. So this patch allows us to do the checking on this
> and make sure that we print the peer stats correctly.
>
> Signed-off-by: Chun-Yeow Yeoh <yeohchunyeow@gmail.com>
Based on Michal's comments I'm going to drop this one, we need to find a
better way to solve this.
--
Kalle Valo
^ permalink raw reply [flat|nested] 22+ messages in thread
end of thread, other threads:[~2014-03-28 12:47 UTC | newest]
Thread overview: 22+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-03-25 7:55 [PATCH] ath10k: fix printing of peer stats in non-AP firmware Chun-Yeow Yeoh
2014-03-25 7:55 ` Chun-Yeow Yeoh
2014-03-25 8:11 ` Michal Kazior
2014-03-25 8:11 ` Michal Kazior
2014-03-25 8:33 ` Yeoh Chun-Yeow
2014-03-25 8:33 ` Yeoh Chun-Yeow
2014-03-25 8:45 ` Michal Kazior
2014-03-25 8:45 ` Michal Kazior
2014-03-25 8:54 ` Yeoh Chun-Yeow
2014-03-25 8:54 ` Yeoh Chun-Yeow
2014-03-25 8:57 ` Michal Kazior
2014-03-25 8:57 ` Michal Kazior
2014-03-25 9:38 ` Yeoh Chun-Yeow
2014-03-25 9:38 ` Yeoh Chun-Yeow
2014-03-25 9:47 ` Michal Kazior
2014-03-25 9:47 ` Michal Kazior
2014-03-25 10:31 ` Yeoh Chun-Yeow
2014-03-25 10:31 ` Yeoh Chun-Yeow
2014-03-25 16:09 ` Ben Greear
2014-03-25 16:09 ` Ben Greear
2014-03-28 12:47 ` Kalle Valo
2014-03-28 12:47 ` Kalle Valo
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.