Hi! > > >(2) the race between the delivery of > > > the continuation signal and the freezer is damn hard to trigger (still I > > > think > > > I can wirte some artificial code that would trigger this, although it > > > would > > > involve a kernel thread sending SIGCONT to a user space process - provided > > > it's permissible ;-)). > > > > It is not really permissible, but I do not see how it would hurt... > > > > With that patch, we put TASK_STOPPED tasks into refrigerator, along > > with everyone else. SIGCONT is *not* going to help them out of > > refrigerator. > > > > We also ignore TASK_STOPPED now, so we'll not proceed until all > > stopped tasks are in refrigerator. > > > > Now... I'm not 100% sure I got the ptrace() cases right (and did not > > test that)... but TASK_STOPPED cases look pretty trivial to me now. > > Please note that currently (and with your patch) freezeable() returns 0 for
I cerainly wanted freezable to return 1 for TASK_STOPPED... and I think this part of my patch should be doing it...? 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))) return 0; return 1; } > TASK_STOPPED, so we don't set PF_FREEZE for them and my patch changes > _exactly_ that and adds some minor (optional) modifications to the error > paths. -- 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 _______________________________________________ Suspend-devel mailing list Suspend-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/suspend-devel