Hi!
> > Actually, what do you think about this patch? It removes special
> > handling of TASK_TRACED, and should do the trick, too...
> >
> > diff --git a/kernel/power/process.c b/kernel/power/process.c
> > index 7bcc976..d56e494 100644
> > --- a/kernel/power/process.c
> > +++ b/kernel/power/process.c
> > @@ -26,8 +26,7 @@ static inline int freezeable(struct task
> > (p->flags & PF_NOFREEZE) ||
> > (p->exit_state == EXIT_ZOMBIE) ||
> > (p->exit_state == EXIT_DEAD) ||
> > - ((p->exit_state == TASK_TRACED) && frozen(p->parent)) ||
> > - (p->state == TASK_STOPPED))
> > + ((p->exit_state == TASK_TRACED) && frozen(p->parent)))
>
> ... with the exception that I haven't added the last line, since there's some
> code in try_to_freeze_tasks() that does the same and better ...
> > return 0;
> > return 1;
> > }
> > diff --git a/kernel/signal.c b/kernel/signal.c
> > index 9a61944..e305ad1 100644
> > --- a/kernel/signal.c
> > +++ b/kernel/signal.c
> > @@ -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.
> > */
> >
> >
>
> ... and it fails to freeze processes if there's a stopped task (to verify,
> run vi, press ^Z, and try to suspend).
Huh, I thought I've tested that. Too bad, you are right.
> It happens because we shouldn't count the stopped task as freezeable any
> more after we've set PF_FREEZE for it and we can fix that by adding
>
> if (p->state == TASK_STOPPED && freezing(p))
> continue;
No, I'd actually like to force stopped task into refrigerator.
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
