diff options
| author | Richard Henderson <richard.henderson@linaro.org> | 2024-04-08 17:31:05 -1000 |
|---|---|---|
| committer | Richard Henderson <richard.henderson@linaro.org> | 2024-05-26 15:45:27 -0700 |
| commit | 76d8d0f85caf629b4df314e656d20ad6565bab9b (patch) | |
| tree | 89ebae361215da488a5da054a2312a3bc0368e18 /linux-user | |
| parent | d5dc3a927ae7e64bc998d9aa29020426b4e97f8a (diff) | |
| download | qemu-76d8d0f85caf629b4df314e656d20ad6565bab9b.zip qemu-76d8d0f85caf629b4df314e656d20ad6565bab9b.tar.gz qemu-76d8d0f85caf629b4df314e656d20ad6565bab9b.tar.bz2 | |
target/i386: Pass host pointer and size to cpu_x86_{fsave,frstor}
We have already validated the memory region in the course of
validating the signal frame. No need to do it again within
the helper function.
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Diffstat (limited to 'linux-user')
| -rw-r--r-- | linux-user/i386/signal.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/linux-user/i386/signal.c b/linux-user/i386/signal.c index ab760db..dfbb811 100644 --- a/linux-user/i386/signal.c +++ b/linux-user/i386/signal.c @@ -373,7 +373,7 @@ static void setup_sigcontext(CPUX86State *env, __put_user(env->regs[R_ESP], &sc->esp_at_signal); __put_user(env->segs[R_SS].selector, (uint32_t *)&sc->ss); - cpu_x86_fsave(env, fpstate_addr, 1); + cpu_x86_fsave(env, fpstate, sizeof(*fpstate)); fpstate->status = fpstate->swd; magic = (fpkind == FPSTATE_FSAVE ? 0 : 0xffff); __put_user(magic, &fpstate->magic); @@ -702,7 +702,7 @@ static bool frstor_sigcontext(CPUX86State *env, FPStateKind fpkind, * the merge within ENV by loading XSTATE/FXSTATE first, then * overriding with the FSTATE afterward. */ - cpu_x86_frstor(env, fpstate_addr, 1); + cpu_x86_frstor(env, fpstate, sizeof(*fpstate)); return true; } #endif |