Re: [RFC PATCH v2] usb: typec: ucsi: Move long delayed work on system_dfl_long_wq

[Greg Kroah-Hartman <gregkh@linuxfoundation.org>]

On Fri, May 08, 2026 at 04:38:53PM +0200, Marco Crivellari wrote:
> Currently the code enqueue work items using {queue|mod}_delayed_work(),
> using system_long_wq. This workqueue should be used when long works are
> expected and it is a per-cpu workqueue.
> 
> The function(s) end up calling __queue_delayed_work(), which set a global
> timer that could fire anywhere, enqueuing the work where the timer fired.
> 
> Unbound works could benefit from scheduler task placement, to optimize
> performance and power consumption. Long work shouldn't stick to a single
> CPU.
> 
> Recently, a new unbound workqueue specific for long running work has
> been added:
> 
>     c116737e972e ("workqueue: Add system_dfl_long_wq for long unbound works")
> 
> Since the workqueue work doesn't rely on per-cpu variables, there is no
> obvious reason that justify the use of a per-cpu workqueue. So change
> system_long_wq with system_dfl_long_wq so that the work may benefit from
> scheduler task placement.
> 
> Signed-off-by: Marco Crivellari <marco.crivellari@suse.com>
> ---
> Changes in v2:
> - Fixed a wrong change regarding system_long_wq used by queue_work() instead of queue_delayed_work()
> 
> Link to v1: https://lore.kernel.org/all/20260507142726.331206-1-marco.crivellari@suse.com/

What "comments" do you need resolved here before submitting this as a
real patch?

thanks,

greg k-h