diff options
| author | Josh Poimboeuf <jpoimboe@kernel.org> | 2023-02-14 08:05:47 +0100 |
|---|---|---|
| committer | Josh Poimboeuf <jpoimboe@kernel.org> | 2023-03-08 17:44:24 +0100 |
| commit | 243971885418fcf772f18019eb3fabadcf0205d1 (patch) | |
| tree | 5325ee8ea404b40c60c47de082c7d1b4960d0307 /arch/sh/include | |
| parent | powerpc/cpu: Mark start_secondary_resume() __noreturn (diff) | |
| download | linux-243971885418fcf772f18019eb3fabadcf0205d1.tar.xz linux-243971885418fcf772f18019eb3fabadcf0205d1.zip | |
sh/cpu: Make sure play_dead() doesn't return
play_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.
Link: https://lore.kernel.org/r/d0c3ff5349adfe8fd227acc236ae2c278a05eb4c.1676358308.git.jpoimboe@kernel.org
Signed-off-by: Josh Poimboeuf <jpoimboe@kernel.org>
Diffstat (limited to 'arch/sh/include')
| -rw-r--r-- | arch/sh/include/asm/smp-ops.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/arch/sh/include/asm/smp-ops.h b/arch/sh/include/asm/smp-ops.h index e27702130eb6..63866b1595a0 100644 --- a/arch/sh/include/asm/smp-ops.h +++ b/arch/sh/include/asm/smp-ops.h @@ -27,6 +27,7 @@ static inline void plat_smp_setup(void) static inline void play_dead(void) { mp_ops->play_dead(); + BUG(); } extern void register_smp_ops(struct plat_smp_ops *ops); |