Messages in this thread | | | Date | Sat, 20 Aug 2011 10:13:56 +0200 | From | Tejun Heo <> | Subject | Re: [PATCH 01/16] freezer: fix current->state restoration race in refrigerator() |
| |
Hello, Rafael.
On Fri, Aug 19, 2011 at 11:08:59PM +0200, Rafael J. Wysocki wrote: > On Friday, August 19, 2011, Tejun Heo wrote: > > refrigerator() saves current->state before entering frozen state and > > restores it before returning using __set_current_state(); however, > > this is racy, for example, please consider the following sequence. > > > > set_current_state(TASK_INTERRUPTIBLE); > > try_to_sleep(); > > Did you mean try_to_freeze() here?
Heh, yeah, I'm constantly being confused among try_to_freeze(), try_to_sleep() and try_to_wake_up(). Will update. ;)
Thanks.
-- tejun
| |