padata: Remove superfluous might_sleep
authorSteffen Klassert <steffen.klassert@secunet.com>
Thu, 29 Apr 2010 12:40:53 +0000 (14:40 +0200)
committerHerbert Xu <herbert@gondor.apana.org.au>
Mon, 3 May 2010 03:32:11 +0000 (11:32 +0800)
might_sleep() was placed before mutex_lock() in some places.
We remove them because mutex_lock() does might_sleep() too.

Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
kernel/padata.c

index 1209a17b971820a52ae1e49e866d8b52121c8ba8..5fa6ba6f11bd7337d8f4d5cf8f24d48c7c266e73 100644 (file)
@@ -443,8 +443,6 @@ int padata_set_cpumask(struct padata_instance *pinst,
        struct parallel_data *pd;
        int err = 0;
 
-       might_sleep();
-
        mutex_lock(&pinst->lock);
 
        pd = padata_alloc_pd(pinst, cpumask);
@@ -489,8 +487,6 @@ int padata_add_cpu(struct padata_instance *pinst, int cpu)
 {
        int err;
 
-       might_sleep();
-
        mutex_lock(&pinst->lock);
 
        cpumask_set_cpu(cpu, pinst->cpumask);
@@ -527,8 +523,6 @@ int padata_remove_cpu(struct padata_instance *pinst, int cpu)
 {
        int err;
 
-       might_sleep();
-
        mutex_lock(&pinst->lock);
 
        cpumask_clear_cpu(cpu, pinst->cpumask);
@@ -547,8 +541,6 @@ EXPORT_SYMBOL(padata_remove_cpu);
  */
 void padata_start(struct padata_instance *pinst)
 {
-       might_sleep();
-
        mutex_lock(&pinst->lock);
        pinst->flags |= PADATA_INIT;
        mutex_unlock(&pinst->lock);
@@ -562,8 +554,6 @@ EXPORT_SYMBOL(padata_start);
  */
 void padata_stop(struct padata_instance *pinst)
 {
-       might_sleep();
-
        mutex_lock(&pinst->lock);
        pinst->flags &= ~PADATA_INIT;
        mutex_unlock(&pinst->lock);