[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 00/43] Migration: Create options.c for capabilities/params/pr
From: |
Juan Quintela |
Subject: |
Re: [PATCH 00/43] Migration: Create options.c for capabilities/params/properties |
Date: |
Thu, 20 Apr 2023 15:08:52 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru> wrote:
> On 02.03.23 19:33, Juan Quintela wrote:
>> Hi
>> This series move to options.c:
>> - all migration capabilities code
>> - all migration parameters code
>> - all properties code
>> - all qmp commands that only touch the previous
>> And once there:
>> - sort of functions
>> - make consistent and coherent all the functions naming/typing
>> - create accessors for the parameters/capabilties that don't exist
>> - more cleanups here and there.
>> Todo:
>> - There is still capabilities code on savevm.c, but I want this in
>> before moving that code to options.c, but still needs more thought
>> for my part. I.e. should I put vmstate sections in options.c, or
>> should I create new functions to access the capabilities in savevm.c.
>> Please review.
>
> Hmm, I can't apply to current master, neither can patchew:
> https://patchew.org/QEMU/20230302163410.11399-1-quintela@redhat.com/
>
> On which commit the series based on?
It was on top of my spice changes O:-)
But I sent a new version right now on top of the PULL request.
Thanks, Juan.
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [PATCH 00/43] Migration: Create options.c for capabilities/params/properties,
Juan Quintela <=