[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <167848169468.5837.15267243328787227151.tip-bot2@tip-bot2>
Date: Fri, 10 Mar 2023 20:54:54 -0000
From: "tip-bot2 for Josh Poimboeuf" <tip-bot2@...utronix.de>
To: linux-tip-commits@...r.kernel.org
Cc: Guo Ren <guoren@...nel.org>, Josh Poimboeuf <jpoimboe@...nel.org>,
x86@...nel.org, linux-kernel@...r.kernel.org
Subject: [tip: objtool/core] csky/cpu: Make sure arch_cpu_idle_dead() doesn't return
The following commit has been merged into the objtool/core branch of tip:
Commit-ID: 1b2442a835a0acc7d76ae2f48ca7a94f962b29c8
Gitweb: https://git.kernel.org/tip/1b2442a835a0acc7d76ae2f48ca7a94f962b29c8
Author: Josh Poimboeuf <jpoimboe@...nel.org>
AuthorDate: Mon, 13 Feb 2023 23:05:39 -08:00
Committer: Josh Poimboeuf <jpoimboe@...nel.org>
CommitterDate: Mon, 06 Mar 2023 15:34:05 -08:00
csky/cpu: Make sure arch_cpu_idle_dead() doesn't return
arch_cpu_idle_dead() doesn't return. Make that more explicit with a
BUG().
BUG() is preferable to unreachable() because BUG() is a more explicit
failure mode and avoids undefined behavior like falling off the edge of
the function into whatever code happens to be next.
Acked-by: Guo Ren <guoren@...nel.org>
Link: https://lore.kernel.org/r/1e9ecc3d248e82973e80bc336fc9f97e3ba2708d.1676358308.git.jpoimboe@kernel.org
Signed-off-by: Josh Poimboeuf <jpoimboe@...nel.org>
---
arch/csky/kernel/smp.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/csky/kernel/smp.c b/arch/csky/kernel/smp.c
index b45d107..0ec20ef 100644
--- a/arch/csky/kernel/smp.c
+++ b/arch/csky/kernel/smp.c
@@ -317,5 +317,7 @@ void arch_cpu_idle_dead(void)
"jmpi csky_start_secondary"
:
: "r" (secondary_stack));
+
+ BUG();
}
#endif
Powered by blists - more mailing lists