Skip to content

Commit

Permalink
failover: simplify failover_unplug_primary
Browse files Browse the repository at this point in the history
We can calculate device just once.

Signed-off-by: Juan Quintela <quintela@redhat.com>
Message-Id: <20201118083748.1328-27-quintela@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
  • Loading branch information
Juan Quintela authored and mstsirkin committed Dec 8, 2020
1 parent 0e9a65c commit 07a5d81
Showing 1 changed file with 10 additions and 15 deletions.
25 changes: 10 additions & 15 deletions hw/net/virtio-net.c
Expand Up @@ -3146,34 +3146,29 @@ static bool failover_replug_primary(VirtIONet *n, DeviceState *dev,
return !err;
}

static void virtio_net_handle_migration_primary(VirtIONet *n,
MigrationState *s)
static void virtio_net_handle_migration_primary(VirtIONet *n, MigrationState *s)
{
bool should_be_hidden;
Error *err = NULL;
DeviceState *dev = failover_find_primary_device(n);

should_be_hidden = qatomic_read(&n->failover_primary_hidden);

if (!n->primary_dev) {
n->primary_dev = failover_find_primary_device(n);
if (!n->primary_dev) {
return;
}
if (!dev) {
return;
}

should_be_hidden = qatomic_read(&n->failover_primary_hidden);

if (migration_in_setup(s) && !should_be_hidden) {
if (failover_unplug_primary(n, n->primary_dev)) {
vmstate_unregister(VMSTATE_IF(n->primary_dev),
qdev_get_vmsd(n->primary_dev),
n->primary_dev);
qapi_event_send_unplug_primary(n->primary_dev->id);
if (failover_unplug_primary(n, dev)) {
vmstate_unregister(VMSTATE_IF(dev), qdev_get_vmsd(dev), dev);
qapi_event_send_unplug_primary(dev->id);
qatomic_set(&n->failover_primary_hidden, true);
} else {
warn_report("couldn't unplug primary device");
}
} else if (migration_has_failed(s)) {
/* We already unplugged the device let's plug it back */
if (!failover_replug_primary(n, n->primary_dev, &err)) {
if (!failover_replug_primary(n, dev, &err)) {
if (err) {
error_report_err(err);
}
Expand Down

0 comments on commit 07a5d81

Please sign in to comment.