diff options
| author | Lai Jiangshan <laijs@cn.fujitsu.com> | 2014-07-26 12:03:59 +0800 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2014-07-28 14:07:52 -0400 |
| commit | ed1403ec2bd463050e481cd29be40c504021676c (patch) | |
| tree | db9574ae09b6a593d8a3e239f532abf1dcfa7c3a /kernel/kthread.c | |
| parent | ddcb57e2ed0a4d0de5aef06735dd9df98894f818 (diff) | |
| download | op-kernel-dev-ed1403ec2bd463050e481cd29be40c504021676c.zip op-kernel-dev-ed1403ec2bd463050e481cd29be40c504021676c.tar.gz | |
kthread_work: wake up worker only when the worker is idle
If the worker is already executing a work item when another is queued,
we can safely skip wakeup without worrying about stalling queue thus
avoiding waking up the busy worker spuriously. Spurious wakeups
should be fine but still isn't nice and avoiding it is trivial here.
tj: Updated description.
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'kernel/kthread.c')
| -rw-r--r-- | kernel/kthread.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/kthread.c b/kernel/kthread.c index c2390f4..ef48322 100644 --- a/kernel/kthread.c +++ b/kernel/kthread.c @@ -591,7 +591,7 @@ static void insert_kthread_work(struct kthread_worker *worker, list_add_tail(&work->node, pos); work->worker = worker; - if (likely(worker->task)) + if (!worker->current_work && likely(worker->task)) wake_up_process(worker->task); } |
