diff options
author | Thomas Huth <thuth@redhat.com> | 2023-09-14 13:33:05 +0200 |
---|---|---|
committer | Peter Maydell <peter.maydell@linaro.org> | 2023-09-18 13:24:07 +0100 |
commit | 121d4f3473e8d730570688d2c59abed84fcd31a4 (patch) | |
tree | dd91e0a03dc087114e7be5f11d38ce4442dc69a5 | |
parent | bba09464e401e54b6b1609c7707d615e8cb51fba (diff) |
Pass siginfo_t->si_addr to the reginfo_init() function
On s390x, we need the si_addr from the siginfo_t to get to
the address of the illegal instruction (the PSW address in
the ucontext_t is already pointing to the next instruction
there). So let's prepare for that situation and pass the
si_addr to the reginfo_init() function everywhere.
Signed-off-by: Thomas Huth <thuth@redhat.com>
Message-id: 20230914113311.379537-2-thuth@redhat.com
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
-rw-r--r-- | risu.c | 12 | ||||
-rw-r--r-- | risu.h | 2 | ||||
-rw-r--r-- | risu_reginfo_aarch64.c | 2 | ||||
-rw-r--r-- | risu_reginfo_arm.c | 2 | ||||
-rw-r--r-- | risu_reginfo_i386.c | 2 | ||||
-rw-r--r-- | risu_reginfo_loongarch64.c | 2 | ||||
-rw-r--r-- | risu_reginfo_m68k.c | 2 | ||||
-rw-r--r-- | risu_reginfo_ppc64.c | 2 |
8 files changed, 13 insertions, 13 deletions
@@ -106,14 +106,14 @@ static void respond(RisuResult r) } } -static RisuResult send_register_info(void *uc) +static RisuResult send_register_info(void *uc, void *siaddr) { uint64_t paramreg; RisuResult res; RisuOp op; void *extra; - reginfo_init(&ri[MASTER], uc); + reginfo_init(&ri[MASTER], uc, siaddr); op = get_risuop(&ri[MASTER]); /* Write a header with PC/op to keep in sync */ @@ -178,7 +178,7 @@ static void master_sigill(int sig, siginfo_t *si, void *uc) RisuResult r; signal_count++; - r = send_register_info(uc); + r = send_register_info(uc, si->si_addr); if (r == RES_OK) { advance_pc(uc); } else { @@ -232,13 +232,13 @@ static RisuResult recv_register_info(struct reginfo *ri) } } -static RisuResult recv_and_compare_register_info(void *uc) +static RisuResult recv_and_compare_register_info(void *uc, void *siaddr) { uint64_t paramreg; RisuResult res; RisuOp op; - reginfo_init(&ri[APPRENTICE], uc); + reginfo_init(&ri[APPRENTICE], uc, siaddr); res = recv_register_info(&ri[MASTER]); if (res != RES_OK) { @@ -315,7 +315,7 @@ static void apprentice_sigill(int sig, siginfo_t *si, void *uc) RisuResult r; signal_count++; - r = recv_and_compare_register_info(uc); + r = recv_and_compare_register_info(uc, si->si_addr); if (r == RES_OK) { advance_pc(uc); } else { @@ -115,7 +115,7 @@ RisuOp get_risuop(struct reginfo *ri); uintptr_t get_pc(struct reginfo *ri); /* initialize structure from a ucontext */ -void reginfo_init(struct reginfo *ri, ucontext_t *uc); +void reginfo_init(struct reginfo *ri, ucontext_t *uc, void *siaddr); /* return 1 if structs are equal, 0 otherwise. */ int reginfo_is_eq(struct reginfo *r1, struct reginfo *r2); diff --git a/risu_reginfo_aarch64.c b/risu_reginfo_aarch64.c index be47980..1244454 100644 --- a/risu_reginfo_aarch64.c +++ b/risu_reginfo_aarch64.c @@ -82,7 +82,7 @@ int reginfo_size(struct reginfo *ri) } /* reginfo_init: initialize with a ucontext */ -void reginfo_init(struct reginfo *ri, ucontext_t *uc) +void reginfo_init(struct reginfo *ri, ucontext_t *uc, void *siaddr) { int i; struct _aarch64_ctx *ctx, *extra = NULL; diff --git a/risu_reginfo_arm.c b/risu_reginfo_arm.c index 202120b..85a39ac 100644 --- a/risu_reginfo_arm.c +++ b/risu_reginfo_arm.c @@ -118,7 +118,7 @@ static void reginfo_init_vfp(struct reginfo *ri, ucontext_t *uc) } } -void reginfo_init(struct reginfo *ri, ucontext_t *uc) +void reginfo_init(struct reginfo *ri, ucontext_t *uc, void *siaddr) { memset(ri, 0, sizeof(*ri)); /* necessary for memcmp later */ diff --git a/risu_reginfo_i386.c b/risu_reginfo_i386.c index e9730be..834b2ed 100644 --- a/risu_reginfo_i386.c +++ b/risu_reginfo_i386.c @@ -102,7 +102,7 @@ static void *xsave_feature_buf(struct _xstate *xs, int feature) } /* reginfo_init: initialize with a ucontext */ -void reginfo_init(struct reginfo *ri, ucontext_t *uc) +void reginfo_init(struct reginfo *ri, ucontext_t *uc, void *siaddr) { int i, nvecregs; struct _fpstate *fp; diff --git a/risu_reginfo_loongarch64.c b/risu_reginfo_loongarch64.c index af6ab77..16384f1 100644 --- a/risu_reginfo_loongarch64.c +++ b/risu_reginfo_loongarch64.c @@ -81,7 +81,7 @@ static int parse_extcontext(struct sigcontext *sc, struct extctx_layout *extctx) } /* reginfo_init: initialize with a ucontext */ -void reginfo_init(struct reginfo *ri, ucontext_t *context) +void reginfo_init(struct reginfo *ri, ucontext_t *context, void *siaddr) { int i; struct ucontext *uc = (struct ucontext *)context; diff --git a/risu_reginfo_m68k.c b/risu_reginfo_m68k.c index 4c25e77..e29da84 100644 --- a/risu_reginfo_m68k.c +++ b/risu_reginfo_m68k.c @@ -33,7 +33,7 @@ int reginfo_size(struct reginfo *ri) } /* reginfo_init: initialize with a ucontext */ -void reginfo_init(struct reginfo *ri, ucontext_t *uc) +void reginfo_init(struct reginfo *ri, ucontext_t *uc, void *siaddr) { int i; memset(ri, 0, sizeof(*ri)); diff --git a/risu_reginfo_ppc64.c b/risu_reginfo_ppc64.c index 9899b36..bbdd63c 100644 --- a/risu_reginfo_ppc64.c +++ b/risu_reginfo_ppc64.c @@ -42,7 +42,7 @@ int reginfo_size(struct reginfo *ri) } /* reginfo_init: initialize with a ucontext */ -void reginfo_init(struct reginfo *ri, ucontext_t *uc) +void reginfo_init(struct reginfo *ri, ucontext_t *uc, void *siaddr) { int i; |