Commit b3d20916 authored by Lai Jiangshan's avatar Lai Jiangshan Committed by Tejun Heo
Browse files

workqueue: Simplify goto statement 



Use a simple if-statement to replace the cumbersome goto-statement in
workqueue_set_unbound_cpumask().

Cc: Waiman Long <longman@redhat.com>
Signed-off-by: default avatarLai Jiangshan <jiangshan.ljs@antgroup.com>
Signed-off-by: default avatarTejun Heo <tj@kernel.org>
parent 84165883

kernel/workqueue.c

+3 −8
Original line number Diff line number Diff line
@@ -7197,15 +7197,10 @@ static int workqueue_set_unbound_cpumask(cpumask_var_t cpumask) 
	 */

	cpumask_and(cpumask, cpumask, cpu_possible_mask);

	if (!cpumask_empty(cpumask)) {

		apply_wqattrs_lock();

		if (cpumask_equal(cpumask, wq_unbound_cpumask)) {

		ret = 0;

			goto out_unlock;

		}



		apply_wqattrs_lock();

		if (!cpumask_equal(cpumask, wq_unbound_cpumask))

			ret = workqueue_apply_unbound_cpumask(cpumask);



out_unlock:

		if (!ret)

			cpumask_copy(wq_requested_unbound_cpumask, cpumask);

		apply_wqattrs_unlock();