diff options
| author | Yang Yingliang <yangyingliang@huawei.com> | 2023-08-04 05:22:15 +0200 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2023-08-07 20:36:51 +0200 |
| commit | 9680540c0c56a1f75a2d6aab31bf38aa429aa9d9 (patch) | |
| tree | ba5ada7e82537495bc8d57655c91b849c4b9ab2b /kernel | |
| parent | workqueue: add cmdline parameter `workqueue.unbound_cpus` to further constrai... (diff) | |
| download | linux-9680540c0c56a1f75a2d6aab31bf38aa429aa9d9.tar.xz linux-9680540c0c56a1f75a2d6aab31bf38aa429aa9d9.zip | |
workqueue: use LIST_HEAD to initialize cull_list
Use LIST_HEAD() to initialize cull_list instead of open-coding it.
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
Reviewed-by: Lai Jiangshan <jiangshanlai@gmail.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/workqueue.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/kernel/workqueue.c b/kernel/workqueue.c index 83f8993af57c..a90f1e642548 100644 --- a/kernel/workqueue.c +++ b/kernel/workqueue.c @@ -2304,9 +2304,8 @@ static void idle_worker_timeout(struct timer_list *t) static void idle_cull_fn(struct work_struct *work) { struct worker_pool *pool = container_of(work, struct worker_pool, idle_cull_work); - struct list_head cull_list; + LIST_HEAD(cull_list); - INIT_LIST_HEAD(&cull_list); /* * Grabbing wq_pool_attach_mutex here ensures an already-running worker * cannot proceed beyong worker_detach_from_pool() in its self-destruct @@ -3872,10 +3871,8 @@ static void rcu_free_pool(struct rcu_head *rcu) static void put_unbound_pool(struct worker_pool *pool) { DECLARE_COMPLETION_ONSTACK(detach_completion); - struct list_head cull_list; struct worker *worker; - - INIT_LIST_HEAD(&cull_list); + LIST_HEAD(cull_list); lockdep_assert_held(&wq_pool_mutex); |