diff options
author | Lai Jiangshan <laijs@cn.fujitsu.com> | 2014-06-03 15:33:27 +0800 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2014-06-19 15:43:33 -0400 |
commit | f05b558d7e0368e0511a098c4beb9654e41246c4 (patch) | |
tree | 46e5a4baeb2fb82ceacfe3e8db263be12b1819ec /net/hsr | |
parent | fb009e3a998e48fb33436bf9b563c82801a7329c (diff) |
workqueue: clear POOL_DISASSOCIATED in rebind_workers()
The commit a9ab775bcadf ("workqueue: directly restore CPU affinity of
workers from CPU_ONLINE") moved the pool->lock into rebind_workers()
without also moving "pool->flags &= ~POOL_DISASSOCIATED".
There is nothing wrong with "pool->flags &= ~POOL_DISASSOCIATED" not
being moved together, but there isn't any benefit either. We move it
into rebind_workers() and achieve these benefits:
1) Better readability. POOL_DISASSOCIATED is cleared in
rebind_workers() as expected.
2) When POOL_DISASSOCIATED is cleared, we can ensure that all the
running workers of the pool are on the local CPU (pool->cpu).
tj: Cosmetic updates to the code and description.
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'net/hsr')
0 files changed, 0 insertions, 0 deletions