diff options
| author | Oleg Nesterov <[email protected]> | 2011-07-07 19:33:54 +0000 |
|---|---|---|
| committer | Oleg Nesterov <[email protected]> | 2011-07-17 18:23:50 +0000 |
| commit | 961c4675c75112717705fa5c0c53cb9664051479 (patch) | |
| tree | d424199346b5dc1c5d27fe920a66a6ea11989d89 /kernel/trace/trace_functions.c | |
| parent | ptrace: make former thread ID available via PTRACE_GETEVENTMSG after PTRACE_E... (diff) | |
| download | kernel-961c4675c75112717705fa5c0c53cb9664051479.tar.gz kernel-961c4675c75112717705fa5c0c53cb9664051479.zip | |
has_stopped_jobs: s/task_is_stopped/SIGNAL_STOP_STOPPED/
has_stopped_jobs() naively checks task_is_stopped(group_leader). This
was always wrong even without ptrace, group_leader can be dead. And
given that ptrace can change the state to TRACED this is wrong even
in the single-threaded case.
Change the code to check SIGNAL_STOP_STOPPED and simplify the code,
retval + break/continue doesn't make this trivial code more readable.
We could probably add the usual "|| signal->group_stop_count" check
but I don't think this makes sense, the task can start the group-stop
right after the check anyway.
Signed-off-by: Oleg Nesterov <[email protected]>
Acked-by: Tejun Heo <[email protected]>
Diffstat (limited to 'kernel/trace/trace_functions.c')
0 files changed, 0 insertions, 0 deletions