Re: [PATCH 3/3] dmaengine: qcom: bam_dma: convert tasklet to a workqueue

[Bartosz Golaszewski]

On Sat, Nov 22, 2025 at 10:10 AM Vinod Koul <vkoul@xxxxxxxxxx> wrote:
>
> On 06-11-25, 16:44, Bartosz Golaszewski wrote:
> > From: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx>
> >
> > There is nothing in the interrupt handling that requires us to run in
> > atomic context so convert the tasklet to a workqueue.
>
> The reason dmaengine drivers use tasklets is higher priority to handle
> dma interrupts... This is by design for this subsystem.
>
> There was an attempt in past to do conversion away from tasklets [1]:
> We need something similar apprach here. I can queue up first two though
>
> 1: https://lore.kernel.org/all/20240327160314.9982-1-apais@xxxxxxxxxxxxxxxxxxx/
> --
> ~Vinod

Ah, I wasn't aware of this. I tested some simple use cases with QCE
and it worked so I assumed it's fine. Yes, please queue the first two
ones and I'll revisit this later.

Bart