From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 10B35C3271F for ; Fri, 5 Jul 2024 10:52:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:CC:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=vV7WKNqKvNgemIkPR9GZ68xW1u4zL1TG81X9HLZ+FHQ=; b=4x/P53PULv1AfqceHI6YE/Rkg8 wDZOwcxeo1fRDGrZumbFuhBp77cbIZgS2WmIcVVTN5L7uHO9qNbaoNf8qPa9mU0uID9fG458+Qv/V 4yGsg9ouWPaF0hUJjmnwEbo/CxltRNt9GrGIbFUVYnx7n8w7Udq/xfPLUEjBY5b97XdUi8SjcaZte FRQbSCXnFm89DIfOdRqOXudpj1xXD4N0oqXqVg2Bf09Kn/4qwjo0Jzf9UVa22/n2F4q1NrIhGe9mG NQGj2BndlH1lOlyNWyheZHFU1wYL6AABrqt6j840FywsQEzYgNrB9SI+k0V1zNgYVw7bNBJBF4km8 M9ZeOi7w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sPgXv-0000000Fha2-2ZIO; Fri, 05 Jul 2024 10:52:07 +0000 Received: from mx0b-0031df01.pphosted.com ([205.220.180.131]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sPgXs-0000000FhZK-0XAq for ath10k@lists.infradead.org; Fri, 05 Jul 2024 10:52:05 +0000 Received: from pps.filterd (m0279870.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 4659nmmK019469; Fri, 5 Jul 2024 10:51:53 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=quicinc.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= vV7WKNqKvNgemIkPR9GZ68xW1u4zL1TG81X9HLZ+FHQ=; b=L1NGrAF7EvuHETsK 7pT7Wo7iI4+W5J2w0IgA4TT93Yplp/op42JPNqXf4dhN0vTx1czLrdO/jg54ALqC JcGu2F0MFFQexjeOSAyFimkyAZQRpXmtaDYiNvFsR7ZRH7J5LErCozfMkLXveDfH vep24C4+GhUGFI2SQzk4zAKT/S7+ssz2on65PFL/ftHeVp0aJ30bojQy/9CRbjLd 7NXKaJJNFXqiSoRdiT4qdabAv2AJ0mDI61vi/G3niPhzUAR15qej1RRQnyDTIBsJ JItJqeZawfPrwY5HTpeqXchpo15pvli6BL9T2KGjHNS1hhRK2wlf70yHPpobkRX0 xNtaKg== Received: from nalasppmta05.qualcomm.com (Global_NAT1.qualcomm.com [129.46.96.20]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 402abtwntj-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Fri, 05 Jul 2024 10:51:52 +0000 (GMT) Received: from nalasex01a.na.qualcomm.com (nalasex01a.na.qualcomm.com [10.47.209.196]) by NALASPPMTA05.qualcomm.com (8.17.1.19/8.17.1.19) with ESMTPS id 465Appha019368 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Fri, 5 Jul 2024 10:51:51 GMT Received: from [10.231.195.67] (10.80.80.8) by nalasex01a.na.qualcomm.com (10.47.209.196) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1544.9; Fri, 5 Jul 2024 03:51:49 -0700 Message-ID: Date: Fri, 5 Jul 2024 18:51:47 +0800 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: invalid vht params rate 1920 100kbps nss 2 mcs 9 To: Paul Menzel CC: Kalle Valo , James Prestwood , , , LKML , Chun Wu References: <1faa7eee-ed1e-477b-940d-a5cf4478cf73@gmail.com> <87iky7mvxt.fsf@kernel.org> <37ba6cb0-d887-4fcf-b7dc-c93a5fc5900f@gmail.com> <875xu6mtgh.fsf@kernel.org> <082024ce-fdd4-4fb1-8055-6d25f7d2e524@molgen.mpg.de> <462c97dc-f366-4f75-9327-04d9424b819a@quicinc.com> Content-Language: en-US From: Baochen Qiang In-Reply-To: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8bit X-Originating-IP: [10.80.80.8] X-ClientProxiedBy: nasanex01a.na.qualcomm.com (10.52.223.231) To nalasex01a.na.qualcomm.com (10.47.209.196) X-QCInternal: smtphost X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=5800 signatures=585085 X-Proofpoint-GUID: rg1MTPe4QVEOxcKSbw92swn7r98a5fyx X-Proofpoint-ORIG-GUID: rg1MTPe4QVEOxcKSbw92swn7r98a5fyx X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1039,Hydra:6.0.680,FMLib:17.12.28.16 definitions=2024-07-05_06,2024-07-05_01,2024-05-17_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 bulkscore=0 priorityscore=1501 mlxscore=0 clxscore=1015 suspectscore=0 malwarescore=0 lowpriorityscore=0 phishscore=0 adultscore=0 spamscore=0 impostorscore=0 mlxlogscore=999 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.19.0-2406140001 definitions=main-2407050080 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240705_035204_313903_5F883F0C X-CRM114-Status: GOOD ( 32.32 ) X-BeenThere: ath10k@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "ath10k" Errors-To: ath10k-bounces+ath10k=archiver.kernel.org@lists.infradead.org On 7/5/2024 2:55 PM, Paul Menzel wrote: > Dear Baochen, > > > Am 05.07.24 um 04:47 schrieb Baochen Qiang: > >> On 6/26/2024 5:12 PM, Paul Menzel wrote: > >>> Am 26.06.24 um 10:53 schrieb Baochen Qiang: >>> >>>> On 6/18/2024 6:33 PM, Kalle Valo wrote: >>>>> + baochen >>>>> >>>>> James Prestwood writes: >>> >>>>>> On 6/17/24 8:27 AM, Kalle Valo wrote: >>>>>>> James Prestwood writes: >>> >>>>>>>> On 6/16/24 6:10 AM, Paul Menzel wrote: > >>>>>>>>> Linux 6.10-rc3 (commit a3e18a540541) logged the warning below when >>>>>>>>> connecting to a public WiFi: >>>>>>>>> >>>>>>>>>        ath10k_pci 0000:3a:00.0: invalid vht params rate 1920 100kbps nss 2 mcs 9 >>>>>>>> >>>>>>>> This has been reported/discussed [1]. It was hinted that there was a >>>>>>>> firmware fix for this, but none that I tried got rid of it. I got fed >>>>>>>> up enough with the logs filling up with this I patched our kernel to >>>>>>>> remove the warning. AFAICT it appears benign (?). Removing the warning >>>>>>>> was purely "cosmetic" so other devs stopped complaining about it :) >>>>>>>> >>>>>>>> [1] https://www.mail-archive.com/ath10k@lists.infradead.org/msg13406.html >>>>>>> >>>>>>> More reliable link to the discussion: >>>>>>> >>>>>>> https://lore.kernel.org/ath10k/76a816d983e6c4d636311738396f97971b5523fb.1612915444.git.skhan@linuxfoundation.org/ >>>>>>> >>>>>>> I think we should add this workaround I mentioned in 2021: >>>>>>> >>>>>>>       "If the firmware still keeps sending invalid rates we should add a >>>>>>>        specific check to ignore the known invalid values, but not all of >>>>>>>        them." >>>>>>> >>>>>>>       https://lore.kernel.org/ath10k/87h7mktjgi.fsf@codeaurora.org/ >>>>>>> >>>>>>> I guess that would be mcs == 7 and rate == 1440? >>>>>> >>>>>> I think its more than this combination (Paul's are different). >>>>> >>>>> Good point. >>>>> >>>>>> So how many combinations are we willing to add here? Seems like that >>>>>> could get out of hand if there are more than a few invalid >>>>>> combinations. >>>>> >>>>> Yeah, but there haven't been that many different values reported yet, >>>>> right? And I expect that ath10k user base will just get smaller in the >>>>> future so the chances are that we will get less reports. >>>>> >>>>>> Would we also want to restrict the workaround to specific >>>>>> hardware/firmware? >>>>> >>>>> Good idea, limiting per hardware would be simple to implement using >>>>> hw_params. Of course we could even limit this per firmware version using >>>>> enum ath10k_fw_features, but not sure if that's worth all the extra work. >>>>> >>>>> Baochen, do you know more about this firmware bug? Any suggestions? >>>> >>>> OK, there are two issues here: >>>> >>>> 1. invalid HT rate: "ath10k_pci 0000:02:00.0: invalid ht params rate 1440 100kbps nss 2 mcs 7". >>>> >>>> As commented by Wen quite some time ago, this has been fixed from >>>> firmware side, and firmware newer than [ver:241] has the fix >>>> included. >>> This is the issue from 2021, correct? >>> >>>> 2. invaid VHT rate: "ath10k_pci 0000:3a:00.0: invalid vht params rate 1920 100kbps nss 2 mcs 9". >>>> >>>> After checking with firmware team, I thought this is because there is >>>> a mismatch in rate definition between host and firmware: In host, the >>>> rate for 'nss 2 mcs 9' is defined as {1560, 1733}, see >>>> supported_vht_mcs_rate_nss2[]. While in firmware this is defined as >>>> {1730, 1920}. So seems we can update host definition to avoid this >>>> issue. >>> Looking through the logs since May 2024, I have four different logs: >>> >>> 1.  invalid vht params rate 878 100kbps nss 3 mcs 2 >> >> which chip are you using when you hit this nss 3 issue? QCA6174 >> firmware does not support NSS 3 so really weird. > > This is all from the same device Dell XPS 13 9360 with QCA6174 and firmware 288. > > ``` > Mai 20 12:07:09 abreu kernel: Linux version 6.9.0-09705-g08b269af52c0 (build@bohemianrhapsody.molgen.mpg.de) (gcc (Debian 13.2.0-23) 13.2.0, GNU ld (GNU Binutils for Debian) 2. > 42) #147 SMP PREEMPT_DYNAMIC Mon May 20 07:33:23 CEST 2024 > […] > Mai 20 12:07:11 abreu kernel: ath10k_pci 0000:3a:00.0: firmware ver WLAN.RM.4.4.1-00288- api 6 features wowlan,ignore-otp,mfp crc32 bf907c7c > […] > Mai 20 15:37:55 abreu wpa_supplicant[613]: wlp58s0: Trying to associate with e2:b3:70:83:01:af (SSID='public' freq=5500 MHz) > […] > Mai 20 15:37:55 abreu kernel: wlp58s0: authenticate with e2:b3:70:83:01:af (local address=9c:b6:d0:d1:6a:b1) > Mai 20 15:37:55 abreu kernel: wlp58s0: send auth to e2:b3:70:83:01:af (try 1/3) > Mai 20 15:37:55 abreu kernel: wlp58s0: authenticated > Mai 20 15:37:55 abreu kernel: wlp58s0: associate with e2:b3:70:83:01:af (try 1/3) > Mai 20 15:37:55 abreu kernel: wlp58s0: RX AssocResp from e2:b3:70:83:01:af (capab=0x1501 status=0 aid=4) > […] > Mai 20 15:39:29 abreu wpa_supplicant[613]: wlp58s0: CTRL-EVENT-SIGNAL-CHANGE above=1 signal=-55 noise=-97 txrate=300000 > […] > Mai 20 15:54:44 abreu kernel: ath10k_pci 0000:3a:00.0: invalid vht params rate 878 100kbps nss 3 mcs 2 > ``` > > It was some public WiFi in some restaurant. No idea, what hardware they use. Maybe you can deduce this from the MAC address. Then it is QCA6174 definitely. Checked with firmware team and just know that, the TX rate info is generated by firmware directly but for RX rate it is from phy side. From firmware TX rate generation code seems NSS 3 is an impossible value, so it might be an RX rate generated by phy side. But I could not tell for now since the log is not complete. Paul, could you enable full ath10k log and try to reproduce? With full log we can check whether it is a RX rate issue, > >>> 2.  invalid vht params rate 960 100kbps nss 1 mcs 9 >>> 3.  invalid vht params rate 1730 100kbps nss 2 mcs 9 >>> 4.  invalid vht params rate 1920 100kbps nss 2 mcs 9 >> >> OK, these are due to mismatch between host and QCA6174 firmware, we >> can update host to fix them. Kalle, the root cause to these three warnings are clear now and if you agree I can submit patches to fix them. Or I can also wait until the NSS 3 issue is clear. > > Nice. If there would be a test framework to test this, so I do not have to search for a Cisco network, that’d be great. > >>> I believe it’s only happening with Cisco networks. I am happy to test a patch. > > […] > > > Kind regards, > > Paul