motorhead/qemu
1
0
Fork 0
mirror of https://github.com/Motorhead1991/qemu.git synced 2025-08-24 02:21:52 -06:00
Code Issues Projects Releases Packages Wiki Activity Actions

virtio-balloon: Disassociate from the balloon handlers on unplug

Browse source 
Adding a 2nd balloon device after unplugging the first one doesn't work.
Also, the 'info balloon' command should indicate an error mentioning no
balloon device is registered after unplug.

Reproduction steps given by Shaolong Hu:

(qemu) info balloon
Device 'balloon' has not been activated by the guest
(qemu) device_add virtio-balloon-pci,id=balloon1
(qemu) info balloon
balloon: actual=4096
(qemu) balloon 2048
(qemu) info balloon
balloon: actual=2048
(qemu) device_del balloon1
(qemu) info balloon
balloon: actual=4096
(qemu) balloon 2048
(qemu) info balloon
balloon: actual=4096
(qemu) device_del balloon1
Device 'balloon1' not found
(qemu) device_add virtio-balloon-pci,id=balloon1
Another balloon device already registered
Device 'virtio-balloon-pci' could not be initialized
(qemu) device_add virtio-balloon-pci,id=balloon2
Another balloon device already registered
Device 'virtio-balloon-pci' could not be initialized

Reported-by: Shaolong Hu <shu@redhat.com>
Signed-off-by: Amit Shah <amit.shah@redhat.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
This commit is contained in:
Amit Shah 2011-09-09 14:30:40 +05:30 • committed by Anthony Liguori
parent 8a7d552cb3
commit 3b86b13ab0
1 changed files with 2 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

2
hw/virtio-balloon.c
View file

@ -303,6 +303,8 @@ VirtIODevice *virtio_balloon_init(DeviceState *dev)
void virtio_balloon_exit(VirtIODevice *vdev) void virtio_balloon_exit(VirtIODevice *vdev)
{ {
VirtIOBalloon *s = DO_UPCAST(VirtIOBalloon, vdev, vdev); VirtIOBalloon *s = DO_UPCAST(VirtIOBalloon, vdev, vdev);
qemu_remove_balloon_handler(s);
unregister_savevm(s->qdev, "virtio-balloon", s); unregister_savevm(s->qdev, "virtio-balloon", s);
virtio_cleanup(vdev); virtio_cleanup(vdev);
} }