On Thu 04-09-14 15:30:42, Cong Wang wrote: > __thaw_task() no longer clears frozen flag since commit a3201227f803 > (freezer: make freezing() test freeze conditions in effect instead of > TIF_FREEZE). > > Cc: David Rientjes <rient...@google.com> > Cc: "Rafael J. Wysocki" <r...@rjwysocki.net> > Cc: Tejun Heo <t...@kernel.org> > Cc: Andrew Morton <a...@linux-foundation.org> > Cc: Michal Hocko <mho...@suse.cz> > Reviewed-by: Michal Hocko <mho...@suse.cz> > Signed-off-by: Cong Wang <xiyou.wangc...@gmail.com>
Thanks for extending the description. Acked-by: Michal Hocko <mho...@suse.cz> > --- > kernel/freezer.c | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/kernel/freezer.c b/kernel/freezer.c > index dbcb87c..438ecca 100644 > --- a/kernel/freezer.c > +++ b/kernel/freezer.c > @@ -161,12 +161,6 @@ void __thaw_task(struct task_struct *p) > { > unsigned long flags; > > - /* > - * Clear freezing and kick @p if FROZEN. Clearing is guaranteed to > - * be visible to @p as waking up implies wmb. Waking up inside > - * freezer_lock also prevents wakeups from leaking outside > - * refrigerator. > - */ > spin_lock_irqsave(&freezer_lock, flags); > if (frozen(p)) > wake_up_process(p); > -- > 1.8.3.1 > -- Michal Hocko SUSE Labs -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/