From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pavel Machek Subject: Re: [PATCH -mm 2/2]: PM: SMP-safe freezer Date: Mon, 4 Dec 2006 19:44:01 +0000 Message-ID: <20061204194401.GA4199@ucw.cz> References: <200612032318.29030.rjw@sisk.pl> <200612040010.46097.rjw@sisk.pl> <20061204103036.GA10862@elf.ucw.cz> <200612041503.04439.rjw@sisk.pl> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <200612041503.04439.rjw@sisk.pl> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: suspend-devel-bounces@lists.sourceforge.net Errors-To: suspend-devel-bounces@lists.sourceforge.net To: "Rafael J. Wysocki" Cc: suspend-devel List , pm list , Stephen Hemminger List-Id: linux-pm@vger.kernel.org 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 > > > > 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