Re: [PATCH V3] workqueue: fix double unlock bug

2014-04-15 Thread DaeSeok Youn
Hi, Tejun. 2014-04-15 23:43 GMT+09:00 Tejun Heo : > On Tue, Apr 15, 2014 at 08:17:43AM +0900, Daeseok Youn wrote: >> >> Use default pwq when alloc_unbound_pwq() is failed. >> >> And remove "if" condition for whether "pwq" is same as "wq->dfl_pwq" >> when wq_calc_node_cpumask() returns false and

Re: [PATCH V3] workqueue: fix double unlock bug

2014-04-15 Thread Tejun Heo
On Tue, Apr 15, 2014 at 08:17:43AM +0900, Daeseok Youn wrote: > > Use default pwq when alloc_unbound_pwq() is failed. > > And remove "if" condition for whether "pwq" is same as "wq->dfl_pwq" > when wq_calc_node_cpumask() returns false and just use "goto use_dfl_pwq" > > Signed-off-by: Daeseok

Re: [PATCH V3] workqueue: fix double unlock bug

2014-04-15 Thread Tejun Heo
On Tue, Apr 15, 2014 at 08:17:43AM +0900, Daeseok Youn wrote: Use default pwq when alloc_unbound_pwq() is failed. And remove if condition for whether pwq is same as wq-dfl_pwq when wq_calc_node_cpumask() returns false and just use goto use_dfl_pwq Signed-off-by: Daeseok Youn

Re: [PATCH V3] workqueue: fix double unlock bug

2014-04-15 Thread DaeSeok Youn
Hi, Tejun. 2014-04-15 23:43 GMT+09:00 Tejun Heo t...@kernel.org: On Tue, Apr 15, 2014 at 08:17:43AM +0900, Daeseok Youn wrote: Use default pwq when alloc_unbound_pwq() is failed. And remove if condition for whether pwq is same as wq-dfl_pwq when wq_calc_node_cpumask() returns false and just

Re: [PATCH V3] workqueue: fix double unlock bug

2014-04-14 Thread Lai Jiangshan
Hi, Tejun Acked-by: Lai Jiangshan CC: sta...@kernel.org Thanks, Lai On 04/15/2014 07:17 AM, Daeseok Youn wrote: > > Use default pwq when alloc_unbound_pwq() is failed. > > And remove "if" condition for whether "pwq" is same as "wq->dfl_pwq" > when wq_calc_node_cpumask() returns false and

[PATCH V3] workqueue: fix double unlock bug

2014-04-14 Thread Daeseok Youn
Use default pwq when alloc_unbound_pwq() is failed. And remove "if" condition for whether "pwq" is same as "wq->dfl_pwq" when wq_calc_node_cpumask() returns false and just use "goto use_dfl_pwq" Signed-off-by: Daeseok Youn --- V2: replace "if condition" with "goto" as Lai's comment. V3: Use

[PATCH V3] workqueue: fix double unlock bug

2014-04-14 Thread Daeseok Youn
Use default pwq when alloc_unbound_pwq() is failed. And remove if condition for whether pwq is same as wq-dfl_pwq when wq_calc_node_cpumask() returns false and just use goto use_dfl_pwq Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- V2: replace if condition with goto as Lai's comment.

Re: [PATCH V3] workqueue: fix double unlock bug

2014-04-14 Thread Lai Jiangshan
Hi, Tejun Acked-by: Lai Jiangshan la...@cn.fujitsu.com CC: sta...@kernel.org Thanks, Lai On 04/15/2014 07:17 AM, Daeseok Youn wrote: Use default pwq when alloc_unbound_pwq() is failed. And remove if condition for whether pwq is same as wq-dfl_pwq when wq_calc_node_cpumask() returns false