[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] docs/specs/pvpanic: document shutdown event
From: |
Michael S. Tsirkin |
Subject: |
Re: [PATCH] docs/specs/pvpanic: document shutdown event |
Date: |
Tue, 12 Mar 2024 13:55:28 -0400 |
On Sun, Mar 10, 2024 at 04:04:51PM +0100, Thomas Weißschuh wrote:
> Shutdown requests are normally hardware dependent.
> By extending pvpanic to also handle shutdown requests, guests can
> submit such requests with an easily implementable and cross-platform
> mechanism.
>
> Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>
Okay I merged this to get us out of the bind.
Pls post a fixup patch adding something like (to be implemented)
so people reading this do not become confused.
> ---
> This patch was split out from my earlier pvpanic-shutdown series [0].
> The original series dropped the usage of the linux/pvpanic.h UAPI
> header.
> As we decided to keep the linux header [1] this first commit only adds
> the spec definition.
> Then a patch to Linux will add the new events to linux/pvpanic.h.
> After this the rest of the changes to QEMU will be submitted again.
>
> [0]
> https://lore.kernel.org/qemu-devel/20240208-pvpanic-shutdown-v6-0-965580ac057b@t-8ch.de/
> [1]
> https://lore.kernel.org/lkml/20240213053953-mutt-send-email-mst@kernel.org/
> ---
> docs/specs/pvpanic.rst | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/docs/specs/pvpanic.rst b/docs/specs/pvpanic.rst
> index f894bc19555f..61a80480edb8 100644
> --- a/docs/specs/pvpanic.rst
> +++ b/docs/specs/pvpanic.rst
> @@ -29,6 +29,8 @@ bit 1
> a guest panic has happened and will be handled by the guest;
> the host should record it or report it, but should not affect
> the execution of the guest.
> +bit 2
> + a regular guest shutdown has happened and should be processed by the host
>
> PCI Interface
> -------------
>
> ---
> base-commit: f901bf11b3ddf852e591593b09b8aa7a177f9a0b
> change-id: 20240310-pvpanic-shutdown-spec-4ea2172529e8
>
> Best regards,
> --
> Thomas Weißschuh <thomas@t-8ch.de>