]> git.proxmox.com Git - qemu.git/commit
pc: get rid of builtin pvpanic for "-M pc-1.5"
authorPaolo Bonzini <pbonzini@redhat.com>
Mon, 4 Nov 2013 13:30:48 +0000 (14:30 +0100)
committerPaolo Bonzini <pbonzini@redhat.com>
Thu, 21 Nov 2013 16:38:54 +0000 (17:38 +0100)
commit7839ff593be03a7de3c6760e0b50c565ea751d36
tree8608f89f4a794ef845c063aa369944a8ce9aa494
parent45d285abd7028ac72418c1a22f9298bb898fbfb8
pc: get rid of builtin pvpanic for "-M pc-1.5"

This causes two slight backwards-incompatibilities between "-M pc-1.5"
and 1.5's "-M pc":

(1) a fw_cfg file is removed with this patch.  This is only a problem
if migration stops the virtual machine exactly during fw_cfg enumeration.

(2) after migration, a VM created without an explicit "-device pvpanic"
will stop reporting panics to management.

The first problem only occurs if migration is done at a very, very
early point (and I'm not sure it can happen in practice for reasonable-size
VMs, since it will likely take more time to send the RAM to destination,
than it will take for BIOS to scan fw_cfg).

The second problem only occurs if the guest panics _and_ has a guest
driver _and_ management knows to look at the crash event, so it is
mostly theoretical at this point in time.

Thus keep the code simple, and pretend it was never broken.

Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
hw/i386/pc_piix.c
hw/i386/pc_q35.c
hw/misc/pvpanic.c
include/hw/i386/pc.h