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;
MIME-Version: 1.0
References: <20220915043527.737133-1-venkatch@gmail.com> <238a21a8-c97b-5a38-6c08-9057055bf73f@nbd.name>
 <CABHL29H75U-0dBP14d-6ds-xMPt1eqrvQPuMp3oY8qQS8Y3Dbw@mail.gmail.com> <9acc4159-8223-bbca-a83f-d075660ac6db@nbd.name>
In-Reply-To: <9acc4159-8223-bbca-a83f-d075660ac6db@nbd.name>
From:   Venkat Ch <venkatch@gmail.com>
Date:   Tue, 20 Sep 2022 23:57:43 +0530
Message-ID: <CABHL29GB6DCrdu4FzRfBDSj9A7pwqnWE+wgH2xfkBMLjT3bAxA@mail.gmail.com>
Subject: Re: [PATCH v2] wifi: mac80211: Fix performance issue with mutex_lock
To:     Felix Fietkau <nbd@nbd.name>
Cc:     johannes@sipsolutions.net, linux-wireless@vger.kernel.org,
        Venkat Chimata <venkata@shasta.cloud>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

Hi Felix,

 Thank you. I browsed through the code. There seems to be sleep
operations wcn36xx platform.ath11k  does n't seem to have the sleep
operations in sta_statistics. We are using ath11k based chipset. Will
it impact things if we apply this patch for ath11k boards only as a
platform specific patch?

Thanks & Regards
Venkat

Venkat

On Tue, 20 Sept 2022 at 14:40, Felix Fietkau <nbd@nbd.name> wrote:
>
>
> On 20.09.22 11:05, Venkat Ch wrote:
> > Hi Felix,
> > Thanks for the comments. What part of  drv_sta_statistics could make it
> > sleep? I might be missing something. Please explain.
> It's just a wrapper around the driver .sta_statistics op.
> The documentation in include/net/mac80211.h states that this callback is
> allowed to sleep, and it does so in a few drivers.
>
> - Felix


--
If you rest, you rust