[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 0/1] Do not stop guest when panic event is received
From: |
Paolo Bonzini |
Subject: |
Re: [PATCH 0/1] Do not stop guest when panic event is received |
Date: |
Tue, 20 Oct 2020 19:14:21 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.3.1 |
On 02/10/20 04:41, Alejandro Jimenez wrote:
> The fact that the behavior of hv-crash is also affected is why I chose to
> implement this change as an independent
> option, as opposed to making it a property of the pvpanic device (e.g.
> -device pvpanic,no-panicstop).
>
> Please let me know if you have any comments or suggestions.
Hi Alejandro, sorry for the delayed response.
The concept is fine, and I agree this should not be a device property.
On the other hand, we already have many similar options: -no-reboot,
-no-shutdown, -watchdog-action and now --no-panicstop.
I think it's time to group them into a single option:
* -action reboot=pause|shutdown|none
* -action shutdown=pause|poweroff|none
* -action panic=pause|shutdown|none
* -action watchdog=reset|shutdown|poweroff|pause|debug|none|inject-nmi
where the existing options would translate to the new option, like:
* -no-reboot "-action reboot=shutdown"
* -no-shutdown "-action shutdown=pause"
The implementation should be relatively easy too; there's already an
enum WatchdogAction (that can be renamed to e.g. RunstateAction) and a
parsing function select_watchdog_action that can be changed to just
return the RunstateAction.
Would you like to take a look at this?
Thanks,
Paolo