[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 14/14] migration/multifd: Move ram code into multifd-ram.c
From: |
Peter Xu |
Subject: |
Re: [PATCH v3 14/14] migration/multifd: Move ram code into multifd-ram.c |
Date: |
Thu, 22 Aug 2024 12:25:25 -0400 |
On Thu, Aug 01, 2024 at 09:35:16AM -0300, Fabiano Rosas wrote:
> In preparation for adding new payload types to multifd, move most of
> the ram-related code into multifd-ram.c. Let's try to keep a semblance
> of layering by not mixing general multifd control flow with the
> details of transmitting pages of ram.
>
> There are still some pieces leftover, namely the p->normal, p->zero,
> etc variables that we use for zero page tracking and the packet
> allocation which is heavily dependent on the ram code.
>
> Signed-off-by: Fabiano Rosas <farosas@suse.de>
The movement makes sense to me in general, but let's discuss whether nocomp
may need a better name. It could mean that we may want two new files:
multifd-ram.c to keep generic RAM stuff (which apply to nocomp/zlib/...)
then multifd-plain.c which contains no-comp case, perhaps.
--
Peter Xu
- [PATCH v3 06/14] migration/multifd: Make MultiFDPages_t:offset a flexible array member, (continued)
- [PATCH v3 06/14] migration/multifd: Make MultiFDPages_t:offset a flexible array member, Fabiano Rosas, 2024/08/01
- [PATCH v3 08/14] migration/multifd: Move pages accounting into multifd_send_zero_page_detect(), Fabiano Rosas, 2024/08/01
- [PATCH v3 03/14] migration/multifd: Remove pages->allocated, Fabiano Rosas, 2024/08/01
- [PATCH v3 02/14] migration/multifd: Inline page_size and page_count, Fabiano Rosas, 2024/08/01
- [PATCH v3 10/14] migration/multifd: Don't send ram data during SYNC, Fabiano Rosas, 2024/08/01
- [PATCH v3 14/14] migration/multifd: Move ram code into multifd-ram.c, Fabiano Rosas, 2024/08/01
- Re: [PATCH v3 14/14] migration/multifd: Move ram code into multifd-ram.c,
Peter Xu <=
- [PATCH v3 09/14] migration/multifd: Isolate ram pages packet data, Fabiano Rosas, 2024/08/01
[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