Hi,

On Tuesday, 5 December 2006 15:13, Pavel Machek wrote:
> Hi!
> 
> > > Actually, what do you think about this patch? It removes special
> > > handling of TASK_TRACED, and should do the trick, too...
> > 
> > Well, I don't think so, ....
> 
> > > @@ -1702,7 +1702,9 @@ finish_stop(int stop_count)
> > >           read_unlock(&tasklist_lock);
> > >   }
> > >  
> > > - schedule();
> > > + do {
> > > +         schedule();
> > > + } while (try_to_freeze());
> > >   /*
> > >    * Now we don't run again until continued.
> > >    */
> > 
> > ... because if you want try_to_freeze() here to trigger, then the task in
> > question should have PF_FREEZE set, but we don't set it anywhere.
> 
> ?? We set it to all the processes, so stopped processes get it,
> too...??

No, we don't.  We only set it for the processes for which freezeable() returns 
1.

Greetings,
Rafael


-- 
If you don't have the time to read,
you don't have the time or the tools to write.
                - Stephen King

-------------------------------------------------------------------------
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
_______________________________________________
Suspend-devel mailing list
Suspend-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/suspend-devel

Reply via email to