[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH v1 1/4] target/ppc: fix signal delivery for ppc64a
From: |
Alex Bennée |
Subject: |
Re: [Qemu-arm] [PATCH v1 1/4] target/ppc: fix signal delivery for ppc64abi32 |
Date: |
Tue, 10 Sep 2019 20:45:20 +0100 |
User-agent: |
mu4e 1.3.4; emacs 27.0.50 |
Alex Bennée <address@hidden> writes:
> We were incorrectly setting NIP resulting in a segfault. This fixes
> linux-test for this ABI.
Oops, that was at the bottom of my tree for fixing ppc64abi32 which
showed up broken when testing/next enabled the TCG tests for it.
>
> Signed-off-by: Alex Bennée <address@hidden>
> ---
> linux-user/ppc/signal.c | 4 +++-
> tests/tcg/configure.sh | 1 +
> tests/tcg/multiarch/Makefile.target | 5 -----
> 3 files changed, 4 insertions(+), 6 deletions(-)
>
> diff --git a/linux-user/ppc/signal.c b/linux-user/ppc/signal.c
> index 619a56950df..5b82af6cb62 100644
> --- a/linux-user/ppc/signal.c
> +++ b/linux-user/ppc/signal.c
> @@ -501,7 +501,9 @@ void setup_rt_frame(int sig, struct target_sigaction *ka,
> int i, err = 0;
> #if defined(TARGET_PPC64)
> struct target_sigcontext *sc = 0;
> +#if !defined(TARGET_ABI32)
> struct image_info *image = ((TaskState *)thread_cpu->opaque)->info;
> +#endif
> #endif
>
> rt_sf_addr = get_sigframe(ka, env, sizeof(*rt_sf));
> @@ -557,7 +559,7 @@ void setup_rt_frame(int sig, struct target_sigaction *ka,
> env->gpr[5] = (target_ulong) h2g(&rt_sf->uc);
> env->gpr[6] = (target_ulong) h2g(rt_sf);
>
> -#if defined(TARGET_PPC64)
> +#if defined(TARGET_PPC64) && !defined(TARGET_ABI32)
> if (get_ppc64_abi(image) < 2) {
> /* ELFv1 PPC64 function pointers are pointers to OPD entries. */
> struct target_func_ptr *handler =
> diff --git a/tests/tcg/configure.sh b/tests/tcg/configure.sh
> index 6c4a471aeae..e8a1a1495fc 100755
> --- a/tests/tcg/configure.sh
> +++ b/tests/tcg/configure.sh
> @@ -54,6 +54,7 @@ fi
> : ${cross_cc_cflags_ppc="-m32"}
> : ${cross_cc_ppc64="powerpc-linux-gnu-gcc"}
> : ${cross_cc_cflags_ppc64="-m64"}
> +: ${cross_cc_cflags_ppc64abi32="-mcpu=power8"}
> : ${cross_cc_ppc64le="powerpc64le-linux-gnu-gcc"}
> : ${cross_cc_cflags_s390x="-m64"}
> : ${cross_cc_cflags_sparc="-m32 -mv8plus -mcpu=ultrasparc"}
> diff --git a/tests/tcg/multiarch/Makefile.target
> b/tests/tcg/multiarch/Makefile.target
> index 6b1e30e2fec..e6893b2e283 100644
> --- a/tests/tcg/multiarch/Makefile.target
> +++ b/tests/tcg/multiarch/Makefile.target
> @@ -12,11 +12,6 @@ VPATH += $(MULTIARCH_SRC)
> MULTIARCH_SRCS =$(notdir $(wildcard $(MULTIARCH_SRC)/*.c))
> MULTIARCH_TESTS =$(MULTIARCH_SRCS:.c=)
>
> -# FIXME: ppc64abi32 linux-test seems to have issues but the other basic
> tests work
> -ifeq ($(TARGET_NAME),ppc64abi32)
> -BROKEN_TESTS = linux-test
> -endif
> -
> # Update TESTS
> TESTS += $(filter-out $(BROKEN_TESTS), $(MULTIARCH_TESTS))
--
Alex Bennée
- [Qemu-arm] [PATCH v1 0/4] ELF and (macro) safety, Alex Bennée, 2019/09/10
- [Qemu-arm] [PATCH v1 1/4] target/ppc: fix signal delivery for ppc64abi32, Alex Bennée, 2019/09/10
- Re: [Qemu-arm] [PATCH v1 1/4] target/ppc: fix signal delivery for ppc64abi32,
Alex Bennée <=
- [Qemu-arm] [PATCH v1 3/4] elf: move elf_ops.h into include/elf/ and rename, Alex Bennée, 2019/09/10
- [Qemu-arm] [PATCH v1 4/4] elf: move ELF_ARCH definition to elf-arch.h, Alex Bennée, 2019/09/10
- Re: [Qemu-arm] [Qemu-devel] [PATCH v1 4/4] elf: move ELF_ARCH definition to elf-arch.h, Aleksandar Markovic, 2019/09/10
- Re: [Qemu-arm] [Qemu-devel] [PATCH v1 4/4] elf: move ELF_ARCH definition to elf-arch.h, Alex Bennée, 2019/09/11
- Re: [Qemu-arm] [Qemu-devel] [PATCH v1 4/4] elf: move ELF_ARCH definition to elf-arch.h, Aleksandar Markovic, 2019/09/13
- Re: [Qemu-arm] [Qemu-devel] [PATCH v1 4/4] elf: move ELF_ARCH definition to elf-arch.h, Richard Henderson, 2019/09/14
- Re: [Qemu-arm] [Qemu-devel] [PATCH v1 4/4] elf: move ELF_ARCH definition to elf-arch.h, Alex Bennée, 2019/09/14
- Re: [Qemu-arm] [Qemu-devel] [PATCH v1 4/4] elf: move ELF_ARCH definition to elf-arch.h, Richard Henderson, 2019/09/14