[PATCH 2/2] ath10k: add per peer tx stats support for 10.2.4

Kalle Valo kvalo at qca.qualcomm.com
Thu Nov 30 08:38:54 PST 2017


Toke Høiland-Jørgensen <toke at toke.dk> writes:

>>> +struct ath10k_10_2_peer_tx_stats {
>>> +	u8 ratecode[PEER_STATS_FOR_NO_OF_PPDUS];
>>> +	u8 success_pkts[PEER_STATS_FOR_NO_OF_PPDUS];
>>> +	__le16 success_bytes[PEER_STATS_FOR_NO_OF_PPDUS];
>>> +	u8 retry_pkts[PEER_STATS_FOR_NO_OF_PPDUS];
>>> +	__le16 retry_bytes[PEER_STATS_FOR_NO_OF_PPDUS];
>>> +	u8 failed_pkts[PEER_STATS_FOR_NO_OF_PPDUS];
>>> +	__le16 failed_bytes[PEER_STATS_FOR_NO_OF_PPDUS];
>>> +	u8 flags[PEER_STATS_FOR_NO_OF_PPDUS];
>>> +	__le32 tx_duration;
>>> +	u8 tx_ppdu_cnt;
>>> +	u8 peer_id;
>>> +} __packed;
>>
>>Toke, hopefully the tx_duration value here helps with ATF
>>implementation
>>using QCA988X.
>
> Awesome! What's the semantics of this field? Just total 
> duration spent serving that station in the reporting interval?
> Does it include retry attempts?

I have no clue :) I just noticed this while I was reviewing the patch
internally and immediately recalled our discussions at Seoul. I can try
to find out, but that will take a long time as I have way too much stuff
pending at the moment. Hopefully someone more knowledgeable (Anilkumar?)
can chime in and help.

-- 
Kalle Valo


More information about the ath10k mailing list