arm64/sme: Disable streaming mode and ZA when flushing CPU state
authorMark Brown <broonie@kernel.org>
Tue, 19 Apr 2022 11:22:30 +0000 (12:22 +0100)
committerCatalin Marinas <catalin.marinas@arm.com>
Fri, 22 Apr 2022 17:51:19 +0000 (18:51 +0100)
Both streaming mode and ZA may increase power consumption when they are
enabled and streaming mode makes many FPSIMD and SVE instructions undefined
which will cause problems for any kernel mode floating point so disable
both when we flush the CPU state. This covers both kernel_neon_begin() and
idle and after flushing the state a reload is always required anyway.

Signed-off-by: Mark Brown <broonie@kernel.org>
Reviewed-by: Catalin Marinas <catalin.marinas@arm.com>
Link: https://lore.kernel.org/r/20220419112247.711548-23-broonie@kernel.org
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
arch/arm64/kernel/fpsimd.c

index 94f06e9..9592cdd 100644 (file)
@@ -1759,6 +1759,15 @@ static void fpsimd_flush_cpu_state(void)
 {
        WARN_ON(!system_supports_fpsimd());
        __this_cpu_write(fpsimd_last_state.st, NULL);
+
+       /*
+        * Leaving streaming mode enabled will cause issues for any kernel
+        * NEON and leaving streaming mode or ZA enabled may increase power
+        * consumption.
+        */
+       if (system_supports_sme())
+               sme_smstop();
+
        set_thread_flag(TIF_FOREIGN_FPSTATE);
 }