summaryrefslogtreecommitdiffstats
path: root/kernel/workqueue.c
diff options
context:
space:
mode:
authorLai Jiangshan <laijs@linux.alibaba.com>2021-12-07 15:35:38 +0800
committerTejun Heo <tj@kernel.org>2021-12-09 12:15:41 -1000
commit3e5f39ea33b1189ccaa4ae2a9de2bce07753d2e0 (patch)
tree1c9324ccc1c6477a438b6989cacc9f9ca982b682 /kernel/workqueue.c
parentccf45156fd167a234baf038c11c1f367c7ccabd4 (diff)
downloadlinux-3e5f39ea33b1189ccaa4ae2a9de2bce07753d2e0.tar.bz2
workqueue: Remove the advanced kicking of the idle workers in rebind_workers()
The commit 6d25be5782e4 ("sched/core, workqueues: Distangle worker accounting from rq lock") changed the schedule callbacks for workqueue and removed the local-wake-up functionality. Now the wakingup of workers is done by normal fashion and workers not yet migrated to the specific CPU in concurrency managed pool can also be woken up by workers that already bound to the specific cpu now. So this advanced kicking of the idle workers to migrate them to the associated CPU is unneeded now. Signed-off-by: Lai Jiangshan <laijs@linux.alibaba.com> Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'kernel/workqueue.c')
-rw-r--r--kernel/workqueue.c11
1 files changed, 0 insertions, 11 deletions
diff --git a/kernel/workqueue.c b/kernel/workqueue.c
index 2964dbb783fe..f7f4a5fc7736 100644
--- a/kernel/workqueue.c
+++ b/kernel/workqueue.c
@@ -5058,17 +5058,6 @@ static void rebind_workers(struct worker_pool *pool)
unsigned int worker_flags = worker->flags;
/*
- * A bound idle worker should actually be on the runqueue
- * of the associated CPU for local wake-ups targeting it to
- * work. Kick all idle workers so that they migrate to the
- * associated CPU. Doing this in the same loop as
- * replacing UNBOUND with REBOUND is safe as no worker will
- * be bound before @pool->lock is released.
- */
- if (worker_flags & WORKER_IDLE)
- wake_up_process(worker->task);
-
- /*
* We want to clear UNBOUND but can't directly call
* worker_clr_flags() or adjust nr_running. Atomically
* replace UNBOUND with another NOT_RUNNING flag REBOUND.