Linux-ARM-Kernel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Bradley Morgan <bradley.morgan@linux.dev>
To: leitao@debian.org
Cc: akpm@linux-foundation.org,
	angelogioacchino.delregno@collabora.com, ebiederm@xmission.com,
	kernel-team@meta.com, linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org,
	matthias.bgg@gmail.com, npiggin@gmail.com, oleg@redhat.com,
	peterz@infradead.org, stable@vger.kernel.org,
	syzbot+8fdf0d8e10bdde1c2e88@syzkaller.appspotmail.com
Subject: Re: [PATCH] reboot: halt when power off fails
Date: Fri, 13 Nov 2026 15:51:01 +0000	[thread overview]
Message-ID: <2D88DE92-A563-41B0-AB0B-5F9EA5CE3EF4@linux.dev> (raw)
In-Reply-To: <avWGu4l_nXP_eFyh@gmail.com>

On 12 November 2026 09:33:22 GMT, Breno Leitao <leitao@debian.org> wrote:
>On Wed, Nov 11, 2026 at 09:02:38PM +0100, Oleg Nesterov wrote:
>> On 11/10, Breno Leitao wrote:
>> >
>> > --- a/kernel/reboot.c
>> > +++ b/kernel/reboot.c
>> > @@ -791,6 +791,9 @@ SYSCALL_DEFINE4(reboot, int, magic1, int, magic2,
>unsigned int, cmd,
>> >
>> >  	case LINUX_REBOOT_CMD_POWER_OFF:
>> >  		kernel_power_off();
>> > +		pr_emerg("Power off failed: System halted instead\n");
>> > +		/* kernel_power_off() already ran the shutdown, only halt. */
>> > +		machine_halt();
>> 
>> Agreed. Even if I don't think my ack has any value in this area:
>
>Thanks, Oleg.
>
>I tried to find out who merges changes to this file and it is not clear.
>akpm seems to be the closest, as most of the changes went through him,
>and somehow he was not CCed here. :-|
>
>Andrew, would you be willing to take this one?

No maintainer, Andrew merges.

I tried asking to be one, no avail, whatever.

>
>thanks,
>--breno
>
>

--- Thanks!


      reply	other threads:[~2026-11-13 15:51 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-11-10 15:24 [PATCH] reboot: halt when power off fails Breno Leitao
2026-11-10 15:50 ` Bradley Morgan
2026-11-11 20:02 ` Oleg Nesterov
2026-11-12  9:33   ` Breno Leitao
2026-11-13 15:51     ` Bradley Morgan [this message]

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=2D88DE92-A563-41B0-AB0B-5F9EA5CE3EF4@linux.dev \
    --to=bradley.morgan@linux.dev \
    --cc=akpm@linux-foundation.org \
    --cc=angelogioacchino.delregno@collabora.com \
    --cc=ebiederm@xmission.com \
    --cc=kernel-team@meta.com \
    --cc=leitao@debian.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=matthias.bgg@gmail.com \
    --cc=npiggin@gmail.com \
    --cc=oleg@redhat.com \
    --cc=peterz@infradead.org \
    --cc=stable@vger.kernel.org \
    --cc=syzbot+8fdf0d8e10bdde1c2e88@syzkaller.appspotmail.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox