Received: by 2002:a05:6359:c8b:b0:c7:702f:21d4 with SMTP id go11csp181209rwb; Wed, 28 Sep 2022 00:52:41 -0700 (PDT) X-Google-Smtp-Source: AMsMyM5tx9Q6fDDs26gz6V2V0X+AfQ4JiAa3/ZyVDlHNZd+tZXqTtfaWVRv/XB7g4TnNXiCrTQog X-Received: by 2002:a05:6402:3988:b0:44e:6f08:ddfb with SMTP id fk8-20020a056402398800b0044e6f08ddfbmr31534066edb.89.1664351561018; Wed, 28 Sep 2022 00:52:41 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1664351561; cv=none; d=google.com; s=arc-20160816; b=yyvCsPNH8cfEDOZgFVBGqpt92m4DOnx6DV3Jv15hrmSRWzImgPCOSWTVYnnOsfTMxp EZmrfPPnSXcu32XnmdiYTaZa8xdAC6rQ4FmqtfniC4QCj8YctwTu/Gw8yUHeIg4OvV62 Dsz2KFkDbVWxc/nN1o6daA95sZUwimoLxP7ffdq6HtBq7AcZJ0G30UDWQDQ3v0pDBk+r Et+KfaIrItzlEo3HQFxqgbYMfh4Ya4GGJ2vF9/0X3pSjL4qi9M9M/OB+tkdkf8e4JTu8 my3gCWNekd1zQMBoUPRJyjzDM13xpS3Q44cLFIegxsmrcczIx5nnuheTCyt1BfXLpEVd jHTw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:date:message-id:user-agent:cc:to:references :in-reply-to:from:subject:content-transfer-encoding:mime-version :dkim-signature; bh=h8zlJsU45aJaz+1D4r/Oin9kmCjV0CH8wlqtdCfHwRs=; b=T1N2+SJh2/yene5A96pE4LW7SWwYSjGK/kRwA9Tb+TGFCiLozk8YziYwuRRsukjJ2U D8dZNkmipcg+f9+tjrTl85q3V1t55z+JKE4r0/ABkwUa2ZIQmSzdb/UB6vBHwzwg06HK +pjy012rel3KnkHwsj/SbVpGSirAzjuuuTkg/U1Bd9W26u9I0Q+sbukf/l1cxFIdXDjN i4BEUctaKqqAep/ewiRygYGFdtEGF+EfIR79XiHBSTK6kdZJC4fMrCbQVJ5maXZBOP4f 4s6EDvDFZmifCkODfW9TQRJ9gMJ0/pgITQnt849MV/PexvlSKtLM+oI4MU2FeqaFMjsw pMOw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=IYgh5dEN; 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=kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id x20-20020a05640226d400b00451ba82d25esi4337568edd.333.2022.09.28.00.52.25; Wed, 28 Sep 2022 00:52:41 -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=@kernel.org header.s=k20201202 header.b=IYgh5dEN; 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=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232020AbiI1Hs4 (ORCPT + 59 others); Wed, 28 Sep 2022 03:48:56 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37024 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233881AbiI1Hsf (ORCPT ); Wed, 28 Sep 2022 03:48:35 -0400 Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6B5F6122076 for ; Wed, 28 Sep 2022 00:45:07 -0700 (PDT) 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 4663FB81E8A for ; Wed, 28 Sep 2022 07:45:05 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 04E85C433C1; Wed, 28 Sep 2022 07:45:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1664351103; bh=6SQSuRMC+R7kxqaZdTa6Rn0P4+/7LiIcZwMuBTCDzDc=; h=Subject:From:In-Reply-To:References:To:Cc:Date:From; b=IYgh5dEN6FoWcOdJgKFGwshdH5xYxJYWLajRdzgfyhmbgAQkkiaWhUCgOhGlCWBRv XjPRri4VGzN19lBWOX9svGTg2uVOJDZehHn1mx1XrroguzjNrRv15xQUBEDr549ooZ y5lKqr3oW52FAiurIeT16u7pckcAw1qoK4wY2q1Nvzr6tRoXrw3ip+6yR33USqSCwU u/jLOUeABAoioLCl2LgE5gJ+tMXjc9duu8ZIRJzxrwLMqCsQTvBu8IK8OsVylXMQjL /TBqrVqlMLxbCwuF7BxakpwWcEPS0XbdxcuV/1tdySoduHEBnpzh/Q5z/48owNeL3h plKoDqP+DsATg== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Re: [PATCH] wifi: ath11k: stop tx queues immediately upon firmware exit From: Kalle Valo In-Reply-To: <20220923170235.18873-1-quic_adisi@quicinc.com> References: <20220923170235.18873-1-quic_adisi@quicinc.com> To: Aditya Kumar Singh Cc: , , Aditya Kumar Singh User-Agent: pwcli/0.1.1-git (https://github.com/kvalo/pwcli/) Python/3.7.3 Message-ID: <166435109959.3623.15599848077820461188.kvalo@kernel.org> Date: Wed, 28 Sep 2022 07:45:02 +0000 (UTC) X-Spam-Status: No, score=-7.2 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_HI, SPF_HELO_NONE,SPF_PASS 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 Aditya Kumar Singh wrote: > Currently, recovery flag is set immediately upon firmware > exit but tx queues are stopped once firmware arrives back > and is ready which is during ath11k_core_restart. Once > ieee80211 hw restart is completed, tx queues are resumed. > If during the time delta between firmware exit and firmware > ready, mac80211 send packets, currently ath11k will drop it > since recovery flag will be set. But warning prints will > come - > "ath11k c000000.wifi: failed to transmit frame -108" > > If more tx packets are there, this could lead to flooding > of above print. > > However, actually tx queues should be stopped immediately > when firmware leaves. This will prevent packets to get > dropped when firmware is recovering. > > Add fix to stop tx queues immediately after firmware exit. > > Tested-on: IPQ8074 hw2.0 AHB WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1 > > Signed-off-by: Aditya Kumar Singh > Signed-off-by: Kalle Valo Patch applied to ath-next branch of ath.git, thanks. 81e60b2dfb27 wifi: ath11k: stop tx queues immediately upon firmware exit -- https://patchwork.kernel.org/project/linux-wireless/patch/20220923170235.18873-1-quic_adisi@quicinc.com/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches