GitHub-Mirror/Star64_linux
2
0
Fork 0
mirror of https://github.com/Fishwaldo/Star64_linux.git synced 2025-04-03 04:54:00 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

workqueue: use system_highpri_wq for highpri workers in rebind_workers()

Browse source 
In rebind_workers(), we do inserting a work to rebind to cpu for busy workers.
Currently, in this case, we use only system_wq. This makes a possible
error situation as there is mismatch between cwq->pool and worker->pool.

To prevent this, we should use system_highpri_wq for highpri worker
to match theses. This implements it.

tj: Rephrased comment a bit.

Signed-off-by: Joonsoo Kim <js1304@gmail.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
This commit is contained in:
Joonsoo Kim 2012-08-15 23:25:40 +09:00 • committed by Tejun Heo
parent 1aabe902ca
commit e2b6a6d570
1 changed files with 14 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
kernel/workqueue.c
View file

@ -1741,6 +1741,7 @@ retry:
/* rebind busy workers */ /* rebind busy workers */
for_each_busy_worker(worker, i, pos, gcwq) { for_each_busy_worker(worker, i, pos, gcwq) {
struct work_struct *rebind_work = &worker->rebind_work; struct work_struct *rebind_work = &worker->rebind_work;
struct workqueue_struct *wq;
/* morph UNBOUND to REBIND */ /* morph UNBOUND to REBIND */
worker->flags &= ~WORKER_UNBOUND; worker->flags &= ~WORKER_UNBOUND;
@ -1750,9 +1751,18 @@ retry:
work_data_bits(rebind_work))) work_data_bits(rebind_work)))
continue; continue;
/* wq doesn't matter, use the default one */
debug_work_activate(rebind_work); debug_work_activate(rebind_work);
insert_work(get_cwq(gcwq->cpu, system_wq), rebind_work,
/*
* wq doesn't really matter but let's keep @worker->pool
* and @cwq->pool consistent for sanity.
*/
if (worker_pool_pri(worker->pool))
wq = system_highpri_wq;
else
wq = system_wq;
insert_work(get_cwq(gcwq->cpu, wq), rebind_work,
worker->scheduled.next, worker->scheduled.next,
work_color_to_flags(WORK_NO_COLOR)); work_color_to_flags(WORK_NO_COLOR));
} }