Received: by 2002:a05:6358:45e:b0:b5:b6eb:e1f9 with SMTP id 30csp2030939rwe; Fri, 2 Sep 2022 07:35:18 -0700 (PDT) X-Google-Smtp-Source: AA6agR7T2adTZU2jqqsQRlId6Ou5SZMtabOBEXzlwfImBmMHz536udJKRRazg7KEtQyj0ouXWjLd X-Received: by 2002:a17:906:7315:b0:741:5b1b:920d with SMTP id di21-20020a170906731500b007415b1b920dmr19894262ejc.484.1662129318601; Fri, 02 Sep 2022 07:35:18 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1662129318; cv=none; d=google.com; s=arc-20160816; b=SH8ii5cIS1fvbBg7xIsiH4PRFhNoH8ldi/fLQZ3olnkCNmwENEUy2ifdfNEsVvrklC o/t9qwcmlx+ELQeTG8GSuT/OZg8QZs7DHpSFMC/ei9/r6F6ky+0NnPeQ11ig3jISmA2d GRN/pTFMGV2MzjoSvSQq9R9wVCi58PwbMlZr84z7IU/7IwTpbOofZD28X9ZZGtysuYqz maVcgtIHKLA9+HUTHVcdv422TRuq1VPLkXODo8CjTn2pBzdjT+fyUChEr/6kF+a1Up1Z nDtHQmzQBVf7rIS2K3Zy5V8Th4s2ZeKR4gLgtCt5tmXyuEu6MGp6ja/rAcf0+47Lc1K3 UQIg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :date:message-id:dkim-signature; bh=yrVgbjKyM9x851YnG1EHLsKb2RvPpHD7kwjSNJSkK1k=; b=r4492iLNSHHqpiNJWCB5sW8YoaxrwfGxYp6WvbSutzxbBnFqCxlCnYd76fSZK0SKbv IprByMm1O7G2b+sPI1x4PoGH488tqXC9pSdGRDSyU6MCC+2XnNgXR2Kw7zbnS8+HnckJ OCur/3EZs/TZuy2nIm9lgH3TZtGqvaMapaPTYubSLaZc5UILe/+Qyr9Inam643IqHgxn mKkdo4IJXgPbhAsXahLOLf/qFPrlQGe4SuDAZe/tbD2PxDecW+rWI0j9x7Ea2R3HfWt3 cKMXuONabtY5HJ2J0bEsxdj3woSVRrfiUCxpuAotozQ36hnmi02lKrd2V/Z/1ZnvXzmh EnbQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@quicinc.com header.s=qcppdkim1 header.b=kZDaeADi; spf=pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-wireless-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=quicinc.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id 8-20020a508e08000000b0043d54ba0a4dsi1638798edw.327.2022.09.02.07.34.59; Fri, 02 Sep 2022 07:35:18 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@quicinc.com header.s=qcppdkim1 header.b=kZDaeADi; spf=pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-wireless-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=quicinc.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236657AbiIBOWs (ORCPT + 64 others); Fri, 2 Sep 2022 10:22:48 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42992 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236270AbiIBOW2 (ORCPT ); Fri, 2 Sep 2022 10:22:28 -0400 Received: from mx0a-0031df01.pphosted.com (mx0a-0031df01.pphosted.com [205.220.168.131]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 67CF23A2 for ; Fri, 2 Sep 2022 06:48:15 -0700 (PDT) Received: from pps.filterd (m0279867.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.17.1.5/8.17.1.5) with ESMTP id 282COIFP004632; Fri, 2 Sep 2022 13:17:47 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=quicinc.com; h=message-id : date : mime-version : subject : to : cc : references : from : in-reply-to : content-type : content-transfer-encoding; s=qcppdkim1; bh=yrVgbjKyM9x851YnG1EHLsKb2RvPpHD7kwjSNJSkK1k=; b=kZDaeADivx9XEuo2UNDo+jP6as6/QV1D99TINh/yGu44DvZF3QNYvnpXaPtbjJp1d3iQ emDnMkRnLNrQFWEai2l+y/H9WqA8WcEnEhGcX84h8yBk0nTm+AMLsE1lEgUtZNTBUgfZ RP+qohDkJClv2n2tUIENi9kt3ZMmQGn+C92eJqtuA6oYbRRjTuPucUuFCT9WrCQO+Q49 kjSjcRNTXhpH4gepgtqt39qeD1232jZ1QA53LHnNwyX3mK4i863JyCgv8Jcc8RRPnG/M sFv6ShcWkIhk037EWsH1ghKDKYP0qwWclfNqrXsi34gB3/E7Ko+5fN/WmZMGLouc8rpT tw== Received: from nalasppmta03.qualcomm.com (Global_NAT1.qualcomm.com [129.46.96.20]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 3jbdqqgy98-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Fri, 02 Sep 2022 13:17:47 +0000 Received: from nalasex01a.na.qualcomm.com (nalasex01a.na.qualcomm.com [10.47.209.196]) by NALASPPMTA03.qualcomm.com (8.17.1.5/8.17.1.5) with ESMTPS id 282DHkvl029579 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Fri, 2 Sep 2022 13:17:46 GMT Received: from [10.216.25.75] (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.986.29; Fri, 2 Sep 2022 06:17:44 -0700 Message-ID: Date: Fri, 2 Sep 2022 18:47:41 +0530 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.8.1 Subject: Re: [PATCH 1/3] ath11k: Enable threaded NAPI on WCN6750 Content-Language: en-US To: Kalle Valo CC: , References: <20220720135113.15755-1-quic_mpubbise@quicinc.com> <20220720135113.15755-2-quic_mpubbise@quicinc.com> <87czceaqng.fsf@kernel.org> From: Manikanta Pubbisetty In-Reply-To: <87czceaqng.fsf@kernel.org> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit 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-ORIG-GUID: oDOx8m2aFnlwYjcQ4X_jsyYr6EwrTW-1 X-Proofpoint-GUID: oDOx8m2aFnlwYjcQ4X_jsyYr6EwrTW-1 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.205,Aquarius:18.0.895,Hydra:6.0.517,FMLib:17.11.122.1 definitions=2022-09-02_02,2022-08-31_03,2022-06-22_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 suspectscore=0 phishscore=0 clxscore=1015 impostorscore=0 adultscore=0 malwarescore=0 mlxlogscore=999 lowpriorityscore=0 spamscore=0 priorityscore=1501 bulkscore=0 mlxscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2207270000 definitions=main-2209020063 X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_LOW, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org On 9/2/2022 6:18 PM, Kalle Valo wrote: > Manikanta Pubbisetty writes: > >> Enable threaded NAPI on WCN6750. Unlike traditional NAPI poll which >> runs in softirq context and on the core which scheduled the NAPI, >> threaded NAPI makes use of kernel threads which are under direct >> control of the scheduler and helps in balancing the NAPI processing >> load across multiple CPUs thereby improving throughput. >> >> In the case of WCN6750, enabling threaded NAPI has improved >> 160 MHz RX throughput by nearly 400 Mbps. This should give similar >> gains for other ath11k devices as well, therefore enable threaded >> NAPI on all other devices. >> >> Tested-on: WCN6750 hw1.0 AHB WLAN.MSL.1.0.1-00887-QCAMSLSWPLZ-1 >> >> Signed-off-by: Manikanta Pubbisetty >> --- >> drivers/net/wireless/ath/ath11k/ahb.c | 1 + >> drivers/net/wireless/ath/ath11k/pcic.c | 1 + >> 2 files changed, 2 insertions(+) >> >> diff --git a/drivers/net/wireless/ath/ath11k/ahb.c b/drivers/net/wireless/ath/ath11k/ahb.c >> index d7d33d5cdfc5..e44e2f29a88f 100644 >> --- a/drivers/net/wireless/ath/ath11k/ahb.c >> +++ b/drivers/net/wireless/ath/ath11k/ahb.c >> @@ -314,6 +314,7 @@ static void ath11k_ahb_ext_irq_enable(struct ath11k_base *ab) >> struct ath11k_ext_irq_grp *irq_grp = &ab->ext_irq_grp[i]; >> >> if (!irq_grp->napi_enabled) { >> + dev_set_threaded(&irq_grp->napi_ndev, true); >> napi_enable(&irq_grp->napi); >> irq_grp->napi_enabled = true; >> } >> diff --git a/drivers/net/wireless/ath/ath11k/pcic.c b/drivers/net/wireless/ath/ath11k/pcic.c >> index cf12b98c480d..c703db19de51 100644 >> --- a/drivers/net/wireless/ath/ath11k/pcic.c >> +++ b/drivers/net/wireless/ath/ath11k/pcic.c >> @@ -440,6 +440,7 @@ void ath11k_pcic_ext_irq_enable(struct ath11k_base *ab) >> struct ath11k_ext_irq_grp *irq_grp = &ab->ext_irq_grp[i]; >> >> if (!irq_grp->napi_enabled) { >> + dev_set_threaded(&irq_grp->napi_ndev, true); >> napi_enable(&irq_grp->napi); >> irq_grp->napi_enabled = true; >> } > > The commit log claims that this enabled _only_ on WCN6750 but aren't we > enabling it on all ath11k hardware, or am I missing something? I admit I > didn't check this very carefully. > > (reads the commit log one more time) > > Ah, in the last sentence you mention that it's enabled on all hardware. > That's quite easy to miss and the commit log is quite misleading, please > emphasise already in the title and the first sentence that this is for > all hardware. My Bad, yes you right. The patch was made initially only for WCN6750 and was enabled later for all devices. > > Also more testing would be nice. Enabling something like this with > testing only on one hardware family (WCN7850) can be risky. I always get > warm fuzzy feelings if a patch is tested with all three hardware > families we currently support: > > * IPQ8074 etc > * QCA6390 etc > * WCN7850 > WCN7850 should be an ath12k device If I'm correct. Regardless of the chip family, even I feel that the tput changes like these should be tested on all the chipsets. Availability of the hardware and time are something which are stopping me in testing the changes on all supported targets. As I said, I had made the changes only to WCN6750 initially (using a hw_param). Can we take that approach for now and enable this for other targets only if required & upon thorough testing? Thanks, Manikanta