[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH v2 6/8] failover: pci: unregister ROM on unplug
From: |
Juan Quintela |
Subject: |
Re: [RFC PATCH v2 6/8] failover: pci: unregister ROM on unplug |
Date: |
Wed, 25 Aug 2021 17:12:38 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Laurent Vivier <lvivier@redhat.com> wrote:
> The intend of failover is to allow a VM with a VFIO networking card to
> be migrated without disrupting the network operation by switching
> to a virtio-net device during the migration.
>
> This simple change allows a simulated device like e1000e to be tested
> rather than a vfio device, even if it's useless in real life it can help
> to debug failover.
>
> This is interesting to developers that want to test failover on
> a system with no vfio device. Moreover it simplifies host networking
> configuration as we can use the same bridge for virtio-net and
> the other failover networking device.
>
> Without this change the migration of a system configured with failover
> fails with:
>
> ...
> -device virtio-net-pci,id=virtionet0,failover=on,... \
> -device e1000,failover_pair_id=virtionet0,... \
> ...
>
> (qemu) migrate ...
>
> Unknown ramblock "0000:00:01.1:00.0/e1000e.rom", cannot accept migration
> error while loading state for instance 0x0 of device 'ram'
> load of migration failed: Invalid argument
>
> This happens because QEMU correctly unregisters the interface vmstate but
> not the ROM one. This patch fixes that.
>
> Signed-off-by: Laurent Vivier <lvivier@redhat.com>
Reviewed-by: Juan Quintela <quintela@redhat.com>
One could even defend that unpluging the device and *NOT* unpluging the
ROM is a bug, independently of failover, no?
Later, Juan.
- [RFC PATCH v2 0/8] virtio-net failover cleanup and new features, Laurent Vivier, 2021/08/20
- [RFC PATCH v2 1/8] qdev: add an Error parameter to the DeviceListener hide_device() function, Laurent Vivier, 2021/08/20
- [RFC PATCH v2 2/8] qdev/qbus: remove failover specific code, Laurent Vivier, 2021/08/20
- [RFC PATCH v2 3/8] failover: virtio-net: remove failover_primary_hidden flag, Laurent Vivier, 2021/08/20
- [RFC PATCH v2 5/8] failover: hide the PCI device if the virtio-net device is not present, Laurent Vivier, 2021/08/20
- [RFC PATCH v2 4/8] failover: pci: move failover hotplug/unplug code into pci subsystem, Laurent Vivier, 2021/08/20
- [RFC PATCH v2 6/8] failover: pci: unregister ROM on unplug, Laurent Vivier, 2021/08/20
- Re: [RFC PATCH v2 6/8] failover: pci: unregister ROM on unplug,
Juan Quintela <=
- [RFC PATCH v2 7/8] pci: automatically unplug a PCI card before migration, Laurent Vivier, 2021/08/20
- [RFC PATCH v2 8/8] failover: qemu-opts: manage hidden device list, Laurent Vivier, 2021/08/20