From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0b-0031df01.pphosted.com (mx0b-0031df01.pphosted.com [205.220.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4BCC9317147 for ; Wed, 15 Apr 2026 03:16:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.180.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776222967; cv=none; b=aZP80d2rOAoxmJcvlZAl6xWtzVLRuYvbyVDiVwLjxiZBY14kzvui9qFIFLzrX9BBSKuFRwUL7jp7bUplfSPw9Zi6JJa9EvbtgoXhYgyKHRmqSJcfw/Md/q+v0K+Q3/0Ilgp+ITHwKaXG0PgCHLIkfNx+TddeFC8KACsTZBPYjCY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776222967; c=relaxed/simple; bh=ypT6dvJbVlTDEt1n6AY6fbHaty4dOtTSID2Kk1+7IKc=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=ResBsaUVSaUvcGuyRx0qGmWtm7LSqcxcrwMnWvR5CfNKle2+k9fuQog+/5gLRDyzHL2TMym6h8SJ6kHxpUeR5VI8kgbdQZWGcFWVuCBEpjAC+vabWwa3fDZs8b5BQSE6FzeHvYk3k2VfjMCIJ+gdt9kcEt0ECyTHTVFHLPbuj2E= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=TPgveN5k; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=iacqPlmJ; arc=none smtp.client-ip=205.220.180.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="TPgveN5k"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="iacqPlmJ" Received: from pps.filterd (m0279873.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 63EGr6G1030017 for ; Wed, 15 Apr 2026 03:16:05 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= yAotFAiqJiGXdaNX/JI+JOyH84jDiUIJy+533r0Cdeo=; b=TPgveN5kaPfd2c0S /fQsg6oUmfmg0McWRDIFM0cGbvmN9XS5Hh+ExrzHF+xkeuPdAWcbclRk4RtiEkfd yGicujRDwusINCGSmi2w26sveK8QJJwtuby7tLHcp/GSXECHNn8oD7C599QzoJU5 nUOyaqavBnRdfWUxHP2kOoS0WO7mtfQkSwsJ8etq9oZp4QKJeoemHDBT5GobgTVm MXRKWE/4KsXdKqFH2ZDTCvaqjBpE90UVa8f59oNZziY1QTfAuFFSkVJIfqQyCXS2 x5wDik6C2Q62uyriPuX6Ov0VWpRuoj2PvWJna5bjtDgJd+dGk2NMt1VMLjWZGOE1 hHm3EA== Received: from mail-pj1-f70.google.com (mail-pj1-f70.google.com [209.85.216.70]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4dhkgmk41d-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Wed, 15 Apr 2026 03:16:05 +0000 (GMT) Received: by mail-pj1-f70.google.com with SMTP id 98e67ed59e1d1-35dacc40f92so6498027a91.2 for ; Tue, 14 Apr 2026 20:16:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1776222964; x=1776827764; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:content-language:from :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=yAotFAiqJiGXdaNX/JI+JOyH84jDiUIJy+533r0Cdeo=; b=iacqPlmJJ/ojJE7i8WlMvS/U7f69UD4GeN2w1toMdKQartNKoCpsuRKfpb7pb4OOvQ 0W0CyJ9Q2UEhIXeDTxHoWEFaBLSHJaa7EVYSeq1jjKHwcwfFcdiTz3HiGKb96a5jzGR1 BxHdLG/uz8hNT0ABthfbvhlhOQFYRPuRLeMzC5YU2R4MvX04VD474D5V5Yo2pLuA5mLG JbEEnDGEJofV4cJoKPLz4vOm3YWUHaQ6TCmCbuKclBTLuS0VeXL37bJc4bBG6qg1XYKL ruTPmaoGyC3A7lW3B2kf7GCCeN1ayRLgSqlJNJKi9t/1r7HeVjPcY29eNFu1lRVdIbh+ Ay6Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1776222964; x=1776827764; h=content-transfer-encoding:in-reply-to:content-language:from :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=yAotFAiqJiGXdaNX/JI+JOyH84jDiUIJy+533r0Cdeo=; b=SysD6I6CoSLIhECnFByH48HIxBrW7A65lDHwGJ0P2pJsuTGG4rQI2R3pWfbpfu7u5s vo0jJ0d+kE7hx6kFC61ZdxLilL/Ke3c0S7r7O6wwUgFXu5y4m8l0Er5p9sheT4urTM1s /t8uDs54cKZPttE5bNdA5h75O+hsv3QU7QYGnoqYWd9x1BwdCAufJERsSukSZpCooz1B Hs2OzPD7ASojuWtKgXNZexgeCs56CQWQQiEU2aEbkHk5hnKhTrDY6KW0gUwk9lsIZjVh 284fatYizzhzSqMN9AoLxHbDefeQ3WZh+Pwp3O8jnrlTlgLPk7N7970bYz420fZGL3cG ISKw== X-Forwarded-Encrypted: i=1; AFNElJ8/qUxYgs6/YU2FxWzz8LlmK7yAoJ5i7mAlIWj0GXJ1aaUtXciqKWuQqamssk9DbztfYEgv84rVaB7wvjA=@vger.kernel.org X-Gm-Message-State: AOJu0YzmFLtaLx+IhDo33P36//HzvLsFzmGvkFaXh/sduuJ28x7l4/7b L7IL09EGzJKUjOsOmbolEmI7JP0tLP5fnZDBMtD5uw85jPOTla626WoIf4N7Sww637NXzsaKckO 825r0tpC4zkvDgt6uyCYkEo72cPosbzH4c73pa9tGO1nnjQ4NSdtyd5tJSSuWc8vLN2A= X-Gm-Gg: AeBDieto0+hDiFyGcjjQ47jJT1lrxk+GD1XAj5x9Ry6kApVkDcDWLxq2SWIlExIdTwY G171Qcz3xNhjc2PKkAbHFgQi8Ikjh9+7YJnm0OBuB50pnjsF9bZymQz5wqxIvxT9mckeg7M+JTX dD60nGsqzlCQ0cIsd09EK4EQeU6lBH/PBU330EMUq8v6DtQLleno+9ZgXTxU6GPOYZbgYcvHbXr VnXT8F5jcSr67ykCHQk+HcJuvBZaGbzZyLbQkFAxoLc5x5XwdfgSElsoF08lOM6xt6MuLN6gaa+ kM9cW45XLKPCbqdLmxkLc5IF6EznNxJuP7nZ5FBnaU6Gp/m+dzq7ttYKgwdm0hS7OmxIMhwRHhH ebj8C61MU/yvFf3os4Zmu6J7wjSoKMkMgvukH2vTt45GmFRgzJNrLB+zO187xRZ3DFRE7rAHMvZ LoOgyu1d9Am3v+hgXkdz+nJNtBjvPhRg== X-Received: by 2002:a17:90b:4c4e:b0:35d:a2d3:5c31 with SMTP id 98e67ed59e1d1-35e42858252mr21452647a91.29.1776222964155; Tue, 14 Apr 2026 20:16:04 -0700 (PDT) X-Received: by 2002:a17:90b:4c4e:b0:35d:a2d3:5c31 with SMTP id 98e67ed59e1d1-35e42858252mr21452614a91.29.1776222963700; Tue, 14 Apr 2026 20:16:03 -0700 (PDT) Received: from [10.133.33.118] (tpe-colo-wan-fw-bordernet.qualcomm.com. [103.229.16.4]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-35fd3203a76sm357994a91.13.2026.04.14.20.16.01 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 14 Apr 2026 20:16:03 -0700 (PDT) Message-ID: Date: Wed, 15 Apr 2026 11:16:00 +0800 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] ath11k: workaround firmware bug where peer_id=0 To: Matthew Leach Cc: Jeff Johnson , linux-wireless@vger.kernel.org, ath11k@lists.infradead.org, linux-kernel@vger.kernel.org, kernel@collabora.com References: <20260326-ath11k-null-peerid-workaround-v1-1-0c2fd53202f8@collabora.com> <87h5pxlpg4.fsf@collabora.com> <7dbc3836-c42c-4cbb-a50a-011d82a0ee81@oss.qualcomm.com> <87a4v54s88.fsf@collabora.com> From: Baochen Qiang Content-Language: en-US In-Reply-To: <87a4v54s88.fsf@collabora.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNDE1MDAyOCBTYWx0ZWRfX0nBfva6Mg3K9 yUIqJL+pQh+qhMPwVicgMYkFItlXHnZO49UyXL+UHF32jC+mRKM9o4x3bsfFqk97OIhiGGVoNvV E2NiWO+BBU7yct3xfpjc0xriwBu4y+MLSGGx/0ck3/VMQJ1yROzKo1UkEQN3q1ui0T86gRR9EZ4 5KxxmTi+8LmQOLyKB7VufDIAzEqUnw0Zq0caa8e7XY7U2GE2T7Sf1Sn0WHBpIEtBuSrqSAkoCIm 1ogMaiTVrVk/gxGGY1v+VgVf7Ar/6EyZV5U7d+DA8uJfxOmGtsBjixknQNotGFDrGRkOyb0D2xA Q1y78plXmJsOItcXQBlEqIG58EAcoT4o4u57YdeOMQDFIJnbLr86Iq/2x5bHQcEa+wTJ3UzjGQ0 8pNCxcRlkGMqegQP60sscrW+mWgzzog557pGeqt9rKgnwJT3cEqTArzlNFpQprxCVHhx8vIDXq6 bKR4n4TZJ9AFRsmkZuA== X-Proofpoint-ORIG-GUID: wKCTHurbzws4e1u0RjD255xOJVqprRXL X-Proofpoint-GUID: wKCTHurbzws4e1u0RjD255xOJVqprRXL X-Authority-Analysis: v=2.4 cv=HJLz0Itv c=1 sm=1 tr=0 ts=69df02f5 cx=c_pps a=0uOsjrqzRL749jD1oC5vDA==:117 a=nuhDOHQX5FNHPW3J6Bj6AA==:17 a=IkcTkHD0fZMA:10 a=A5OVakUREuEA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=rJkE3RaqiGZ5pbrm-msn:22 a=EUspDBNiAAAA:8 a=QX4gbG5DAAAA:8 a=p6f2LkNNnSkKDXlKyPwA:9 a=QEXdDO2ut3YA:10 a=mQ_c8vxmzFEMiUWkPHU9:22 a=AbAUZ8qAyYyZVLSsDulk:22 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.51,FMLib:17.12.100.49 definitions=2026-04-14_04,2026-04-13_04,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 lowpriorityscore=0 spamscore=0 bulkscore=0 priorityscore=1501 impostorscore=0 clxscore=1015 malwarescore=0 phishscore=0 adultscore=0 suspectscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2604070000 definitions=main-2604150028 On 4/14/2026 8:54 PM, Matthew Leach wrote: > Hi Baochen, > > Baochen Qiang writes: > >> On 3/30/2026 3:57 PM, Matthew Leach wrote: >>> Hello, >>> >>> Matthew Leach writes: >>> > > [...] > >> for chips like QCA2066 and WCN6855 etc 0 is a valid value, however >> this is not for chips like QCN9074 etc. >> >> so a possible fix would be to add hardware ops based on chips: for >> QCN9074 we keep the existing validation on 0 in the ops, while for >> QCA2066 the ops is a null func. Or even simper we can remove the >> validation for all chips. > > In that case, does it make sense to remove the condition check > > if (rxcb->peer_id) > > in ath11k_dp_rx_h_find_peer()? It looks like this has been used as a > small optimisation, where if peer_id isn't valid it skips checking for > it in the peer hash table. However, if on newer chips peer_id=0 is > valid, we should remove this? yeah, I think so. This check was also based on the non-zero peer id assumption. > > Regards,