diff options
author | Eric W. Biederman <ebiederm@xmission.com> | 2022-04-26 16:30:17 -0500 |
---|---|---|
committer | Eric W. Biederman <ebiederm@xmission.com> | 2022-05-11 14:33:33 -0500 |
commit | c200e4bb44e80b343c09841e7caaaca0aac5e5fa (patch) | |
tree | 5318db1a0ff49083693b76d52968338c5aaa19f9 /arch/um/kernel/process.c | |
parent | e71ba124078e391879e0bf111529fa2d630d106c (diff) | |
download | linux-stable-c200e4bb44e80b343c09841e7caaaca0aac5e5fa.tar.gz linux-stable-c200e4bb44e80b343c09841e7caaaca0aac5e5fa.tar.bz2 linux-stable-c200e4bb44e80b343c09841e7caaaca0aac5e5fa.zip |
ptrace/um: Replace PT_DTRACE with TIF_SINGLESTEP
User mode linux is the last user of the PT_DTRACE flag. Using the flag to indicate
single stepping is a little confusing and worse changing tsk->ptrace without locking
could potentionally cause problems.
So use a thread info flag with a better name instead of flag in tsk->ptrace.
Remove the definition PT_DTRACE as uml is the last user.
Cc: stable@vger.kernel.org
Acked-by: Johannes Berg <johannes@sipsolutions.net>
Tested-by: Kees Cook <keescook@chromium.org>
Reviewed-by: Oleg Nesterov <oleg@redhat.com>
Link: https://lkml.kernel.org/r/20220505182645.497868-3-ebiederm@xmission.com
Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
Diffstat (limited to 'arch/um/kernel/process.c')
-rw-r--r-- | arch/um/kernel/process.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/um/kernel/process.c b/arch/um/kernel/process.c index 80504680be08..88c5c7844281 100644 --- a/arch/um/kernel/process.c +++ b/arch/um/kernel/process.c @@ -335,7 +335,7 @@ int singlestepping(void * t) { struct task_struct *task = t ? t : current; - if (!(task->ptrace & PT_DTRACE)) + if (!test_thread_flag(TIF_SINGLESTEP)) return 0; if (task->thread.singlestep_syscall) |