work around gdb issues recognizing sigreturn trampoline on x86_64
authorRich Felker <dalias@aerifal.cx>
Sun, 13 Nov 2016 00:43:37 +0000 (19:43 -0500)
committerRich Felker <dalias@aerifal.cx>
Sun, 13 Nov 2016 00:54:43 +0000 (19:54 -0500)
commit54991729fd1e3d3a0cb71884d758d86afe6da9e0
tree220f219af4477a552995f66de3d26a71e603c26b
parent15094943050eb9a564f409323070e50b40f78816
work around gdb issues recognizing sigreturn trampoline on x86_64

gdb can only backtrace/unwind across signal handlers if it recognizes
the sa_restorer trampoline. for x86_64, gdb first attempts to
determine the symbol name for the function in which the program
counter resides and match it against "__restore_rt". if no name can be
found (e.g. in the case of a stripped binary), the exact instruction
sequence is matched instead.

when matching the function name, however, gdb's unwind code wrongly
considers the interval [sym,sym+size] rather than [sym,sym+size).
thus, if __restore_rt begins immediately after another function, gdb
wrongly identifies pc as lying within the previous adjacent function.
this patch adds a nop before __restore_rt to preclude that
possibility. it also removes the symbol name __restore and replaces it
with a macro since the stability of whether gdb identifies the
function as __restore_rt or __restore is not clear.

for the no-symbols case, the instruction sequence is changed to use
%rax rather than %eax to match what gdb expects.

based on patch by Szabolcs Nagy, with extended description and
corresponding x32 changes added.
arch/x32/ksigaction.h [new file with mode: 0644]
arch/x86_64/ksigaction.h [new file with mode: 0644]
src/signal/x32/restore.s
src/signal/x86_64/restore.s