Commit 65e6757b authored by Kautuk Consul's avatar Kautuk Consul Committed by Greg Kroah-Hartman

devtmpfsd: fix task state handling

- 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: default avatarKautuk Consul <consul.kautuk@gmail.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 72f8c0bf
...@@ -413,10 +413,9 @@ static int devtmpfsd(void *p) ...@@ -413,10 +413,9 @@ static int devtmpfsd(void *p)
} }
spin_lock(&req_lock); spin_lock(&req_lock);
} }
set_current_state(TASK_INTERRUPTIBLE); __set_current_state(TASK_INTERRUPTIBLE);
spin_unlock(&req_lock); spin_unlock(&req_lock);
schedule(); schedule();
__set_current_state(TASK_RUNNING);
} }
return 0; return 0;
out: out:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment