[PATCH ath-next v2] wifi: ath12k: Add support for transmit histogram stats

Vasanthakumar Thiagarajan quic_vthiagar at quicinc.com
Thu Jun 26 20:44:00 PDT 2025



On 6/26/2025 11:55 PM, Jeff Johnson wrote:
> On 6/26/2025 11:18 AM, Jeff Johnson wrote:
>> On 6/26/2025 10:20 AM, Vasanthakumar Thiagarajan wrote:
>>> On 6/17/2025 4:51 PM, Roopni Devanathan wrote:
>>>> From: Saleemuddin Shaik <quic_saleemud at quicinc.com>
>>>> +struct ath12k_htt_tx_histogram_stats_tlv {
>>>> +	__le32 rate_retry_mcs_drop_cnt;
>>>> +	__le32 mcs_drop_rate[ATH12K_HTT_TX_PDEV_STATS_NUM_MCS_DROP_COUNTERS];
>>>> +	__le32 per_histogram_cnt[ATH12K_HTT_TX_PDEV_STATS_NUM_PER_COUNTERS];
>>>> +	__le32 low_latency_rate_cnt;
>>>> +	__le32 su_burst_rate_drop_cnt;
>>>> +	__le32 su_burst_rate_drop_fail_cnt;
>>>> +};
>>>
>>> Since this strcuture represents the message format used between host and firmware, pls add
>>> __packed annotation even though it may not have any effect in this case.
>>
>> looks like I missed this in a few other structs as well, at least:
>> ath12k_htt_tx_pdev_rate_stats_tlv
>> ath12k_htt_rx_pdev_rate_ext_stats_tlv
>>
>> perhaps i'll take this as-is and then have another patch that adds __packed to
>> all _tlv structs that are missing it?
> 
> actually let me fix this instance in pending, and we can fix other existing
> ones separately

Sounds good, thanks.


Vasanth



More information about the ath12k mailing list