Re: [PATCH qemu-server/docs/manager/ha-manager 0/5] add pvpanic device support

[Dominik Csapak <d.csapak@proxmox.com>]

ok most code looks good to me, but I seemingly didn't properly
explained last time what i meant with ha integration:

instead of letting Ha parse the guest state and act accordingly
my suggestion was more to let the user configure
the '-action panic=XX' commandline too

there one can e.g. set 'exit-failure' which exits the qemu
process completely and ha would automatically notice it's down
during the next round and start it again, no need
to touch the ha code at all

On 9/14/26 11:20 AM, Kaiyang Wu wrote:
> This patchset adds pvpanic device support to QEMU virtual machines to
> provide extra virtual machine status (guest-panicked) when virtual
> machines panic [0]. The HA stack now restarts a "started" VM when it
> panics.
> 
> [0]: https://www.qemu.org/docs/master/specs/pvpanic.html
> 
> v1: https://lore.proxmox.com/pve-devel/20260827103529.393388-1-wukaiyang@loongfans.cn/
> 
> Changes v1 -> v2:
> - Remove the pvpanic ISA device support to simplify the code
> - Move the pvpanic setting from Hardware to Options
> - Restart a HA-managed VM when it panics to restore the "started" state
> 
> 
> qemu-server:
> 
> Kaiyang Wu (1):
>    qemuserver: add pvpanic device
> 
>   src/PVE/API2/Qemu.pm      |  1 +
>   src/PVE/QemuServer.pm     | 11 +++++++++++
>   src/PVE/QemuServer/PCI.pm |  1 +
>   3 files changed, 13 insertions(+)
> 
> 
> docs:
> 
> Kaiyang Wu (1):
>    qm: add document section for the pvpanic device
> 
>   qm.adoc | 22 ++++++++++++++++++++++
>   1 file changed, 22 insertions(+)
> 
> 
> manager:
> 
> Kaiyang Wu (1):
>    ui: qemu: add pvpanic device support
> 
>   www/manager6/qemu/Options.js | 20 ++++++++++++++++++++
>   1 file changed, 20 insertions(+)
> 
> 
> ha-manager:
> 
> Kaiyang Wu (2):
>    vm resource: return running status -1 for panicked VMs
>    lrm: stop and restart panicked VMs to keep the "started" state
> 
>   src/PVE/HA/LRM.pm             | 9 ++++++++-
>   src/PVE/HA/Resources/PVEVM.pm | 5 ++++-
>   2 files changed, 12 insertions(+), 2 deletions(-)
>