Re: [PATCH 2/2] workqueue: defer wake_up_process() outside pool->lock on hot paths

[Sebastian Andrzej Siewior]

On 2026-05-26 14:08:06 [-0400], Breno Leitao wrote:
> --- a/kernel/workqueue.c
> +++ b/kernel/workqueue.c
> @@ -2423,6 +2424,15 @@ static void __queue_work(int cpu, struct workqueue_struct *wq,
>  
>  out:
>  	raw_spin_unlock(&pool->lock);
> +	/*
> +	 * Issue the wakeup after dropping pool->lock to shorten the
> +	 * locked region on this hot enqueue path.  kick_pool_pick() did all
> +	 * of the work that required the lock (worker selection and
> +	 * wake_cpu setup); the wake_up_process() itself only needs to
> +	 * take the target rq->lock.
> +	 */
> +	if (wake_p)
> +		wake_up_process(wake_p);

What about using wake_q_add() to grab a worker(s) and then wake_up_q()
to wake the worker outside of the lock?

>  	rcu_read_unlock();
>  }
>  

Sebastian