From 2cb61f76be3b17d5ad42ba3b7b23c7bf98253a0b Mon Sep 17 00:00:00 2001
From: Lai Jiangshan <jiangshan.ljs@antgroup.com>
Date: Thu, 11 Jul 2024 16:35:44 +0800
Subject: [PATCH] workqueue: Remove the unneeded cpumask empty check in
wq_calc_pod_cpumask()
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 | 4 ----
1 file changed, 4 deletions(-)
diff --git a/kernel/workqueue.c b/kernel/workqueue.c
index 9b7c1fcd934b8..298ce6b4989d3 100644
--- a/kernel/workqueue.c
+++ b/kernel/workqueue.c
@@ -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 */
--
GitLab