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 48378C6FA82 for ; Thu, 8 Sep 2022 12:15:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:References :In-Reply-To:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=Q/FM+tHP1oA8/ig3QDlvh4TWbxt04Qc3WljZf3lS6Ms=; b=APXfFK39s6aw2N xuERmiOtgqIV7191SrOpfJ0+tmJBK0YDGPlvhKnAtdhuOP65T42bXQj1YumnIMp8Mp86xJYX77dAX 6O9BJw2NzSz0s3iNOrx/hF2/DTbC2c1pVLgFxQZ3XfTk4RvvUek1KyJbS0CydwQ9y/+//QsBi/6sA ZgTBejvROVHLvn6z5ewKVaNZOvYqIqjXCYfRoZ5r8QzpUI7ZLiPQ/2NOUBZeWIG1Fj39UA86cXYRu +zczIUMdKTXpYMGq0pAQzv94/DxRLJSHZ9JN2/wJtG3ZVwyKbtnyi3rnR00k3o2aWzLaRxT5lNgTt CrG8wbbl4ukD2yUcFedA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oWGRc-0037pN-4R; Thu, 08 Sep 2022 12:15:44 +0000 Received: from ams.source.kernel.org ([2604:1380:4601:e00::1]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oWGRY-0037mi-Q3 for ath11k@lists.infradead.org; Thu, 08 Sep 2022 12:15:42 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 77771B820C5; Thu, 8 Sep 2022 12:15:38 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 1F000C433C1; Thu, 8 Sep 2022 12:15:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1662639337; bh=7CYWYb+qA5ZDBgkdQGwl08oLEGOaV3jooXzyK0p8b/E=; h=From:To:Cc:Subject:In-Reply-To:References:Date:From; b=U0NnvlI/DeqtccTbKaeTkksnwSVr/ILYgSSh7X9pewsi7OOU9TMYfS3H8GeMNd+Qa tglqgg+L6qZF0ud49tCDkBty0HeIQF0iFwUJY6ITtcY9jBPIAKiYLuB4gqCbMn7HBQ H239kd7bR5sZJkzvHIdS6Z332v8HXcFPWQlI/jDb9Mrlktik7MQ0Pb6XocUMqUmjrt w/cmlV8GMPXnkLPLJD7jRYfRAytDuVBLxxfZYLdyX5XRAS3NUp2jywI5GeRhT0AogG pZbdBvWGXhYB7iu6sCDpzQ7OXbpWPUW4d7xuHG+wKE0a0pdSzN5Ibtj2d2E3HRp5+F VFH4u5Rp020BA== From: Kalle Valo To: Robert Marko Cc: Manikanta Pubbisetty , ath11k@lists.infradead.org, linux-wireless@vger.kernel.org Subject: Re: [PATCH 1/3] ath11k: Enable threaded NAPI on WCN6750 In-Reply-To: (Robert Marko's message of "Thu, 8 Sep 2022 10:49:30 +0200") References: <20220720135113.15755-1-quic_mpubbise@quicinc.com> <20220720135113.15755-2-quic_mpubbise@quicinc.com> <87czceaqng.fsf@kernel.org> <7615c815-ca2e-c6f1-b627-6e10f1272442@quicinc.com> <5c07bcd5-574c-b13b-44c2-ef8ed9f23cf0@quicinc.com> <878rmucqhu.fsf@kernel.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) Date: Thu, 08 Sep 2022 15:15:31 +0300 Message-ID: <87tu5i83ks.fsf@kernel.org> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220908_051541_045027_15B5676D X-CRM114-Status: GOOD ( 17.42 ) X-BeenThere: ath11k@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "ath11k" Errors-To: ath11k-bounces+ath11k=archiver.kernel.org@lists.infradead.org Robert Marko writes: > On Thu, Sep 8, 2022 at 8:46 AM Kalle Valo wrote: > >> >> Manikanta Pubbisetty writes: >> >> > On 9/2/2022 7:16 PM, Robert Marko wrote: >> >> On Fri, Sep 2, 2022 at 3:42 PM Manikanta Pubbisetty >> >> wrote: >> >>> >> >>> On 9/2/2022 6:50 PM, Robert Marko wrote: >> >>>> On Fri, Sep 2, 2022 at 3:18 PM Manikanta Pubbisetty >> >>>> wrote: >> >>>>> >> >>>>> 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? >> >>>> >> >>>> I can tell you that on IPQ8074 threaded NAPI really improves perfromance. >> >>>> >> >>> >> >>> Great. Do you have any test results on IPQ8074? >> >> >> >> I dont have full test results, but on Poco F2 Pro as the client @80MHz AX >> >> I got ~720Mbps compared to ~550Mbps before. >> >> >> >> I can tell you that in OpenWrt, we have had it enabled for 6+ months >> >> at this point >> >> and its been really good. >> >> >> > >> > That's a significant improvement, great to hear that. We have another >> > strong reason to have this change in upstream driver. >> >> Indeed, thanks Robert. Is the code in openwrt similar as what >> Manikanta's patch does? > > It's actually identical, we are also just calling dev_set_threaded > before napi_enable. Nice, even better :) -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches -- ath11k mailing list ath11k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath11k