2012-03-16 10:25:27

by Raja Mani

Subject: [PATCH] ath6kl: Add ARP offload related statistic info in tgt_stats

From: Raja Mani <[email protected]>

Firmware reports the below ARP offload related information
while sending the target statistic event to the host.

* Number of ARP packets received.
* Number of packets matched with the device IP addr.
* Number of ARP response packet sent to the remote.

This patch adds the additional debug prints in debugfs
entry tgt_stats. It will be useful to know the ARP offload
execution status.

Signed-off-by: Raja Mani <[email protected]>
---
drivers/net/wireless/ath/ath6kl/debug.c | 6 ++++++
drivers/net/wireless/ath/ath6kl/main.c | 4 ++++
2 files changed, 10 insertions(+), 0 deletions(-)

diff --git a/drivers/net/wireless/ath/ath6kl/debug.c b/drivers/net/wireless/ath/ath6kl/debug.c
index 56e1cb1..31cdca1 100755
--- a/drivers/net/wireless/ath/ath6kl/debug.c
+++ b/drivers/net/wireless/ath/ath6kl/debug.c
@@ -622,6 +622,12 @@ static ssize_t read_file_tgt_stats(struct file *file, char __user *user_buf,
"Num disconnects", tgt_stats->cs_discon_cnt);
len += scnprintf(buf + len, buf_len - len, "%20s %10d\n",
"Beacon avg rssi", tgt_stats->cs_ave_beacon_rssi);
+ len += scnprintf(buf + len, buf_len - len, "%20s %10d\n",
+ "ARP pkt received", tgt_stats->arp_received);
+ len += scnprintf(buf + len, buf_len - len, "%20s %10d\n",
+ "ARP pkt matched", tgt_stats->arp_matched);
+ len += scnprintf(buf + len, buf_len - len, "%20s %10d\n",
+ "ARP pkt replied", tgt_stats->arp_replied);

if (len > buf_len)
len = buf_len;
diff --git a/drivers/net/wireless/ath/ath6kl/main.c b/drivers/net/wireless/ath/ath6kl/main.c
index bd8388c..8c1b44a 100644
--- a/drivers/net/wireless/ath/ath6kl/main.c
+++ b/drivers/net/wireless/ath/ath6kl/main.c
@@ -756,6 +756,10 @@ static void ath6kl_update_target_stats(struct ath6kl_vif *vif, u8 *ptr, u32 len)
stats->wow_evt_discarded +=
le16_to_cpu(tgt_stats->wow_stats.wow_evt_discarded);

+ stats->arp_received = le32_to_cpu(tgt_stats->arp_stats.arp_received);
+ stats->arp_replied = le32_to_cpu(tgt_stats->arp_stats.arp_replied);
+ stats->arp_matched = le32_to_cpu(tgt_stats->arp_stats.arp_matched);
+
if (test_bit(STATS_UPDATE_PEND, &vif->flags)) {
clear_bit(STATS_UPDATE_PEND, &vif->flags);
wake_up(&ar->event_wq);
--
1.7.1

2012-03-20 09:36:30

by Kalle Valo

Subject: Re: [PATCH] ath6kl: Add ARP offload related statistic info in tgt_stats

On 03/16/2012 12:24 PM, [email protected] wrote:
> From: Raja Mani <[email protected]>
>
> Firmware reports the below ARP offload related information
> while sending the target statistic event to the host.
>
> * Number of ARP packets received.
> * Number of packets matched with the device IP addr.
> * Number of ARP response packet sent to the remote.
>
> This patch adds the additional debug prints in debugfs
> entry tgt_stats. It will be useful to know the ARP offload
> execution status.
>
> Signed-off-by: Raja Mani <[email protected]>

Thanks, applied.

Kalle