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

[Kaiyang Wu <wukaiyang2003@gmail.com>]

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> 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
> >
> > 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(-)
> >
>
>
>