From 5ec06ab3ef1f7cb0a1c86b210c277d9387400c10 Mon Sep 17 00:00:00 2001 From: Christoph Lameter Date: Tue, 6 Nov 2007 11:33:52 -0800 Subject: [PATCH] cpu alloc: workqueue conversion Signed-off-by: Christoph Lameter --- kernel/workqueue.c | 27 ++++++++++++++------------- 1 file changed, 14 insertions(+), 13 deletions(-) Index: linux-2.6/kernel/workqueue.c =================================================================== --- linux-2.6.orig/kernel/workqueue.c 2008-05-07 22:15:36.000000000 -0700 +++ linux-2.6/kernel/workqueue.c 2008-05-21 21:42:02.000000000 -0700 @@ -99,7 +99,7 @@ { if (unlikely(is_single_threaded(wq))) cpu = singlethread_cpu; - return per_cpu_ptr(wq->cpu_wq, cpu); + return CPU_PTR(wq->cpu_wq, cpu); } /* @@ -398,7 +398,7 @@ lock_acquire(&wq->lockdep_map, 0, 0, 0, 2, _THIS_IP_); lock_release(&wq->lockdep_map, 1, _THIS_IP_); for_each_cpu_mask(cpu, *cpu_map) - flush_cpu_workqueue(per_cpu_ptr(wq->cpu_wq, cpu)); + flush_cpu_workqueue(CPU_PTR(wq->cpu_wq, cpu)); } EXPORT_SYMBOL_GPL(flush_workqueue); @@ -478,7 +478,7 @@ cpu_map = wq_cpu_map(wq); for_each_cpu_mask(cpu, *cpu_map) - wait_on_cpu_work(per_cpu_ptr(wq->cpu_wq, cpu), work); + wait_on_cpu_work(CPU_PTR(wq->cpu_wq, cpu), work); } static int __cancel_work_timer(struct work_struct *work, @@ -598,21 +598,21 @@ int cpu; struct work_struct *works; - works = alloc_percpu(struct work_struct); + works = CPU_ALLOC(struct work_struct, GFP_KERNEL); if (!works) return -ENOMEM; get_online_cpus(); for_each_online_cpu(cpu) { - struct work_struct *work = per_cpu_ptr(works, cpu); + struct work_struct *work = CPU_PTR(works, cpu); INIT_WORK(work, func); set_bit(WORK_STRUCT_PENDING, work_data_bits(work)); - __queue_work(per_cpu_ptr(keventd_wq->cpu_wq, cpu), work); + __queue_work(CPU_PTR(keventd_wq->cpu_wq, cpu), work); } flush_workqueue(keventd_wq); put_online_cpus(); - free_percpu(works); + CPU_FREE(works); return 0; } @@ -661,7 +661,7 @@ BUG_ON(!keventd_wq); - cwq = per_cpu_ptr(keventd_wq->cpu_wq, cpu); + cwq = CPU_PTR(keventd_wq->cpu_wq, cpu); if (current == cwq->thread) ret = 1; @@ -672,7 +672,7 @@ static struct cpu_workqueue_struct * init_cpu_workqueue(struct workqueue_struct *wq, int cpu) { - struct cpu_workqueue_struct *cwq = per_cpu_ptr(wq->cpu_wq, cpu); + struct cpu_workqueue_struct *cwq = CPU_PTR(wq->cpu_wq, cpu); cwq->wq = wq; spin_lock_init(&cwq->lock); @@ -730,7 +730,8 @@ if (!wq) return NULL; - wq->cpu_wq = alloc_percpu(struct cpu_workqueue_struct); + wq->cpu_wq = CPU_ALLOC(struct cpu_workqueue_struct, + GFP_KERNEL|__GFP_ZERO); if (!wq->cpu_wq) { kfree(wq); return NULL; @@ -814,10 +815,10 @@ spin_unlock(&workqueue_lock); for_each_cpu_mask(cpu, *cpu_map) - cleanup_workqueue_thread(per_cpu_ptr(wq->cpu_wq, cpu)); + cleanup_workqueue_thread(CPU_PTR(wq->cpu_wq, cpu)); put_online_cpus(); - free_percpu(wq->cpu_wq); + CPU_FREE(wq->cpu_wq); kfree(wq); } EXPORT_SYMBOL_GPL(destroy_workqueue); @@ -838,7 +839,7 @@ } list_for_each_entry(wq, &workqueues, list) { - cwq = per_cpu_ptr(wq->cpu_wq, cpu); + cwq = CPU_PTR(wq->cpu_wq, cpu); switch (action) { case CPU_UP_PREPARE: