On 4/9/24 07:02, Richard Henderson wrote:
v1:
https://lore.kernel.org/qemu-devel/20230524054647.1093758-1-richard.hender...@linaro.org/
But v1 isn't particularly complet or korrect.
Disconnect fpstate from sigframe, just like the kernel does.
Return the separate portions of the frame from get_sigframe.
Alter all of the target fpu routines to access memory that
has already been translated and sized.
With the exception of patch 22, and with small nits in patches 1/19/23:
Reviewed-by: Paolo Bonzini <pbonz...@redhat.com>
r~
Richard Henderson (28):
target/i386: Add tcg/access.[ch]
target/i386: Convert do_fldt, do_fstt to X86Access
target/i386: Convert helper_{fbld,fbst}_ST0 to X86Access
target/i386: Convert do_fldenv to X86Access
target/i386: Convert do_fstenv to X86Access
target/i386: Convert do_fsave, do_frstor to X86Access
target/i386: Convert do_xsave_{fpu,mxcr,sse} to X86Access
target/i386: Convert do_xrstor_{fpu,mxcr,sse} to X86Access
tagret/i386: Convert do_fxsave, do_fxrstor to X86Access
target/i386: Convert do_xsave_* to X86Access
target/i386: Convert do_xrstor_* to X86Access
target/i386: Split out do_xsave_chk
target/i386: Add rbfm argument to cpu_x86_{xsave,xrstor}
target/i386: Add {hw,sw}_reserved to X86LegacyXSaveArea
linux-user/i386: Drop xfeatures_size from sigcontext arithmetic
linux-user/i386: Remove xfeatures from target_fpstate_fxsave
linux-user/i386: Replace target_fpstate_fxsave with X86LegacyXSaveArea
linux-user/i386: Split out struct target_fregs_state
linux-user/i386: Fix -mregparm=3 for signal delivery
linux-user/i386: Return boolean success from restore_sigcontext
linux-user/i386: Return boolean success from xrstor_sigcontext
linux-user/i386: Fix allocation and alignment of fp state
target/i386: Honor xfeatures in xrstor_sigcontext
target/i386: Convert do_xsave to X86Access
target/i386: Convert do_xrstor to X86Access
target/i386: Pass host pointer and size to cpu_x86_{fsave,frstor}
target/i386: Pass host pointer and size to cpu_x86_{fxsave,fxrstor}
target/i386: Pass host pointer and size to cpu_x86_{xsave,xrstor}
target/i386/cpu.h | 57 ++-
target/i386/tcg/access.h | 40 ++
linux-user/i386/signal.c | 669 ++++++++++++++++++-------------
target/i386/tcg/access.c | 160 ++++++++
target/i386/tcg/fpu_helper.c | 561 ++++++++++++++++----------
tests/tcg/x86_64/test-1648.c | 33 ++
target/i386/tcg/meson.build | 1 +
tests/tcg/x86_64/Makefile.target | 1 +
8 files changed, 1014 insertions(+), 508 deletions(-)
create mode 100644 target/i386/tcg/access.h
create mode 100644 target/i386/tcg/access.c
create mode 100644 tests/tcg/x86_64/test-1648.c