Re: [PATCH v6 0/5] support inflight migration

Vladimir Sementsov-Ogievskiy <[email protected]> Wed, 14 Jan 2026 09:19:10 +0300
Newsgroups dev.linux.lists.virtio-fs,org.nongnu.qemu-devel
Message-ID <[email protected]>
On 13.01.26 21:56, Peter Xu wrote:
> On Tue, Jan 13, 2026 at 01:12:42PM -0500, Stefan Hajnoczi wrote:
>> On Tue, Jan 13, 2026 at 02:58:09PM +0500, Alexandr Moshkov wrote:
>>
>> Peter: Please review the migration aspects (especially the vmstates).
>> Thank you!
> 
> Looks good from my side as long as it's based on VMSD, I appreciate that
> change from the old versions where it used to use qemufile APIs.
> 
> The major question here is if this series depends on Vladimir's other
> series

No, it does not. And if we can proceed with merging these series first, I'll
be happy to rebase on top of it.

> while there's still one patch that is not-for-merge:
> 
> https://lore.kernel.org/all/[email protected]/#t
> 
> Does it automatically mark this series RFC as well?
> 
> Personally speaking, a new migration cap would work all fine, we should
> have discussed it somewhere previously.  Said that, "local-vhost-user-blk"
> capability is likely not the right one.  IMHO it should be either "local"
> or "fd-passing" / "fd-passthrough" (or something generic) as the name.  If
> we are not sure if we will leverage more than "passing the FDs around", we
> can make it as simple as "local" as a new migration capability.
> 
> Then migration's misc.h should export a function migrate_is_local() then
> device code can probe that in its own vmstate handling paths on save/load.
> 
> A note to Vladimir: please remember to add a check to enforce UNIX socket
> when a formal patch 23 will be proposed some day, no matter what is the
> name of the capability.  It should fail qmp "migrate" or qmp
> "migrate_incoming" command if the main URI is not a unix socket.
> 

Thanks! I'll keep that in mind when prepare next version.

-- 
Best regards,
Vladimir