On Sat, Apr 2, 2022 at 1:21 AM Sören Tempel <soe...@soeren-tempel.net> wrote:
>
> Thanks for committing a first fix! Unfortunately, your changes don't
> work on ppc64le musl since you are now still using .regs on ppc64le the
> include of asm/ptrace.h (as added in the v1 of my patch) is missing.
> Hence, your patch fails to compile on ppc64le musl with the following
> error message:
>
>         go-signal.c:230:63: error: invalid use of undefined type 'struct 
> pt_regs'
>           230 |         ret.sigpc = 
> ((ucontext_t*)(context))->uc_mcontext.regs->nip;
>
> If you want to continue using .regs on ppc64le an include of
> asm/ptrace.h is needed since both glibc and musl declare `struct
> pt_regs` as an incomplete type (with glibc asm/ptrace.h is included
> indirectly by other headers used by go-signal.c it seems).
>
> See https://gcc.gnu.org/pipermail/gcc-patches/2022-January/587520.html
>
> Would be nice if this could be fixed :)

Sorry, I guess I misread your patch.

What is the right standalone code for the PPC64 musl case?  Thanks.

Ian

Reply via email to