[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 12/14] migration/multifd: Allow multifd sync without flush
|
From: |
Peter Xu |
|
Subject: |
Re: [PATCH v3 12/14] migration/multifd: Allow multifd sync without flush |
|
Date: |
Thu, 22 Aug 2024 12:03:47 -0400 |
On Thu, Aug 01, 2024 at 09:35:14AM -0300, Fabiano Rosas wrote:
> Separate the multifd sync from flushing the client data to the
> channels. These two operations are closely related but not strictly
> necessary to be executed together.
>
> The multifd sync is intrinsic to how multifd works. The multiple
> channels operate independently and may finish IO out of order in
> relation to each other. This applies also between the source and
> destination QEMU.
>
> Flushing the data that is left in the client-owned data structures
> (e.g. MultiFDPages_t) prior to sync is usually the right thing to do,
> but that is particular to how the ram migration is implemented with
> several passes over dirty data.
>
> Make these two routines separate, allowing future code to call the
> sync by itself if needed. This also allows the usage of
> multifd_ram_send to be isolated to ram code.
What's the usage of sync but not flush here?
--
Peter Xu
- Re: [PATCH v3 09/14] migration/multifd: Isolate ram pages packet data, (continued)
[PATCH v3 01/14] migration/multifd: Reduce access to p->pages, Fabiano Rosas, 2024/08/01
[PATCH v3 04/14] migration/multifd: Pass in MultiFDPages_t to file_write_ramblock_iov, Fabiano Rosas, 2024/08/01
[PATCH v3 07/14] migration/multifd: Replace p->pages with an union pointer, Fabiano Rosas, 2024/08/01
[PATCH v3 11/14] migration/multifd: Replace multifd_send_state->pages with client data, Fabiano Rosas, 2024/08/01
[PATCH v3 12/14] migration/multifd: Allow multifd sync without flush, Fabiano Rosas, 2024/08/01
- Re: [PATCH v3 12/14] migration/multifd: Allow multifd sync without flush,
Peter Xu <=
- Re: [PATCH v3 12/14] migration/multifd: Allow multifd sync without flush, Peter Xu, 2024/08/22
- Re: [PATCH v3 12/14] migration/multifd: Allow multifd sync without flush, Fabiano Rosas, 2024/08/22
- Re: [PATCH v3 12/14] migration/multifd: Allow multifd sync without flush, Peter Xu, 2024/08/22
- Re: [PATCH v3 12/14] migration/multifd: Allow multifd sync without flush, Fabiano Rosas, 2024/08/22
- Re: [PATCH v3 12/14] migration/multifd: Allow multifd sync without flush, Peter Xu, 2024/08/22
[PATCH v3 13/14] migration/multifd: Register nocomp ops dynamically, Fabiano Rosas, 2024/08/01
Re: [PATCH v3 00/14] migration/multifd: Remove multifd_send_state->pages, Fabiano Rosas, 2024/08/01