diff options
| author | Sergei Trofimovich <[email protected]> | 2021-03-13 05:08:23 +0000 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2021-03-13 19:27:31 +0000 |
| commit | 0ceb1ace4a2778e34a5414e5349712ae4dc41d85 (patch) | |
| tree | 67bc0a86e5d33a62555f416e31e0f322182f7952 /drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | |
| parent | mm/userfaultfd: fix memory corruption due to writeprotect (diff) | |
| download | kernel-0ceb1ace4a2778e34a5414e5349712ae4dc41d85.tar.gz kernel-0ceb1ace4a2778e34a5414e5349712ae4dc41d85.zip | |
ia64: fix ia64_syscall_get_set_arguments() for break-based syscalls
In https://bugs.gentoo.org/769614 Dmitry noticed that
`ptrace(PTRACE_GET_SYSCALL_INFO)` does not work for syscalls called via
glibc's syscall() wrapper.
ia64 has two ways to call syscalls from userspace: via `break` and via
`eps` instructions.
The difference is in stack layout:
1. `eps` creates simple stack frame: no locals, in{0..7} == out{0..8}
2. `break` uses userspace stack frame: may be locals (glibc provides
one), in{0..7} == out{0..8}.
Both work fine in syscall handling cde itself.
But `ptrace(PTRACE_GET_SYSCALL_INFO)` uses unwind mechanism to
re-extract syscall arguments but it does not account for locals.
The change always skips locals registers. It should not change `eps`
path as kernel's handler already enforces locals=0 and fixes `break`.
Tested on v5.10 on rx3600 machine (ia64 9040 CPU).
Link: https://lkml.kernel.org/r/[email protected]
Link: https://bugs.gentoo.org/769614
Signed-off-by: Sergei Trofimovich <[email protected]>
Reported-by: Dmitry V. Levin <[email protected]>
Cc: Oleg Nesterov <[email protected]>
Cc: John Paul Adrian Glaubitz <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_object.c')
0 files changed, 0 insertions, 0 deletions
