Messages in this thread | | | Date | Fri, 2 Sep 2011 19:08:17 +0200 | From | Oleg Nesterov <> | Subject | Re: [PATCH pm-freezer 3/4] freezer: restructure __refrigerator() |
| |
On 08/31, Tejun Heo wrote: > > for (;;) { > set_current_state(TASK_UNINTERRUPTIBLE); > + > + spin_lock_irq(&freezer_lock); > + current->flags |= PF_FROZEN; > if (!freezing(current) || > - (check_kthr_stop && kthread_should_stop())) > + (check_kthr_stop && kthread_should_stop())) { > + current->flags &= ~PF_FROZEN; > + break;
Argh! sorry I missed this...
This "break" is typo, it should be removed.
Otherwise I believe this is correct.
Oleg.
| |