Re: [PATCH 1/7] workqueue: rename worker_pool->assoc_mutex to ->manager_mutex

2013-03-14 Thread Tejun Heo
Hey, Lai. On Thu, Mar 14, 2013 at 9:00 AM, Lai Jiangshan wrote: >> @@ -4095,7 +4109,7 @@ static void wq_unbind_fn(struct work_struct *work) >> pool->flags |= POOL_DISASSOCIATED; >> >> spin_unlock_irq(&pool->lock); >> - mutex_unlock(&pool->assoc_mutex)

Re: [PATCH 1/7] workqueue: rename worker_pool->assoc_mutex to ->manager_mutex

2013-03-14 Thread Lai Jiangshan
On Thu, Mar 14, 2013 at 10:57 AM, Tejun Heo wrote: > Manager operations are currently governed by two mutexes - > pool->manager_arb and ->assoc_mutex. The former is used to decide who > gets to be the manager and the latter to exclude the actual manager > operations including creation and destruc

[PATCH 1/7] workqueue: rename worker_pool->assoc_mutex to ->manager_mutex

2013-03-13 Thread Tejun Heo
Manager operations are currently governed by two mutexes - pool->manager_arb and ->assoc_mutex. The former is used to decide who gets to be the manager and the latter to exclude the actual manager operations including creation and destruction of workers. Anyone who grabs ->manager_arb must perfor