devtmpfsd: fix task state handling
authorKautuk Consul <consul.kautuk@gmail.com>
Tue, 15 Nov 2011 22:52:34 +0000 (14:52 -0800)
committerGreg Kroah-Hartman <gregkh@suse.de>
Wed, 16 Nov 2011 00:07:39 +0000 (16:07 -0800)
- Set the state to TASK_INTERRUPTIBLE using __set_current_state()
  instead of set_current_state() as the spin_unlock is an implicit memory
  barrier.

- After return from schedule(), there is no need to set the current
  state to TASK_RUNNING - a call to schedule() always returns in
  TASK_RUNNING state.

Signed-off-by: Kautuk Consul <consul.kautuk@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/base/devtmpfs.c

index a4760e095ff51b36a58871cdf9329bf4573f6c1d..2bb4bff3af7d33f73424242d59f68972d5bf2b45 100644 (file)
@@ -413,10 +413,9 @@ static int devtmpfsd(void *p)
                        }
                        spin_lock(&req_lock);
                }
-               set_current_state(TASK_INTERRUPTIBLE);
+               __set_current_state(TASK_INTERRUPTIBLE);
                spin_unlock(&req_lock);
                schedule();
-               __set_current_state(TASK_RUNNING);
        }
        return 0;
 out: