]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
block: Add WQ_PERCPU to alloc_workqueue users
authorMarco Crivellari <marco.crivellari@suse.com>
Mon, 23 Feb 2026 09:29:19 +0000 (10:29 +0100)
committerJens Axboe <axboe@kernel.dk>
Fri, 17 Apr 2026 20:38:22 +0000 (14:38 -0600)
This continues the effort to refactor workqueue APIs, which began with
the introduction of new workqueues and a new alloc_workqueue flag in:

   commit 128ea9f6ccfb ("workqueue: Add system_percpu_wq and system_dfl_wq")
   commit 930c2ea566af ("workqueue: Add new WQ_PERCPU flag")

The refactoring is going to alter the default behavior of
alloc_workqueue() to be unbound by default.

With the introduction of the WQ_PERCPU flag (equivalent to !WQ_UNBOUND),
any alloc_workqueue() caller that doesn’t explicitly specify WQ_UNBOUND
must now use WQ_PERCPU. For more details see the Link tag below.

In order to keep alloc_workqueue() behavior identical, explicitly request
WQ_PERCPU.

Link: https://lore.kernel.org/all/20250221112003.1dSuoGyc@linutronix.de/
Suggested-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Marco Crivellari <marco.crivellari@suse.com>
Link: https://patch.msgid.link/20260223092920.60424-2-marco.crivellari@suse.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
block/bio-integrity-auto.c
block/bio.c
block/blk-core.c

index ebd17f47e0f9ea25f5d31d4a2a24b867f02814ec..353eed632fcc9e578e2631570aef91b4a6a6b06f 100644 (file)
@@ -125,7 +125,7 @@ static int __init blk_integrity_auto_init(void)
         * Make it highpri CPU intensive wq with max concurrency of 1.
         */
        kintegrityd_wq = alloc_workqueue("kintegrityd", WQ_MEM_RECLAIM |
-                                        WQ_HIGHPRI | WQ_CPU_INTENSIVE, 1);
+                                        WQ_HIGHPRI | WQ_CPU_INTENSIVE | WQ_PERCPU, 1);
        if (!kintegrityd_wq)
                panic("Failed to create kintegrityd\n");
        return 0;
index c52a0bd1e8993dcc5941e42d7481b66214754bd3..4d46af0cd25633b29fc97b1eaa6c30d25a24680c 100644 (file)
@@ -1958,7 +1958,7 @@ int bioset_init(struct bio_set *bs,
 
        if (flags & BIOSET_NEED_RESCUER) {
                bs->rescue_workqueue = alloc_workqueue("bioset",
-                                                       WQ_MEM_RECLAIM, 0);
+                                                       WQ_MEM_RECLAIM | WQ_PERCPU, 0);
                if (!bs->rescue_workqueue)
                        goto bad;
        }
index 474700ffaa1c895cb28f5fb0ec9e6b1f5c601b74..17450058ea6d846d536dd43d83aa752e6876087a 100644 (file)
@@ -1282,7 +1282,7 @@ int __init blk_dev_init(void)
 
        /* used for unplugging and affects IO latency/throughput - HIGHPRI */
        kblockd_workqueue = alloc_workqueue("kblockd",
-                                           WQ_MEM_RECLAIM | WQ_HIGHPRI, 0);
+                                           WQ_MEM_RECLAIM | WQ_HIGHPRI | WQ_PERCPU, 0);
        if (!kblockd_workqueue)
                panic("Failed to create kblockd\n");