[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/7] target/openrisc: add shutdown logic
From: |
Stafford Horne |
Subject: |
Re: [Qemu-devel] [PATCH 2/7] target/openrisc: add shutdown logic |
Date: |
Thu, 28 Apr 2022 20:47:58 +0900 |
On Thu, Apr 28, 2022 at 01:16:51PM +0200, Jason A. Donenfeld wrote:
> Hey again,
>
> On Thu, Apr 28, 2022 at 02:04:29AM +0200, Jason A. Donenfeld wrote:
> > By the way, would this also help the reboot case? That's
> > `reboot(RB_AUTOBOOT);`, which does:
> >
> > machine_restart() ->
> > do_kernel_restart() ->
> > atomic_notifier_chain_register(&restart_handler_list, nb) ->
> > ???
> >
> > As far as I can tell, nothing is wired into the reboot case for
> > OpenRISC. Is this something that could be fixed in the kernel without
> > having to patch QEMU? If so, then I could effectively get shutdown for
> > my CI with the -no-reboot option, which is what I'm already doing for a
> > few platforms.
>
> I added 13 for this:
> https://lists.librecores.org/pipermail/openrisc/2022-April/003884.html
>
> When you go add these nops to the specification, please remember to add
> one for reboot too. Then, once that kernel code is merged and the
> specification published, it'll be sensible to add shutdown and reboot
> support to QEMU, per Peter's description.
This sounds fair.
-Stafford