From: Lai Jiangshan <laijs@cn.fujitsu.com>
Date: Tue, 22 Jul 2014 05:04:49 +0000 (+0800)
Subject: workqueue: remove the misnamed out_unlock label in get_unbound_pool()
X-Git-Tag: firefly_0821_release~176^2~3517^2~3
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=3fb1823c093ebe1869d34005837f64df64713780;p=firefly-linux-kernel-4.4.55.git

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>
---

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)
 	hash_for_each_possible(unbound_pool_hash, pool, hash_node, hash) {
 		if (wqattrs_equal(pool->attrs, attrs)) {
 			pool->refcnt++;
-			goto out_unlock;
+			return pool;
 		}
 	}
 
@@ -3501,7 +3501,7 @@ static struct worker_pool *get_unbound_pool(const struct workqueue_attrs *attrs)
 
 	/* install */
 	hash_add(unbound_pool_hash, &pool->hash_node, hash);
-out_unlock:
+
 	return pool;
 fail:
 	if (pool)