Re: [PATCH 0/2] wifi: ath10k: fix invalid VHT parameters

[James Prestwood <prestwoj@gmail.com>]

Hi Baochen,

On 8/19/24 2:45 AM, Baochen Qiang wrote:
>
> On 8/14/2024 11:46 PM, James Prestwood wrote:
>> Hi Baochen,
>>
>> On 7/15/24 5:38 AM, James Prestwood wrote:
>>> Hi Baochen,
>>>
>>> I've compiled both changes and so far those messages are gone.
>>>
>>> Tested-By: James Prestwood <prestwoj@gmail.com> # on QCA6174 hw3.2
>> I just noticed another one actually:
>>
>> invalid vht params rate 585 100kbps nss 1 mcs 4
> Ah :(
>
> James, could you share some info about this? like hardware target, firmware version, AP model, and any specific steps to hit this.

This is running with your fixes in this patch set, which did fix the 
majority of the "invalid vht params" messages, but then recently I 
noticed some others.

The hardware:

kernel: ath10k_pci 0000:02:00.0: qca6174 hw3.2 target 0x05030000 chip_id 
0x00340aff sub 168c:3363
kernel: ath10k_pci 0000:02:00.0: firmware ver WLAN.RM.4.4.1-00288- api 6 
features wowlan,ignore-otp,mfp crc32 bf907c7c

And its actually a few more than I thought:

ath10k_pci 0000:02:00.0: invalid vht params rate 900 100kbps nss 3 mcs 4
ath10k_pci 0000:02:00.0: invalid vht params rate 867 100kbps nss 2 mcs 6
ath10k_pci 0000:02:00.0: invalid vht params rate 540 100kbps nss 3 mcs 3
ath10k_pci 0000:02:00.0: invalid vht params rate 1500 100kbps nss 3 mcs 7

I unfortunately don't have a way to reproduce it. We just see this now 
somewhat rarely, e.g. 13 of the above in the last 3 days. I assume its 
just when the MCS/NSS changes and we hit a rate that isn't mapped right 
like the few you fixed.

Thanks,

James


>
>> Thanks,
>>
>> James
>>
>>> On 7/10/24 7:03 PM, Baochen Qiang wrote:
>>>> supported_vht_mcs_rate_nss1 and supported_vht_mcs_rate_nss2 in host do not
>>>> align with firmware's definitions, resulting in some warnings. Fix them.
>>>>
>>>> Baochen Qiang (2):
>>>>     wifi: ath10k: fix invalid VHT parameters in
>>>>       supported_vht_mcs_rate_nss1
>>>>     wifi: ath10k: fix invalid VHT parameters in
>>>>       supported_vht_mcs_rate_nss2
>>>>
>>>>    drivers/net/wireless/ath/ath10k/mac.c | 4 ++--
>>>>    1 file changed, 2 insertions(+), 2 deletions(-)
>>>>
>>>>
>>>> base-commit: db1ce56e6e1d395dd42a3cd6332a871d9be59c45