Messages in this thread | | | Subject | Re: [PATCH v5 2/3] arch/arm64: fpsimd: Introduce fpsimd_save_and_flush_cpu_state() and use it | From | Marc Zyngier <> | Date | Tue, 4 Jun 2019 12:52:27 +0100 |
| |
On 04/06/2019 12:12, Catalin Marinas wrote: > On Tue, May 21, 2019 at 06:21:38PM +0100, Julien Grall wrote: >> The only external user of fpsimd_save() and fpsimd_flush_cpu_state() is >> the KVM FPSIMD code. >> >> A following patch will introduce a mechanism to acquire owernship of the >> FPSIMD/SVE context for performing context management operations. Rather >> than having to export the new helpers to get/put the context, we can just >> introduce a new function to combine fpsimd_save() and >> fpsimd_flush_cpu_state(). >> >> This has also the advantage to remove any external call of fpsimd_save() >> and fpsimd_flush_cpu_state(), so they can be turned static. >> >> Lastly, the new function can also be used in the PM notifier. >> >> Signed-off-by: Julien Grall <julien.grall@arm.com> >> Reviewed-by: Dave Martin <Dave.Martin@arm.com> > [...] >> --- >> arch/arm64/include/asm/fpsimd.h | 4 +--- >> arch/arm64/kernel/fpsimd.c | 17 +++++++++++++---- >> arch/arm64/kvm/fpsimd.c | 4 +--- >> 3 files changed, 15 insertions(+), 10 deletions(-) > > Marc, since this touches KVM, can I get an ack on this patch? I plan to > queue the series.
Sure. Please add my
Acked-by: Marc Zyngier <marc.zyngier@arm.com>
to the whole series.
Thanks,
M. -- Jazz is not dead. It just smells funny...
| |