On Mon, Nov 22, 2021 at 03:57:32PM +0000, Marc Zyngier wrote:

Zenghui Yu [off-list ref] wrote:

quoted

Nit: This removes the only user of __sve_save_state() helper. Should we
still keep it in fpsimd.S?

I was in two minds about that, as I'd like to eventually be able to
use SVE for protected guests, where the hypervisor itself has to be in
charge of the FP/SVE save-restore.

But that's probably several months away, and I can always revert a
deletion patch if I need to, so let's get rid of it now.

While we're on the subject of potential future work we might in future
want to not disable SVE on every syscall if (as seems likely) it turns
out that that's more performant for small vector lengths which would
mean some minor reshuffling here to do something like convert the saved
state to FPSIMD and drop TIF_SVE in _vcpu_load_fp().  As with using SVE
in protected guests that can just be done when needed though.