diff options
| author | Oleg Nesterov <oleg@redhat.com> | 2014-12-10 15:45:27 -0800 |
|---|---|---|
| committer | Moyster <oysterized@gmail.com> | 2019-05-02 18:25:09 +0200 |
| commit | 7dc881b97b5be13f62c59cf7525e1971f5140594 (patch) | |
| tree | f53d908132f4cb4ca01d703800c749eb74b867a7 /include/linux/workqueue.h | |
| parent | bc57f59f8c2467f18622bdbbbb0bda515b5a9581 (diff) | |
exit: reparent: cleanup the changing of ->parent
1. Cosmetic, but "if (t->parent == father)" looks a bit confusing.
We need to change t->parent if and only if t is not traced.
2. If we actually want this BUG_ON() to ensure that parent/ptrace
match each other, then we should also take ptrace_reparented()
case into account too.
3. Change this code to use for_each_thread() instead of deprecated
while_each_thread().
[dan.carpenter@oracle.com: silence a bogus static checker warning]
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Cc: Aaron Tomlin <atomlin@redhat.com>
Cc: Alexey Dobriyan <adobriyan@gmail.com>
Cc: "Eric W. Biederman" <ebiederm@xmission.com>,
Cc: Sterling Alexander <stalexan@redhat.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Roland McGrath <roland@hack.frob.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Change-Id: If69ec59b8eef170a67088da1aecaf1fad92226cf
Diffstat (limited to 'include/linux/workqueue.h')
0 files changed, 0 insertions, 0 deletions