Re: [PATCH] wifi: brcmfmac: avoid sleeping tx locks in netpoll context

From: Arend van Spriel

Date: Tue Oct 06 2026 - 04:27:43 EST


On 06/10/2026 06:18, Karl Mehltretter wrote:
On Sun, Oct 04, 2026 at 05:29:10AM +0100, Arend van Spriel wrote:
Is there an opportunity to reuse/refactor brcmf_sdio_prec_enq() rather than
adding a separate brcmf_sdio_prec_enq_irqoff() helper? For example, passing
a can_evict parameter:


Thanks for the review!

Sharing the helper with a can_evict argument is cleaner.

One queue user does not hold txq_lock: brcmf_fws_bus_txq_cleanup()
dequeues through brcmu_pktq_pdeq_match().

Lockdep instrumentation in fcmode=1 and 2 confirmed that cleanup enters
without txq_lock.

This is an observation unrelated to the patch, right? You mentioned in the patch it only covers fcmode=0, right?

I combined the helpers, but kept brcmu_pktq_penq() for normal calls.
Sure.

Regards,
Arend