workqueue: remove the confusing POOL_FREEZING
authorLai Jiangshan <laijs@cn.fujitsu.com>
Thu, 22 May 2014 11:01:16 +0000 (19:01 +0800)
committerTejun Heo <tj@kernel.org>
Thu, 22 May 2014 15:35:51 +0000 (11:35 -0400)
commit74b414ead1133972817d3ce7b934356150d03a7d
tree8942db6fda58c213804b66f8825fd736366b1597
parent1037de36edae30f1ddc0a2532decd50f92ac4901
workqueue: remove the confusing POOL_FREEZING

Currently, the global freezing state is propagated to worker_pools via
POOL_FREEZING and then to each workqueue; however, the middle step -
propagation through worker_pools - can be skipped as long as one or
more max_active adjustments happens for each workqueue after the
update to the global state is visible.  The global workqueue freezing
state and the max_active adjustments during workqueue creation and
[un]freezing are serialized with wq_pool_mutex, so it's trivial to
guarantee that max_actives stay in sync with global freezing state.

POOL_FREEZING is unnecessary and makes the code more confusing and
complicates freeze_workqueues_begin() and thaw_workqueues() by
requiring them to walk through all pools.

Remove POOL_FREEZING and use workqueue_freezing directly instead.

tj: Description and comment updates.

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