workqueue: Remove the unneeded cpumask empty check in wq_calc_pod_cpumask()
authorLai Jiangshan <jiangshan.ljs@antgroup.com>
Thu, 11 Jul 2024 08:35:44 +0000 (16:35 +0800)
committerTejun Heo <tj@kernel.org>
Thu, 11 Jul 2024 22:50:34 +0000 (12:50 -1000)
The cpumask empty check in wq_calc_pod_cpumask() has long been useless.
It just works purely as documents which states that the cpumask is not
possible empty after the function returns.

Now the code above is even more explicit that the cpumask is not empty,
so the document-only empty check can be removed.

Signed-off-by: Lai Jiangshan <jiangshan.ljs@antgroup.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
kernel/workqueue.c

index 9b7c1fcd934b82bce1d5c6ffe6e14d6f2483bfa3..298ce6b4989d389293b31021267c0b83dc859ba2 100644 (file)
@@ -5150,10 +5150,6 @@ static void wq_calc_pod_cpumask(struct workqueue_attrs *attrs, int cpu,
                cpumask_copy(attrs->__pod_cpumask, attrs->cpumask);
                return;
        }
-
-       if (cpumask_empty(attrs->__pod_cpumask))
-               pr_warn_once("WARNING: workqueue cpumask: online intersect > "
-                               "possible intersect\n");
 }
 
 /* install @pwq into @wq and return the old pwq, @cpu < 0 for dfl_pwq */