workqueue: separate out init_worker_pool() from init_workqueues()

This will be used to implement unbound pools with custom attributes.

This patch doesn't introduce any functional changes.

Signed-off-by: Tejun Heo <tj@kernel.org>
Reviewed-by: Lai Jiangshan <laijs@cn.fujitsu.com>

1 files changed, 21 insertions, 16 deletions

diff --git a/kernel/workqueue.c b/kernel/workqueue.c
index 16f7f8d79d35..094f16668e1b 100644
--- a/kernel/workqueue.c
+++ b/kernel/workqueue.c
@@ -3123,6 +3123,26 @@ int keventd_up(void)
         return system_wq != NULL;
 }
 
+static void init_worker_pool(struct worker_pool *pool)
+{
+        spin_lock_init(&pool->lock);
+        pool->flags |= POOL_DISASSOCIATED;
+        INIT_LIST_HEAD(&pool->worklist);
+        INIT_LIST_HEAD(&pool->idle_list);
+        hash_init(pool->busy_hash);
+
+        init_timer_deferrable(&pool->idle_timer);
+        pool->idle_timer.function = idle_worker_timeout;
+        pool->idle_timer.data = (unsigned long)pool;
+
+        setup_timer(&pool->mayday_timer, pool_mayday_timeout,
+                    (unsigned long)pool);
+
+        mutex_init(&pool->manager_arb);
+        mutex_init(&pool->assoc_mutex);
+        ida_init(&pool->worker_ida);
+}
+
 static int alloc_and_link_pwqs(struct workqueue_struct *wq)
 {
         bool highpri = wq->flags & WQ_HIGHPRI;
@@ -3790,23 +3810,8 @@ static int __init init_workqueues(void)
                 struct worker_pool *pool;
 
                 for_each_std_worker_pool(pool, cpu) {
-                        spin_lock_init(&pool->lock);
+                        init_worker_pool(pool);
                         pool->cpu = cpu;
-                        pool->flags |= POOL_DISASSOCIATED;
-                        INIT_LIST_HEAD(&pool->worklist);
-                        INIT_LIST_HEAD(&pool->idle_list);
-                        hash_init(pool->busy_hash);
-
-                        init_timer_deferrable(&pool->idle_timer);
-                        pool->idle_timer.function = idle_worker_timeout;
-                        pool->idle_timer.data = (unsigned long)pool;
-
-                        setup_timer(&pool->mayday_timer, pool_mayday_timeout,
-                                    (unsigned long)pool);
-
-                        mutex_init(&pool->manager_arb);
-                        mutex_init(&pool->assoc_mutex);
-                        ida_init(&pool->worker_ida);
 
                         /* alloc pool ID */
                         BUG_ON(worker_pool_assign_id(pool));