All of lore.kernel.org
 help / color / mirror / Atom feed
From: "tip-bot2 for Josh Poimboeuf" <tip-bot2@linutronix.de>
To: linux-tip-commits@vger.kernel.org
Cc: Josh Poimboeuf <jpoimboe@kernel.org>,
	x86@kernel.org, linux-kernel@vger.kernel.org
Subject: [tip: objtool/core] x86/cpu: Make sure play_dead() doesn't return
Date: Fri, 10 Mar 2023 20:54:49 -0000	[thread overview]
Message-ID: <167848168944.5837.18023341418882320171.tip-bot2@tip-bot2> (raw)
In-Reply-To: <11e6ac1cf10f92967882926e3ac16287b50642f2.1676358308.git.jpoimboe@kernel.org>

The following commit has been merged into the objtool/core branch of tip:

Commit-ID:     a02f50b573b3c90a3633a5ab67435fe4c0de144d
Gitweb:        https://git.kernel.org/tip/a02f50b573b3c90a3633a5ab67435fe4c0de144d
Author:        Josh Poimboeuf <jpoimboe@kernel.org>
AuthorDate:    Mon, 13 Feb 2023 23:05:51 -08:00
Committer:     Josh Poimboeuf <jpoimboe@kernel.org>
CommitterDate: Wed, 08 Mar 2023 08:44:26 -08:00

x86/cpu: Make sure play_dead() doesn't return

After commit 076cbf5d2163 ("x86/xen: don't let xen_pv_play_dead()
return"), play_dead() never returns.  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/11e6ac1cf10f92967882926e3ac16287b50642f2.1676358308.git.jpoimboe@kernel.org
Signed-off-by: Josh Poimboeuf <jpoimboe@kernel.org>
---
 arch/x86/include/asm/smp.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/x86/include/asm/smp.h b/arch/x86/include/asm/smp.h
index b4dbb20..8f628e0 100644
--- a/arch/x86/include/asm/smp.h
+++ b/arch/x86/include/asm/smp.h
@@ -96,6 +96,7 @@ static inline void __cpu_die(unsigned int cpu)
 static inline void play_dead(void)
 {
 	smp_ops.play_dead();
+	BUG();
 }
 
 static inline void smp_send_reschedule(int cpu)

  parent reply	other threads:[~2023-03-10 20:55 UTC|newest]

Thread overview: 289+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-14  7:05 [PATCH v2 00/24] cpu,sched: Mark arch_cpu_idle_dead() __noreturn Josh Poimboeuf
2023-02-14  7:05 ` Josh Poimboeuf
2023-02-14  7:05 ` Josh Poimboeuf
2023-02-14  7:05 ` Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 01/24] alpha/cpu: Expose arch_cpu_idle_dead()'s prototype declaration Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-03-10 20:54   ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 02/24] alpha/cpu: Make sure arch_cpu_idle_dead() doesn't return Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-03-10 20:54   ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 03/24] arm/cpu: " Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14 11:15   ` Russell King (Oracle)
2023-02-14 11:15     ` Russell King (Oracle)
2023-02-14 11:15     ` Russell King (Oracle)
2023-02-14 11:15     ` Russell King (Oracle)
2023-02-14 18:39     ` Josh Poimboeuf
2023-02-14 18:39       ` Josh Poimboeuf
2023-02-14 18:39       ` Josh Poimboeuf
2023-02-14 18:39       ` Josh Poimboeuf
2023-02-14 18:39       ` Josh Poimboeuf
2023-02-14 22:01       ` Russell King (Oracle)
2023-02-14 22:01         ` Russell King (Oracle)
2023-02-14 22:01         ` Russell King (Oracle)
2023-02-14 22:01         ` Russell King (Oracle)
2023-02-16 18:38         ` [PATCH v2.1 03/24] arm/cpu: Add unreachable() to arch_cpu_idle_dead() Josh Poimboeuf
2023-02-16 18:38           ` Josh Poimboeuf
2023-02-16 18:38           ` Josh Poimboeuf
2023-02-16 18:38           ` Josh Poimboeuf
2023-02-16 18:38           ` Josh Poimboeuf
2023-03-10 20:54           ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 04/24] arm64/cpu: Mark cpu_die() __noreturn Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  8:13   ` Philippe Mathieu-Daudé
2023-02-14  8:13     ` Philippe Mathieu-Daudé
2023-02-14  8:13     ` Philippe Mathieu-Daudé
2023-02-14  8:13     ` Philippe Mathieu-Daudé
2023-02-15 13:09     ` Mark Rutland
2023-02-15 13:09       ` Mark Rutland
2023-02-15 13:09       ` Mark Rutland
2023-02-15 13:09       ` Mark Rutland
2023-02-15 19:45       ` Josh Poimboeuf
2023-02-15 19:45         ` Josh Poimboeuf
2023-02-15 19:45         ` Josh Poimboeuf
2023-02-15 19:45         ` Josh Poimboeuf
2023-02-15 19:45         ` Josh Poimboeuf
2023-02-16 18:41         ` [PATCH v2.1 " Josh Poimboeuf
2023-02-16 18:41           ` Josh Poimboeuf
2023-02-16 18:41           ` Josh Poimboeuf
2023-02-16 18:41           ` Josh Poimboeuf
2023-02-16 18:41           ` Josh Poimboeuf
2023-03-02 10:55           ` Philippe Mathieu-Daudé
2023-03-02 10:55             ` Philippe Mathieu-Daudé
2023-03-02 10:55             ` Philippe Mathieu-Daudé
2023-03-02 10:55             ` Philippe Mathieu-Daudé
2023-03-02 10:55             ` Philippe Mathieu-Daudé
2023-03-10 20:54           ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 05/24] csky/cpu: Make sure arch_cpu_idle_dead() doesn't return Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-03-10 20:54   ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 06/24] ia64/cpu: Mark play_dead() __noreturn Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  8:07   ` Philippe Mathieu-Daudé
2023-02-14  8:07     ` Philippe Mathieu-Daudé
2023-02-14  8:07     ` Philippe Mathieu-Daudé
2023-02-14  8:07     ` Philippe Mathieu-Daudé
2023-03-10 20:54   ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 07/24] loongarch/cpu: Make sure play_dead() doesn't return Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-03-10 20:54   ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 08/24] loongarch/cpu: Mark play_dead() __noreturn Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  8:06   ` Philippe Mathieu-Daudé
2023-02-14  8:06     ` Philippe Mathieu-Daudé
2023-02-14  8:06     ` Philippe Mathieu-Daudé
2023-02-14  8:06     ` Philippe Mathieu-Daudé
2023-03-10 20:54   ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 09/24] mips/cpu: Expose play_dead()'s prototype definition Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:46   ` Philippe Mathieu-Daudé
2023-02-14  7:46     ` Philippe Mathieu-Daudé
2023-02-14  7:46     ` Philippe Mathieu-Daudé
2023-02-14  7:46     ` Philippe Mathieu-Daudé
2023-02-14 18:11     ` Josh Poimboeuf
2023-02-14 18:11       ` Josh Poimboeuf
2023-02-14 18:11       ` Josh Poimboeuf
2023-02-14 18:11       ` Josh Poimboeuf
2023-02-15  8:29       ` Philippe Mathieu-Daudé
2023-02-15  8:29         ` Philippe Mathieu-Daudé
2023-02-15  8:29         ` Philippe Mathieu-Daudé
2023-02-15  8:29         ` Philippe Mathieu-Daudé
2023-02-16 18:42         ` [PATCH v2.1 " Josh Poimboeuf
2023-02-16 18:42           ` Josh Poimboeuf
2023-02-16 18:42           ` Josh Poimboeuf
2023-02-16 18:42           ` Josh Poimboeuf
2023-03-01 18:16           ` Josh Poimboeuf
2023-03-01 18:16             ` Josh Poimboeuf
2023-03-01 18:16             ` Josh Poimboeuf
2023-03-01 18:16             ` Josh Poimboeuf
2023-03-02 10:54             ` Philippe Mathieu-Daudé
2023-03-02 10:54               ` Philippe Mathieu-Daudé
2023-03-02 10:54               ` Philippe Mathieu-Daudé
2023-03-02 10:54               ` Philippe Mathieu-Daudé
2023-03-10 20:54           ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 10/24] mips/cpu: Make sure play_dead() doesn't return Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:50   ` Philippe Mathieu-Daudé
2023-02-14  7:50     ` Philippe Mathieu-Daudé
2023-02-14  7:50     ` Philippe Mathieu-Daudé
2023-02-14  7:50     ` Philippe Mathieu-Daudé
2023-03-10 20:54   ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 11/24] mips/cpu: Mark play_dead() __noreturn Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:50   ` Philippe Mathieu-Daudé
2023-02-14  7:50     ` Philippe Mathieu-Daudé
2023-02-14  7:50     ` Philippe Mathieu-Daudé
2023-02-14  7:50     ` Philippe Mathieu-Daudé
2023-03-10 20:54   ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 12/24] powerpc/cpu: Mark start_secondary_resume() __noreturn Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:19   ` Christophe Leroy
2023-02-14  7:19     ` Christophe Leroy
2023-02-14  7:19     ` Christophe Leroy
2023-02-14  7:19     ` Christophe Leroy
2023-02-14  7:19     ` Christophe Leroy
2023-03-10 20:54   ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 13/24] sh/cpu: Make sure play_dead() doesn't return Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:57   ` Philippe Mathieu-Daudé
2023-02-14  7:57   ` Philippe Mathieu-Daudé
2023-02-14  7:57     ` Philippe Mathieu-Daudé
2023-02-14  7:57     ` Philippe Mathieu-Daudé
2023-02-14 18:28     ` Josh Poimboeuf
2023-02-14 18:28       ` Josh Poimboeuf
2023-02-14 18:28       ` Josh Poimboeuf
2023-02-14 18:28       ` Josh Poimboeuf
2023-03-10 20:54   ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 14/24] sh/cpu: Mark play_dead() __noreturn Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-03-10 20:54   ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 15/24] sh/cpu: Expose arch_cpu_idle_dead()'s prototype definition Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-03-10 20:54   ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 16/24] sparc/cpu: Mark cpu_play_dead() __noreturn Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:52   ` Philippe Mathieu-Daudé
2023-02-14  7:52     ` Philippe Mathieu-Daudé
2023-02-14  7:52     ` Philippe Mathieu-Daudé
2023-02-14  7:52     ` Philippe Mathieu-Daudé
2023-03-10 20:54   ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 17/24] x86/cpu: Make sure play_dead() doesn't return Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  8:05   ` Philippe Mathieu-Daudé
2023-02-14  8:05     ` Philippe Mathieu-Daudé
2023-02-14  8:05     ` Philippe Mathieu-Daudé
2023-02-14  8:05   ` Philippe Mathieu-Daudé
2023-03-10 20:54   ` tip-bot2 for Josh Poimboeuf [this message]
2023-02-14  7:05 ` [PATCH v2 18/24] x86/cpu: Mark play_dead() __noreturn Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-03-10 20:54   ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 19/24] xtensa/cpu: Make sure cpu_die() doesn't return Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:55   ` Philippe Mathieu-Daudé
2023-02-14  7:55   ` Philippe Mathieu-Daudé
2023-02-14  7:55     ` Philippe Mathieu-Daudé
2023-02-14  7:55     ` Philippe Mathieu-Daudé
2023-02-14 18:23     ` Josh Poimboeuf
2023-02-14 18:23       ` Josh Poimboeuf
2023-02-14 18:23       ` Josh Poimboeuf
2023-02-14 18:23       ` Josh Poimboeuf
2023-02-14 19:29       ` Steven Rostedt
2023-02-14 19:29         ` Steven Rostedt
2023-02-14 19:29         ` Steven Rostedt
2023-02-14 19:29         ` Steven Rostedt
2023-02-14 19:48       ` Max Filippov
2023-02-14 19:48         ` Max Filippov
2023-02-14 19:48         ` Max Filippov
2023-02-14 19:48         ` Max Filippov
2023-02-14 20:19         ` Josh Poimboeuf
2023-02-14 20:19           ` Josh Poimboeuf
2023-02-14 20:19           ` Josh Poimboeuf
2023-02-14 20:19           ` Josh Poimboeuf
2023-02-14 20:19           ` Josh Poimboeuf
2023-03-10 20:54   ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 20/24] xtensa/cpu: Mark cpu_die() __noreturn Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14 12:47   ` Max Filippov
2023-02-14 12:47     ` Max Filippov
2023-02-14 12:47     ` Max Filippov
2023-02-14 12:47     ` Max Filippov
2023-02-14 12:47     ` Max Filippov
2023-03-10 20:54   ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 21/24] sched/idle: Make sure weak version of arch_cpu_idle_dead() doesn't return Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-03-10 20:54   ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 22/24] objtool: Include weak functions in 'global_noreturns' check Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 23/24] init: Make arch_call_rest_init() and rest_init() __noreturn Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05 ` [PATCH v2 24/24] sched/idle: Mark arch_cpu_idle_dead() __noreturn Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-02-14  7:05   ` Josh Poimboeuf
2023-03-10 20:54   ` [tip: objtool/core] " tip-bot2 for Josh Poimboeuf
2023-02-14  8:38 ` [PATCH v2 25/24] x86/cpu: Expose arch_cpu_idle_dead()'s prototype definition Philippe Mathieu-Daudé
2023-03-10 20:54   ` [tip: objtool/core] " tip-bot2 for Philippe Mathieu-Daudé
2023-02-14  9:25 ` [PATCH v2 00/24] cpu,sched: Mark arch_cpu_idle_dead() __noreturn Philippe Mathieu-Daudé
2023-02-14  9:25   ` Philippe Mathieu-Daudé
2023-02-14  9:25   ` Philippe Mathieu-Daudé
2023-02-14 18:29   ` Josh Poimboeuf
2023-02-14 18:29     ` Josh Poimboeuf
2023-02-14 18:29     ` Josh Poimboeuf
2023-02-14 18:29     ` Josh Poimboeuf
2023-02-14  9:25 ` Philippe Mathieu-Daudé
2023-02-15 22:22 ` Paul E. McKenney
2023-02-15 22:22   ` Paul E. McKenney
2023-02-15 22:22   ` Paul E. McKenney
2023-02-15 22:22   ` Paul E. McKenney
2023-02-15 22:22   ` Paul E. McKenney

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=167848168944.5837.18023341418882320171.tip-bot2@tip-bot2 \
    --to=tip-bot2@linutronix.de \
    --cc=jpoimboe@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-tip-commits@vger.kernel.org \
    --cc=x86@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.