On Tue, 23 Dec 2025 at 01:22, Mark Brown <[email protected]> wrote: > > The hypervisor copies of the SVE save and load functions are prototyped > with third arguments specifying FFR should be accessed but the assembly > functions overwrite whatever is supplied to unconditionally access FFR. > Remove this and use the supplied parameter. > > This has no effect currently since FFR is always present for SVE but will > be important for SME. > > Signed-off-by: Mark Brown <[email protected]>
Reviewed-by: Fuad Tabba <[email protected]> Cheers, /fuad > --- > arch/arm64/kvm/hyp/fpsimd.S | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/arch/arm64/kvm/hyp/fpsimd.S b/arch/arm64/kvm/hyp/fpsimd.S > index e950875e31ce..6e16cbfc5df2 100644 > --- a/arch/arm64/kvm/hyp/fpsimd.S > +++ b/arch/arm64/kvm/hyp/fpsimd.S > @@ -21,13 +21,11 @@ SYM_FUNC_START(__fpsimd_restore_state) > SYM_FUNC_END(__fpsimd_restore_state) > > SYM_FUNC_START(__sve_restore_state) > - mov x2, #1 > sve_load 0, x1, x2, 3 > ret > SYM_FUNC_END(__sve_restore_state) > > SYM_FUNC_START(__sve_save_state) > - mov x2, #1 > sve_save 0, x1, x2, 3 > ret > SYM_FUNC_END(__sve_save_state) > > -- > 2.47.3 >
