Re: [PATCH] migration: failover: reset partially_hotplugged

[Juan Quintela <quintela@redhat.com>]

Laurent Vivier <lvivier@redhat.com> wrote:
> When the card is plugged back, reset the partially_hotplugged flag to false
>
> Bug: https://bugzilla.redhat.com/show_bug.cgi?id=1787194
> Signed-off-by: Laurent Vivier <lvivier@redhat.com>
> ---
>  hw/net/virtio-net.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
> index bd7958b9f0ee..16d20cdee52a 100644
> --- a/hw/net/virtio-net.c
> +++ b/hw/net/virtio-net.c
> @@ -3234,6 +3234,7 @@ static bool failover_replug_primary(VirtIONet *n, DeviceState *dev,
>          }
>          hotplug_handler_plug(hotplug_ctrl, dev, &err);
>      }
> +    pdev->partially_hotplugged = false;
>  
>  out:
>      error_propagate(errp, err);

Reviewed-by: Juan Quintela <quintela@redhat.com>


But while we are at this:
- In case hotplug_handler_pre_plug() fails, we don't reset it.
  No, I have no clue what is the right thing to in that error case.
  We have already plugged it once, so it should work (famous last
  words).

- Independent of this patch, we check if hotplug_ctrl is NULL, but
  nothing else on the tree check for that.
  Ok, device_set_realize() check it, but nothing else, except network
  failover.  I can't see how we are going to be able to arrive there and
  not having it, though.

Later, Juan.