[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 03/43] migration: Create migration_cap_set()
From: |
Juan Quintela |
Subject: |
Re: [PATCH 03/43] migration: Create migration_cap_set() |
Date: |
Thu, 20 Apr 2023 14:32:59 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
"Dr. David Alan Gilbert" <dgilbert@redhat.com> wrote:
> * Juan Quintela (quintela@redhat.com) wrote:
>> And remove the convoluted use of qmp_migrate_set_capabilities() to
>> enable disable MIGRATION_CAPABILITY_BLOCK.
>>
>> Signed-off-by: Juan Quintela <quintela@redhat.com>
>> ---
>> migration/migration.c | 34 ++++++++++++++++------------------
>> 1 file changed, 16 insertions(+), 18 deletions(-)
>>
>> diff --git a/migration/migration.c b/migration/migration.c
>> index 119027a656..e3062530f0 100644
>> --- a/migration/migration.c
>> +++ b/migration/migration.c
>> @@ -1910,25 +1910,24 @@ void migrate_set_state(int *state, int old_state,
>> int new_state)
>> }
>> }
>>
>> -static MigrationCapabilityStatus *migrate_cap_add(MigrationCapability index,
>> - bool state)
>> +static bool migrate_cap_set(int cap, bool value, Error **errp)
>
> Why int cap rather than MigrationCapability ?
It is the index in one array, so it is int. And it is much, much
shorter.
In this particular case I think that we don't lost anything having it as
int, but will not fight over this O:-)
Later, Juan.
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [PATCH 03/43] migration: Create migration_cap_set(),
Juan Quintela <=