diff options
author | Adhemerval Zanella <adhemerval.zanella@linaro.org> | 2017-05-08 11:22:20 -0300 |
---|---|---|
committer | Adhemerval Zanella <adhemerval.zanella@linaro.org> | 2017-05-11 17:27:31 -0300 |
commit | 31073a53d8ff1e8bac53e34cb626dae5aa6ce69c (patch) | |
tree | 65aab5e22dcc488f4d17af6fea7cc837f9fe524c /ChangeLog | |
parent | 488e08b600416ef96d8148fe5239a6387555b1eb (diff) | |
download | glibc-31073a53d8ff1e8bac53e34cb626dae5aa6ce69c.tar glibc-31073a53d8ff1e8bac53e34cb626dae5aa6ce69c.tar.gz glibc-31073a53d8ff1e8bac53e34cb626dae5aa6ce69c.tar.bz2 glibc-31073a53d8ff1e8bac53e34cb626dae5aa6ce69c.zip |
powerpc: Fix signal handling in backtrace
Now with read consolidation which uses SYSCALL_CANCEL macro, a frame
pointer is created in the syscall code and this makes the powerpc
backtrace obtain a bogus entry for the signal handling patch.
It is because it does not setup the correct frame pointer register
(r1) based on the saved value from the kernel sigreturn. It was not
failing because the syscall frame pointer register was the same one
for the next frame (the function that actually called the syscall).
This patch fixes it by setup the next stack frame using the saved
one by the kernel sigreturn. It fixes tst-backtrace{5,6} from
the read consolidation patch.
Checked on powerpc-linux-gnu and powerpc64le-linux-gnu.
* sysdeps/powerpc/powerpc32/backtrace.c (is_sigtramp_address): Use
void* for argument type and use VDSO_SYMBOL macro.
(is_sigtramp_address_rt): Likewise.
(__backtrace): Setup expected frame pointer address for signal
handling.
* sysdeps/powerpc/powerpc64/backtrace.c (is_sigtramp_address): Use
void* for argumetn type and use VSDO_SYMBOL macro.
(__backtrace): Setup expected frame pointer address for signal
handling.
Diffstat (limited to 'ChangeLog')
-rw-r--r-- | ChangeLog | 10 |
1 files changed, 10 insertions, 0 deletions
@@ -1,5 +1,15 @@ 2016-05-10 Adhemerval Zanella <adhemerval.zanella@linaro.org> + * sysdeps/powerpc/powerpc32/backtrace.c (is_sigtramp_address): Use + void* for argument type and use VDSO_SYMBOL macro. + (is_sigtramp_address_rt): Likewise. + (__backtrace): Setup expected frame pointer address for signal + handling. + * sysdeps/powerpc/powerpc64/backtrace.c (is_sigtramp_address): Use + void* for argumetn type and use VSDO_SYMBOL macro. + (__backtrace): Setup expected frame pointer address for signal + handling. + * sysdeps/unix/sysv/linux/writev.c: New file. * sysdeps/unix/sysv/linux/readv.c: New file. |