From c0bc445566302a4994e495438bceb466a5a21ee4 Mon Sep 17 00:00:00 2001 From: Steffen Klassert Date: Wed, 19 May 2010 13:43:46 +1000 Subject: [PATCH] --- yaml --- r: 196731 b: refs/heads/master c: 2b73b07ab8a44ce171e07a328439f311481a7ea7 h: refs/heads/master i: 196729: b29795ce21632c804a58332e6d4958ffe61c9d8d 196727: dc50b25d28fb6e3e6d8c085e840b4501ce89e1a5 v: v3 --- [refs] | 2 +- trunk/kernel/padata.c | 33 +++++++++++++++++++++++++-------- 2 files changed, 26 insertions(+), 9 deletions(-) diff --git a/[refs] b/[refs] index 559000fc3416..1c0341dfbcb2 100644 --- a/[refs] +++ b/[refs] @@ -1,2 +1,2 @@ --- -refs/heads/master: d46a5ac7a7e2045e33c6ad6ffb8cf18a7e86a15a +refs/heads/master: 2b73b07ab8a44ce171e07a328439f311481a7ea7 diff --git a/trunk/kernel/padata.c b/trunk/kernel/padata.c index 6d7ea481b716..ec6b8b7cf951 100644 --- a/trunk/kernel/padata.c +++ b/trunk/kernel/padata.c @@ -417,6 +417,29 @@ static void padata_free_pd(struct parallel_data *pd) kfree(pd); } +static void padata_flush_queues(struct parallel_data *pd) +{ + int cpu; + struct padata_queue *queue; + + for_each_cpu(cpu, pd->cpumask) { + queue = per_cpu_ptr(pd->queue, cpu); + flush_work(&queue->pwork); + } + + del_timer_sync(&pd->timer); + + if (atomic_read(&pd->reorder_objects)) + padata_reorder(pd); + + for_each_cpu(cpu, pd->cpumask) { + queue = per_cpu_ptr(pd->queue, cpu); + flush_work(&queue->swork); + } + + BUG_ON(atomic_read(&pd->refcnt) != 0); +} + static void padata_replace(struct padata_instance *pinst, struct parallel_data *pd_new) { @@ -428,11 +451,7 @@ static void padata_replace(struct padata_instance *pinst, synchronize_rcu(); - while (atomic_read(&pd_old->refcnt) != 0) - yield(); - - flush_workqueue(pinst->wq); - + padata_flush_queues(pd_old); padata_free_pd(pd_old); pinst->flags &= ~PADATA_RESET; @@ -695,12 +714,10 @@ void padata_free(struct padata_instance *pinst) synchronize_rcu(); - while (atomic_read(&pinst->pd->refcnt) != 0) - yield(); - #ifdef CONFIG_HOTPLUG_CPU unregister_hotcpu_notifier(&pinst->cpu_notifier); #endif + padata_flush_queues(pinst->pd); padata_free_pd(pinst->pd); free_cpumask_var(pinst->cpumask); kfree(pinst);