[PATCH 1/1] workqueue: use NR_STD_WORKER_POOLS instead of hardcoded value

[PATCH 1/1] workqueue: use NR_STD_WORKER_POOLS instead of hardcoded value

[Maninder Singh]

use NR_STD_WORKER_POOLS for irq_work_fns[] array definition.
NR_STD_WORKER_POOLS is also 2, but better to use MACRO.
Initialization loop for_each_bh_worker_pool() also uses same MACRO.

Signed-off-by: Maninder Singh <maninder1.s@samsung.com>
---
 kernel/workqueue.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kernel/workqueue.c b/kernel/workqueue.c
index 08b1c786b463..16b65a7f1ac2 100644
--- a/kernel/workqueue.c
+++ b/kernel/workqueue.c
@@ -7932,7 +7932,7 @@ void __init workqueue_init_early(void)
 {
 	struct wq_pod_type *pt = &wq_pod_types[WQ_AFFN_SYSTEM];
 	int std_nice[NR_STD_WORKER_POOLS] = { 0, HIGHPRI_NICE_LEVEL };
-	void (*irq_work_fns[2])(struct irq_work *) = { bh_pool_kick_normal,
+	void (*irq_work_fns[NR_STD_WORKER_POOLS])(struct irq_work *) = { bh_pool_kick_normal,
 						       bh_pool_kick_highpri };
 	int i, cpu;
 
-- 
2.34.1

Re: [PATCH 1/1] workqueue: use NR_STD_WORKER_POOLS instead of hardcoded value

[Tejun Heo]

Hello,

On Tue, Apr  7, 2026 at 09:12:15AM +0530, Maninder Singh wrote:
> use NR_STD_WORKER_POOLS for irq_work_fns[] array definition.
> NR_STD_WORKER_POOLS is also 2, but better to use MACRO.
> Initialization loop for_each_bh_worker_pool() also uses same MACRO.

Applied to wq/for-7.1 with a small formatting tweak so the longer
declaration doesn't leave the initializer misaligned:

	void (*irq_work_fns[NR_STD_WORKER_POOLS])(struct irq_work *) =
		{ bh_pool_kick_normal, bh_pool_kick_highpri };

Thanks.

--
tejun