Re: [PATCH v3] failover: fix unplug pending detection

[Laurent Vivier]

On 18/10/2021 10:27, Michael S. Tsirkin wrote:
> On Mon, Oct 18, 2021 at 09:19:16AM +0200, Laurent Vivier wrote:
> > Hi,
> >
> > I don't understand if there are some issues
>
> Gerd did identify some issues, you felt they aren't related to the patch
> and need to be addressed separately.
>
> Gerd posted patches that are supposed to address them since.
> "try improve native hotplug for pcie root ports"
> Could you please either
> - test and report that your series depend on
>    Gerd's one to now work without the issues.
>    preferably by reposting a patch that applies on top.
> - test and report that the functionality is still partially
>    broken but explain in the commit log that this is not due
>    to the patch itself, and not made worse.
>
> in both cases please CC reviewers: Daniel, Gerd.

I'm writing a test in tests/qtest that tests virtio-net failover, and I've added a test 
that checks the migration doesn't start while the card is not unplugged.

I've run the test on top of current qemu master (where Gerd's series is merged) and the 
problem still exists.

I will re-send this fix and the test in the same series.

Thanks,
Laurent