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/[email protected]/
Suggested-by: Tejun Heo <[email protected]>
Signed-off-by: Marco Crivellari <[email protected]>
---
Changes in v2:
- Improved commit log

- Rebased on v7.1-rc3

Link to v1: 
https://lore.kernel.org/all/[email protected]/

 drivers/gpu/drm/bridge/analogix/anx7625.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/bridge/analogix/anx7625.c 
b/drivers/gpu/drm/bridge/analogix/anx7625.c
index 54b02242d6db..8c3154817896 100644
--- a/drivers/gpu/drm/bridge/analogix/anx7625.c
+++ b/drivers/gpu/drm/bridge/analogix/anx7625.c
@@ -2849,7 +2849,8 @@ static int anx7625_i2c_probe(struct i2c_client *client)
        if (platform->pdata.intp_irq) {
                INIT_WORK(&platform->work, anx7625_work_func);
                platform->workqueue = alloc_workqueue("anx7625_work",
-                                                     WQ_FREEZABLE | 
WQ_MEM_RECLAIM, 1);
+                                                     WQ_FREEZABLE | 
WQ_MEM_RECLAIM | WQ_PERCPU,
+                                                     1);
                if (!platform->workqueue) {
                        DRM_DEV_ERROR(dev, "fail to create work queue\n");
                        ret = -ENOMEM;
-- 
2.54.0


Reply via email to