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.