This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 1/7] powerpc: Add CFI information on indirect syscall


Adhemerval Zanella <adhemerval.zanella@linaro.org> writes:

> This patch the required CFI information on powerpc indirect syscall
> so backtrace works correctly on signal handler.
>
> Checked on powerpc-linux-gnu and powerpc64le-linux-gnu.
>
> 	* sysdeps/unix/sysv/linux/powerpc/syscall.S (syscall): Add CFI
> 	information.
> ---
>  ChangeLog                                 | 5 +++++
>  sysdeps/unix/sysv/linux/powerpc/syscall.S | 1 +
>  2 files changed, 6 insertions(+)
>
> diff --git a/sysdeps/unix/sysv/linux/powerpc/syscall.S b/sysdeps/unix/sysv/linux/powerpc/syscall.S
> index 2da91721be..e30f461a17 100644
> --- a/sysdeps/unix/sysv/linux/powerpc/syscall.S
> +++ b/sysdeps/unix/sysv/linux/powerpc/syscall.S
> @@ -19,6 +19,7 @@
>  
>  ENTRY (syscall)
>  	ABORT_TRANSACTION
> +	cfi_def_cfa_offset (0)

Why is this necessary in a function that is defined via ENTRY (or
ENTRY_TOCLESS) that already has cfi_startproc?

Does the error you found mean that parameter simple from cfi_startproc is being
set by mistake?

-- 
Tulio Magno


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]