Commit dabaa740 authored Nov 28, 2023 by Mark Brown Committed by yanhaitao Nov 29, 2023

arm64/fp: Reindent fpsimd_save()

mainline inclusion
from mainline-v5.16-rc1
commit 2d481bd3
category: feature
bugzilla: https://gitee.com/openeuler/kernel/issues/I8E73O
CVE: NA

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=2d481bd3b6361ed16c3ddeb58537f149623d30a0



-------------------------------------------------

Currently all the active code in fpsimd_save() is inside a check for
TIF_FOREIGN_FPSTATE. Reduce the indentation level by changing to return
from the function if TIF_FOREIGN_FPSTATE is set.

Signed-off-by: Mark Brown <broonie@kernel.org>
Link: https://lore.kernel.org/r/20211019172247.3045838-2-broonie@kernel.org


Signed-off-by: Will Deacon <will@kernel.org>
Signed-off-by: Wang ShaoBo <bobo.shaobowang@huawei.com>

parent d261693b

arch/arm64/kernel/fpsimd.c

+20 −18

Original line number	Diff line number	Diff line
		@@ -306,7 +306,9 @@ static void fpsimd_save(void)
		WARN_ON(!system_supports_fpsimd());
		WARN_ON(!have_cpu_fpsimd_context());

		if (!test_thread_flag(TIF_FOREIGN_FPSTATE)) {
		if (test_thread_flag(TIF_FOREIGN_FPSTATE))
		return;

		if (IS_ENABLED(CONFIG_ARM64_SVE) &&
		test_thread_flag(TIF_SVE)) {
		if (WARN_ON(sve_get_vl() != last->sve_vl)) {
		@@ -322,7 +324,7 @@ static void fpsimd_save(void)
		sve_save_state((char *)last->sve_state +
		sve_ffr_offset(last->sve_vl),
		&last->st->fpsr);
		} else
		} else {
		fpsimd_save_state(last->st);
		}
		}