From: Joonsoo Kim Date: Fri, 26 Oct 2012 14:03:49 +0000 (+0900) Subject: workqueue: add WARN_ON_ONCE() on CPU number to wq_worker_waking_up() X-Git-Tag: upstream/snapshot3+hdmi~6110^2 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3657600040a7279e52252af3f9d7e253f4f49ef0;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git workqueue: add WARN_ON_ONCE() on CPU number to wq_worker_waking_up() Recently, workqueue code has gone through some changes and we found some bugs related to concurrency management operations happening on the wrong CPU. When a worker is concurrency managed (!WORKER_NOT_RUNNIG), it should be bound to its associated cpu and woken up to that cpu. Add WARN_ON_ONCE() to verify this. Signed-off-by: Joonsoo Kim Signed-off-by: Tejun Heo --- diff --git a/kernel/workqueue.c b/kernel/workqueue.c index 26f5d16..ae9a056 100644 --- a/kernel/workqueue.c +++ b/kernel/workqueue.c @@ -739,8 +739,10 @@ void wq_worker_waking_up(struct task_struct *task, unsigned int cpu) { struct worker *worker = kthread_data(task); - if (!(worker->flags & WORKER_NOT_RUNNING)) + if (!(worker->flags & WORKER_NOT_RUNNING)) { + WARN_ON_ONCE(worker->pool->gcwq->cpu != cpu); atomic_inc(get_pool_nr_running(worker->pool)); + } } /**