[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/5] migration: Remove use of old MigrationParam
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 3/5] migration: Remove use of old MigrationParams |
Date: |
Wed, 24 May 2017 14:44:33 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) |
I got confused and replied to an old version. Please ignore.
Markus Armbruster <address@hidden> writes:
> Juan Quintela <address@hidden> writes:
>
>> We have change in the previous patch to use migration capabilities for
>> it. Notice that we continue using the old command line flags from
>> migrate command from the time being. Remove the set_params method as
>
> for the time being
>
>> now it is empty.
>>
>> For savevm, one can't do a:
>>
>> savevm -b/-i foo
>>
>> but now one can do:
>>
>> migrate_set_capability block on
>> savevm foo
>>
>> And we can't use block migration. We could disable block capability
>> unconditionally, but it would not be much better.
>
> I think I get what you're trying to say, but only because I have plenty
> of context right now. Let me try to rephrase:
>
> migration: Use new configuration instead of old MigrationParams
>
> The previous commit introduced a MigrationCapability and a
> MigrationParameter for block migration. Use them instead of the old
> MigrationParams.
>
> Take care to reject attempts to combine block migration with
> snapshots, e.g. like this:
>
> migrate_set_capability block on
> savevm foo
>
>> Signed-off-by: Juan Quintela <address@hidden>
>
> Preferably with a commit message I can still understand three weeks from
> now:
> Reviewed-by: Markus Armbruster <address@hidden>
- [Qemu-devel] [PATCH v5 0/5] Remove old MigrationParams, Juan Quintela, 2017/05/17
- [Qemu-devel] [PATCH 1/5] hmp: Use visitor api for hmp_migrate_set_parameter(), Juan Quintela, 2017/05/17
- [Qemu-devel] [PATCH 2/5] migration: Create block capability, Juan Quintela, 2017/05/17
- [Qemu-devel] [PATCH 3/5] migration: Remove use of old MigrationParams, Juan Quintela, 2017/05/17
- [Qemu-devel] [PATCH 4/5] migration: Remove old MigrationParams, Juan Quintela, 2017/05/17
- [Qemu-devel] [PATCH 5/5] block migration: Allow compile time disable, Juan Quintela, 2017/05/17