Re: [RFC PATCH 2/7] RDMA/rxe: Convert the triple tasklets to workqueues

From: Bart Van Assche
Date: Sun Sep 11 2022 - 11:09:06 EST


On 9/11/22 00:10, Yanjun Zhu wrote:
I also implemented a workqueue for rxe. IMO, can we add a variable to
decide to use tasklet or workqueue?

If user prefer using tasklet, he can set the variable to use
tasklet. And the default is tasklet. Set the variable to another
value to use workqueue.

I'm in favor of removing all uses of the tasklet mechanism because of the disadvantages of that mechanism. See also:
* "Eliminating tasklets" (https://lwn.net/Articles/239633/).
* "Modernizing the tasklet API" (https://lwn.net/Articles/830964/).
* Sebastian Andrzej Siewior's opinion about tasklets (https://lore.kernel.org/all/YvovfXMJQAUBsvBZ@xxxxxxxxxxxxx/).

Thanks,

Bart.