Re: [PATCH] migration/ram: Refactor precopy ram loading code

[Philippe Mathieu-Daudé]

On 25/7/23 15:26, Fabiano Rosas wrote:
> From: Nikolay Borisov <nborisov@suse.com>
>
> Extract the ramblock parsing code into a routine that operates on the
> sequence of headers from the stream and another the parses the
> individual ramblock. This makes ram_load_precopy() easier to
> comprehend.
>
> Signed-off-by: Nikolay Borisov <nborisov@suse.com>
> Signed-off-by: Fabiano Rosas <farosas@suse.de>
> ---
> I'm extracting the parts from the fixed-ram migration series [1] that
> could already go in. This patch is one of them.
>
> 1- https://lore.kernel.org/r/20230330180336.2791-1-farosas@suse.de
> ---
>   migration/ram.c | 141 +++++++++++++++++++++++++++---------------------
>   1 file changed, 79 insertions(+), 62 deletions(-)

I'd rather 1 patch extracting parse_ramblock() then another
one for parse_ramblocks(), anyhow:

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>