diff mbox

[v3] iw: avgerage ack rssi changes for data frames

Message ID 1520493581-26708-1-git-send-email-bpothuno@codeaurora.org (mailing list archive)
State Not Applicable
Delegated to: Johannes Berg
Headers show

Commit Message

Balaji Pothunoori March 8, 2018, 7:19 a.m. UTC
This patch is to display the average ack rssi for data
frames. "avg ack signal" field diplay limited by host based on
firmware capablities.

Example output:

localhost ~ #iw dev wlan-5000mhz station dump
Station 34:f3:9a:aa:3b:29 (on wlan-5000mhz)
        inactive time:  5370 ms
        rx bytes:       85321
        rx packets:     576
        tx bytes:       14225
        tx packets:     71
        tx retries:     0
        tx failed:      2
        beacon loss:    0
        rx drop misc:   0
        signal:         -54 dBm
        signal avg:     -53 dBm
        tx bitrate:     866.7 MBit/s VHT-MCS 9 80MHz short GI VHT-NSS 2
        rx bitrate:     866.7 MBit/s VHT-MCS 9 80MHz short GI VHT-NSS 2
        avg ack signal: -56 dBm
        authorized:     yes
        authenticated:  yes
        associated:     yes
        preamble:       short
        WMM/WME:        yes
        MFP:            no
        TDLS peer:      no
        DTIM period:    2
        beacon interval:100
        short preamble: yes
        short slot time:yes
        connected time: 203 seconds

Signed-off-by: Balaji Pothunoori <bpothuno@codeaurora.org>
---
V2:
   -Removed nl80211.h changes and modified the commit log.
V3:
   -Adding version.
---
 station.c | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Johannes Berg March 21, 2018, 9:44 a.m. UTC | #1
On Thu, 2018-03-08 at 12:49 +0530, Balaji Pothunoori wrote:
> This patch is to display the average ack rssi for data
> frames. "avg ack signal" field diplay limited by host based on
> firmware capablities.

I no longer have a corresponding kernel patch pending, so this is
useless?

johannes
Johannes Berg May 23, 2018, 10 a.m. UTC | #2
On Thu, 2018-03-08 at 12:49 +0530, Balaji Pothunoori wrote:
> This patch is to display the average ack rssi for data
> frames. "avg ack signal" field diplay limited by host based on
> firmware capablities.

This doesn't compile, please respin (and you could fix the "average"
typo in the subject)

johannes
Balaji Pothunoori May 23, 2018, 12:45 p.m. UTC | #3
Sure Johannes, I will send another version of patch.

On 2018-05-23 15:30, Johannes Berg wrote:
> On Thu, 2018-03-08 at 12:49 +0530, Balaji Pothunoori wrote:
>> This patch is to display the average ack rssi for data
>> frames. "avg ack signal" field diplay limited by host based on
>> firmware capablities.
> 
> This doesn't compile, please respin (and you could fix the "average"
> typo in the subject)
> 
> johannes
diff mbox

Patch

diff --git a/station.c b/station.c
index f3e3da8..6c567bc 100644
--- a/station.c
+++ b/station.c
@@ -237,6 +237,7 @@  static int print_sta_handler(struct nl_msg *msg, void *arg)
 		[NL80211_STA_INFO_TID_STATS] = { .type = NLA_NESTED },
 		[NL80211_STA_INFO_BSS_PARAM] = { .type = NLA_NESTED },
 		[NL80211_STA_INFO_RX_DURATION] = { .type = NLA_U64 },
+		[NL80211_STA_INFO_DATA_AVG_ACK_SIGNAL] = { .type = NLA_U8 },
 	};
 	char *chain;
 
@@ -338,6 +339,9 @@  static int print_sta_handler(struct nl_msg *msg, void *arg)
 		printf("\n\trx duration:\t%lld us",
 		       (unsigned long long)nla_get_u64(sinfo[NL80211_STA_INFO_RX_DURATION]));
 
+	if (sinfo[NL80211_STA_INFO_DATA_AVG_ACK_SIGNAL])
+		printf("\n\tavg ack signal:\t%d dBm",
+		       (int8_t)nla_get_u8(sinfo[NL80211_STA_INFO_DATA_AVG_ACK_SIGNAL]));
 	if (sinfo[NL80211_STA_INFO_EXPECTED_THROUGHPUT]) {
 		uint32_t thr;