diff options
| author | Breno Leitao <leitao@debian.org> | 2026-03-05 08:15:37 -0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2026-03-19 16:14:48 +0100 |
| commit | bef3caeb4be76da2df6be5398feacdffff157456 (patch) | |
| tree | 5ab17ca6a04f1c4a33bd79a5b64e39f3573d13bb /kernel | |
| parent | 8b7db7197f47409e612cca385efab815b642954f (diff) | |
| download | linux-bef3caeb4be76da2df6be5398feacdffff157456.tar.gz linux-bef3caeb4be76da2df6be5398feacdffff157456.tar.bz2 linux-bef3caeb4be76da2df6be5398feacdffff157456.zip | |
workqueue: Use POOL_BH instead of WQ_BH when checking pool flags
[ Upstream commit f42f9091be9e5ff57567a3945cfcdd498f475348 ]
pr_cont_worker_id() checks pool->flags against WQ_BH, which is a
workqueue-level flag (defined in workqueue.h). Pool flags use a
separate namespace with POOL_* constants (defined in workqueue.c).
The correct constant is POOL_BH. Both WQ_BH and POOL_BH are defined
as (1 << 0) so this has no behavioral impact, but it is semantically
wrong and inconsistent with every other pool-level BH check in the
file.
Fixes: 4cb1ef64609f ("workqueue: Implement BH workqueues to eventually replace tasklets")
Signed-off-by: Breno Leitao <leitao@debian.org>
Acked-by: Song Liu <song@kernel.org>
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/workqueue.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/workqueue.c b/kernel/workqueue.c index 2909c19540ed..a4574c1e276a 100644 --- a/kernel/workqueue.c +++ b/kernel/workqueue.c @@ -6254,7 +6254,7 @@ static void pr_cont_worker_id(struct worker *worker) { struct worker_pool *pool = worker->pool; - if (pool->flags & WQ_BH) + if (pool->flags & POOL_BH) pr_cont("bh%s", pool->attrs->nice == HIGHPRI_NICE_LEVEL ? "-hi" : ""); else |