]> git.karo-electronics.de Git - karo-tx-linux.git/commit
workqueue: UNBOUND -> REBIND morphing in rebind_workers() should be atomic
authorLai Jiangshan <laijs@cn.fujitsu.com>
Sat, 1 Sep 2012 16:28:19 +0000 (00:28 +0800)
committerTejun Heo <tj@kernel.org>
Wed, 5 Sep 2012 00:04:45 +0000 (17:04 -0700)
commit96e65306b81351b656835c15931d1d237b252f27
treeaf06187bebae44b48ca8e68a639a4ddc6b0a3509
parent0d7614f09c1ebdbaa1599a5aba7593f147bf96ee
workqueue: UNBOUND -> REBIND morphing in rebind_workers() should be atomic

The compiler may compile the following code into TWO write/modify
instructions.

worker->flags &= ~WORKER_UNBOUND;
worker->flags |= WORKER_REBIND;

so the other CPU may temporarily see worker->flags which doesn't have
either WORKER_UNBOUND or WORKER_REBIND set and perform local wakeup
prematurely.

Fix it by using single explicit assignment via ACCESS_ONCE().

Because idle workers have another WORKER_NOT_RUNNING flag, this bug
doesn't exist for them; however, update it to use the same pattern for
consistency.

tj: Applied the change to idle workers too and updated comments and
    patch description a bit.

Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Cc: stable@vger.kernel.org
kernel/workqueue.c