Re: [PATCH -mm 2/2]: PM: SMP-safe freezer

[Pavel Machek <pavel@ucw.cz>]

Hi!

> > > To solve first of these problems we need to stop using PF_FROZEN to tell
> > > tasks that they should go to the refrigerator.  Instead, we can introduce
> > > a special TIF_*** flag and use it for this purpose, since it is allowed to
> > > change the other tasks' TIF_*** flags and there are special calls for it.
> > > 
> > > To avoid the freeze_process()-refrigerator() race we can make freeze_process()
> > > to always check the task's PF_FROZEN flag after it's read its "freeze" flag.
> > > We should also make sure that refrigerator() will always reset the task's
> > > "freeze" flag after it's set PF_FROZEN for it.
> > > 
> > > Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
> > 
> > ACK.
> 
> But I think I'll need to add TIF_FROZEN for all architectures, because suspend
> to RAM is supposed to work on all of them, isn't it?

Well, yes, it should be added, but no, I do not think s2ram works on
that many machines.

> Also, this patch is only sufficient with [1/2] or an alternative solution?

I haven't finished looking at that one. I suspect it is okay, but I'm
not sure it is enough.
							Pavel
-- 
Thanks for all the (sleeping) penguins.

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV