diff options
author | Lai Jiangshan <laijs@cn.fujitsu.com> | 2014-07-22 01:04:49 -0400 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2014-07-22 12:10:39 -0400 |
commit | 3fb1823c093ebe1869d34005837f64df64713780 (patch) | |
tree | f8c0a00abfc94ddb45550adeb6b91de8a930335e /kernel/workqueue.c | |
parent | 29b1cb416a2920fbc70041e4382920ae2d86f426 (diff) |
workqueue: remove the misnamed out_unlock label in get_unbound_pool()
After the locking was moved up to the caller of the get_unbound_pool(),
out_unlock label doesn't need to do any unlock operation and the name
became bad, so we just remove this label, and the only usage-site
"goto out_unlock" is subsituted to "return pool".
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'kernel/workqueue.c')
-rw-r--r-- | kernel/workqueue.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/workqueue.c b/kernel/workqueue.c index 0732d33d6dd5..a71cf176dce0 100644 --- a/kernel/workqueue.c +++ b/kernel/workqueue.c | |||
@@ -3463,7 +3463,7 @@ static struct worker_pool *get_unbound_pool(const struct workqueue_attrs *attrs) | |||
3463 | hash_for_each_possible(unbound_pool_hash, pool, hash_node, hash) { | 3463 | hash_for_each_possible(unbound_pool_hash, pool, hash_node, hash) { |
3464 | if (wqattrs_equal(pool->attrs, attrs)) { | 3464 | if (wqattrs_equal(pool->attrs, attrs)) { |
3465 | pool->refcnt++; | 3465 | pool->refcnt++; |
3466 | goto out_unlock; | 3466 | return pool; |
3467 | } | 3467 | } |
3468 | } | 3468 | } |
3469 | 3469 | ||
@@ -3501,7 +3501,7 @@ static struct worker_pool *get_unbound_pool(const struct workqueue_attrs *attrs) | |||
3501 | 3501 | ||
3502 | /* install */ | 3502 | /* install */ |
3503 | hash_add(unbound_pool_hash, &pool->hash_node, hash); | 3503 | hash_add(unbound_pool_hash, &pool->hash_node, hash); |
3504 | out_unlock: | 3504 | |
3505 | return pool; | 3505 | return pool; |
3506 | fail: | 3506 | fail: |
3507 | if (pool) | 3507 | if (pool) |