Re: [PATCH v6 0/8] migration: Add switchover ack capability and VFIO precopy support

[Avihai Horon]

On 22/06/2023 14:34, Cédric Le Goater wrote:
> External email: Use caution opening links or attachments
>
>
> On 6/21/23 21:28, Alex Williamson wrote:
> > On Wed, 21 Jun 2023 14:11:53 +0300
> > Avihai Horon <avihaih@nvidia.com> wrote:
> >
> > > Hello everyone,
> > >
> > > The latest changes to migration qtest made the tests run non-live by
> > > default. I am posting this v6 to change back the switchover-ack
> > > migration test to run live as it used to (because the source VM 
> > > needs to
> > > be running to consider the switchover ACK when deciding to do the
> > > switchover or not).
> > >
> > > Changes from v5 [7]:
> > > * Rebased on latest master branch.
> > > * Made switchover-ack migration test run live again (I kept the R-bs as
> > >    this was the original behavior when they were given).
> > > * Dropped patch #8 (x-allow-pre-copy property). (Alex)
> > > * Adjusted patch #9 commit message according to drop of patch #8.
> > > * Added R-b to patch #9 and Tested-by tags to the series.
> >
> > I think Cédric is going to handle the pull request for this, so...
>
> Yes. I have a branch ready with this series and a few other fixes. Unless
> Juan want to handle it.
>
>
> Avihai,
>
> QEMU seems ready to allow migration when support is available in the
> kernel now and so, we can set "x-enable-migration" to true by default.
> Are you OK with that ?
Yes, sure.
But as per previous off-list discussions, it's not merely changing 
x-enable-migration to true by default, right?
If I recall correctly, Alex wanted to remove the x- prefix, change it to 
ON_OFF_AUTO and some other stuff.

Do you have some patches ready for that? Or you want me to do it?

> There are still some blockers, multiple devices, viommu, these will be
> removed with future changes.

Yes, of course.

Thanks.