summaryrefslogtreecommitdiff
path: root/kernel/entry
diff options
context:
space:
mode:
authorYuxuan Shui <yshuiv7@gmail.com>2021-01-23 14:21:32 +0300
committerThomas Gleixner <tglx@linutronix.de>2021-01-28 15:46:55 +0300
commit41c1a06d1d1544bed9692ba72a5692454eee1945 (patch)
tree8a02e112eeeac0d0953f2a070f452979c9eaa66c /kernel/entry
parent13391c60da3308ed9980de0168f74cce6c62ac1d (diff)
downloadlinux-41c1a06d1d1544bed9692ba72a5692454eee1945.tar.xz
entry: Unbreak single step reporting behaviour
The move of TIF_SYSCALL_EMU to SYSCALL_WORK_SYSCALL_EMU broke single step reporting. The original code reported the single step when TIF_SINGLESTEP was set and TIF_SYSCALL_EMU was not set. The SYSCALL_WORK conversion got the logic wrong and now the reporting only happens when both bits are set. Restore the original behaviour. [ tglx: Massaged changelog and dropped the pointless double negation ] Fixes: 64eb35f701f0 ("ptrace: Migrate TIF_SYSCALL_EMU to use SYSCALL_WORK flag") Signed-off-by: Yuxuan Shui <yshuiv7@gmail.com> Signed-off-by: Thomas Gleixner <tglx@linutronix.de> Reviewed-by: Gabriel Krisman Bertazi <krisman@collabora.com> Link: https://lore.kernel.org/r/877do3gaq9.fsf@m5Zedd9JOGzJrf0
Diffstat (limited to 'kernel/entry')
-rw-r--r--kernel/entry/common.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/entry/common.c b/kernel/entry/common.c
index 378341642f94..6dd82be60df8 100644
--- a/kernel/entry/common.c
+++ b/kernel/entry/common.c
@@ -222,7 +222,7 @@ static inline bool report_single_step(unsigned long work)
*/
static inline bool report_single_step(unsigned long work)
{
- if (!(work & SYSCALL_WORK_SYSCALL_EMU))
+ if (work & SYSCALL_WORK_SYSCALL_EMU)
return false;
return !!(current_thread_info()->flags & _TIF_SINGLESTEP);