diff options
author | Jens Freimann <jfreimann@redhat.com> | 2019-11-20 16:49:48 +0100 |
---|---|---|
committer | Jason Wang <jasowang@redhat.com> | 2019-11-25 23:30:28 +0800 |
commit | 284f42a520cd9f5905abac2fa50397423890de8f (patch) | |
tree | 33e471c9d4372ce0b59533e99a97f1d1d61b42f1 | |
parent | 122e6d2a9c1bf8aa1d51409c15809a82621515b1 (diff) | |
download | qemu-284f42a520cd9f5905abac2fa50397423890de8f.zip qemu-284f42a520cd9f5905abac2fa50397423890de8f.tar.gz qemu-284f42a520cd9f5905abac2fa50397423890de8f.tar.bz2 |
net/virtio: fix dev_unplug_pending
.dev_unplug_pending is set up by virtio-net code indepent of failover
support was set for the device or not. This gives a wrong result when
we check for existing primary devices in migration code.
Fix this by actually calling dev_unplug_pending() instead of just
checking if the function pointer was set. When the feature was not
negotiated dev_unplug_pending() will always return false. This prevents
us from going into the wait-unplug state when there's no primary device
present.
Fixes: 9711cd0dfc3f ("net/virtio: add failover support")
Signed-off-by: Jens Freimann <jfreimann@redhat.com>
Reported-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
-rw-r--r-- | hw/net/virtio-net.c | 3 | ||||
-rw-r--r-- | migration/savevm.c | 3 |
2 files changed, 5 insertions, 1 deletions
diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c index 97a5113..946039c 100644 --- a/hw/net/virtio-net.c +++ b/hw/net/virtio-net.c @@ -3124,6 +3124,9 @@ static bool primary_unplug_pending(void *opaque) VirtIODevice *vdev = VIRTIO_DEVICE(dev); VirtIONet *n = VIRTIO_NET(vdev); + if (!virtio_vdev_has_feature(vdev, VIRTIO_NET_F_STANDBY)) { + return false; + } return n->primary_dev ? n->primary_dev->pending_deleted_event : false; } diff --git a/migration/savevm.c b/migration/savevm.c index 966a9c3..a71b930 100644 --- a/migration/savevm.c +++ b/migration/savevm.c @@ -1119,7 +1119,8 @@ int qemu_savevm_nr_failover_devices(void) int n = 0; QTAILQ_FOREACH(se, &savevm_state.handlers, entry) { - if (se->vmsd && se->vmsd->dev_unplug_pending) { + if (se->vmsd && se->vmsd->dev_unplug_pending && + se->vmsd->dev_unplug_pending(se->opaque)) { n++; } } |