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

[Kaiyang Wu <wukaiyang2003@gmail.com>]

Superseded by 
https://lore.proxmox.com/pve-devel/20261009065813.109006-1-wukaiyang@loongfans.cn/

On 2026-09-23 16:55, Kaiyang Wu wrote:
> Hi Dominik,
> 
> Thanks for the clarification, I'll implement it in v3.
> 
> Best,
> Kaiyang
> 
> On Tue, Sep 22, 2026, 19:38 Dominik Csapak <d.csapak@proxmox.com 
> <mailto:d.csapak@proxmox.com>> wrote:
> 
>     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
>     <https://www.qemu.org/docs/master/specs/pvpanic.html>
>      >
>      > v1: https://lore.proxmox.com/pve-devel/20260827103529.393388-1-
>     wukaiyang@loongfans.cn/ <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(-)
>      >
> 
>