diff options
| author | Joonsoo Kim <js1304@gmail.com> | 2012-08-15 16:25:36 +0200 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2012-08-16 23:21:15 +0200 |
| commit | 330dad5b9c9555632578c00e94e85c122561c5c7 (patch) | |
| tree | bc26f30c0d67f3a80cf5a7c97caaec64f88d9c6a | |
| parent | workqueue: add missing wmb() in clear_work_data() (diff) | |
| download | linux-330dad5b9c9555632578c00e94e85c122561c5c7.tar.xz linux-330dad5b9c9555632578c00e94e85c122561c5c7.zip | |
workqueue: use enum value to set array size of pools in gcwq
Commit 3270476a6c0ce322354df8679652f060d66526dc ('workqueue: reimplement
WQ_HIGHPRI using a separate worker_pool') introduce separate worker_pool
for HIGHPRI. Although there is NR_WORKER_POOLS enum value which represent
size of pools, definition of worker_pool in gcwq doesn't use it.
Using it makes code robust and prevent future mistakes.
So change code to use this enum value.
Signed-off-by: Joonsoo Kim <js1304@gmail.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
| -rw-r--r-- | kernel/workqueue.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/kernel/workqueue.c b/kernel/workqueue.c index 4fef9527a620..49d8f4a0110d 100644 --- a/kernel/workqueue.c +++ b/kernel/workqueue.c @@ -183,7 +183,8 @@ struct global_cwq { struct hlist_head busy_hash[BUSY_WORKER_HASH_SIZE]; /* L: hash of busy workers */ - struct worker_pool pools[2]; /* normal and highpri pools */ + struct worker_pool pools[NR_WORKER_POOLS]; + /* normal and highpri pools */ wait_queue_head_t rebind_hold; /* rebind hold wait */ } ____cacheline_aligned_in_smp; |