Re: [PATCH 01/10] workqueue: use manager lock only to protect worker_idr

2014-05-06 Thread Lai Jiangshan
On Mon, May 5, 2014 at 9:01 PM, Tejun Heo wrote: > On Sun, Apr 27, 2014 at 12:08:56PM +0800, Lai Jiangshan wrote: >> worker_idr is highly bound to managers and is always/only accessed in manager >> lock context. So we don't need pool->lock for it. >> >> Signed-off-by: Lai Jiangshan > ... >> @@

Re: [PATCH 01/10] workqueue: use manager lock only to protect worker_idr

2014-05-06 Thread Lai Jiangshan
On Mon, May 5, 2014 at 9:01 PM, Tejun Heo t...@kernel.org wrote: On Sun, Apr 27, 2014 at 12:08:56PM +0800, Lai Jiangshan wrote: worker_idr is highly bound to managers and is always/only accessed in manager lock context. So we don't need pool-lock for it. Signed-off-by: Lai Jiangshan

Re: [PATCH 01/10] workqueue: use manager lock only to protect worker_idr

2014-05-05 Thread Tejun Heo
On Sun, Apr 27, 2014 at 12:08:56PM +0800, Lai Jiangshan wrote: > worker_idr is highly bound to managers and is always/only accessed in manager > lock context. So we don't need pool->lock for it. > > Signed-off-by: Lai Jiangshan ... > @@ -378,14 +367,14 @@ static void copy_workqueue_attrs(struct

Re: [PATCH 01/10] workqueue: use manager lock only to protect worker_idr

2014-05-05 Thread Tejun Heo
On Sun, Apr 27, 2014 at 12:08:56PM +0800, Lai Jiangshan wrote: worker_idr is highly bound to managers and is always/only accessed in manager lock context. So we don't need pool-lock for it. Signed-off-by: Lai Jiangshan la...@cn.fujitsu.com ... @@ -378,14 +367,14 @@ static void

[PATCH 01/10] workqueue: use manager lock only to protect worker_idr

2014-04-26 Thread Lai Jiangshan
worker_idr is highly bound to managers and is always/only accessed in manager lock context. So we don't need pool->lock for it. Signed-off-by: Lai Jiangshan --- kernel/workqueue.c | 34 ++ 1 files changed, 6 insertions(+), 28 deletions(-) diff --git

[PATCH 01/10] workqueue: use manager lock only to protect worker_idr

2014-04-26 Thread Lai Jiangshan
worker_idr is highly bound to managers and is always/only accessed in manager lock context. So we don't need pool-lock for it. Signed-off-by: Lai Jiangshan la...@cn.fujitsu.com --- kernel/workqueue.c | 34 ++ 1 files changed, 6 insertions(+), 28 deletions(-)