arm64/fpsimd: Don't flush SME register hardware state along with thread
authorMark Brown <broonie@kernel.org>
Mon, 24 Jul 2023 13:09:19 +0000 (14:09 +0100)
committerCatalin Marinas <catalin.marinas@arm.com>
Wed, 26 Jul 2023 17:25:09 +0000 (18:25 +0100)
commit3421ddbe6d64e55b76a67d57cef9e950b3d873ba
tree5741df563dcd2739f3ceb891bff1accb896e3d0b
parentd4d5be94a87872421ea2569044092535aff0b886
arm64/fpsimd: Don't flush SME register hardware state along with thread

We recently changed the fpsimd thread flush to flush the physical SME
state as well as the thread state for the current thread.  Unfortunately
this leads to intermittent corruption in interaction with the lazy
FPSIMD register switching.  When under heavy load such as can be
triggered by the startup phase of fp-stress it is possible that the
current thread may not be scheduled prior to returning to userspace, and
indeed we may end up returning to the last thread that was scheduled on
the PE without ever exiting the kernel to any other task.  If that
happens then we will not reload the register state from memory, leading
to loss of any SME register state.

Since this was purely an attempt to defensively close off potential
problems revert the change.

Fixes: af3215fd0230 ("arm64/fpsimd: Exit streaming mode when flushing tasks")
Signed-off-by: Mark Brown <broonie@kernel.org>
Link: https://lore.kernel.org/r/20230724-arm64-dont-flush-smstate-v1-1-9a8b637ace6c@kernel.org
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
arch/arm64/kernel/fpsimd.c