[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 3/6] migration: Remove 'blk/-b' option from migrate comman
From: |
Fabiano Rosas |
Subject: |
Re: [PATCH v2 3/6] migration: Remove 'blk/-b' option from migrate commands |
Date: |
Mon, 29 Apr 2024 14:18:57 -0300 |
Peter Xu <peterx@redhat.com> writes:
> On Fri, Apr 26, 2024 at 10:14:05AM -0300, Fabiano Rosas wrote:
>> @@ -2003,21 +1997,7 @@ static bool migrate_prepare(MigrationState *s, bool
>> blk, bool resume,
>> }
>> }
>>
>> - if (blk) {
>> - if (migrate_colo()) {
>> - error_setg(errp, "No disk migration is required in COLO mode");
>> - return false;
>> - }
>> - if (migrate_block()) {
>> - error_setg(errp, "Command options are incompatible with "
>> - "current migration capabilities");
>> - return false;
>> - }
>> - if (!migrate_cap_set(MIGRATION_CAPABILITY_BLOCK, true, errp)) {
>> - return false;
>> - }
>> - s->must_remove_block_options = true;
>> - }
>> + s->must_remove_block_options = true;
>
> Can we drop this var too? Perhaps with block_cleanup_parameters()?
Yes, Markus mentioned it in v1 already. Take a look there. There's
several other declarations I missed. v3 is coming soon.
[PATCH v2 4/6] migration: Remove block migration, Fabiano Rosas, 2024/04/26
[PATCH v2 5/6] migration: Remove non-multifd compression, Fabiano Rosas, 2024/04/26
[PATCH v2 6/6] migration: Deprecate fd: for file migration, Fabiano Rosas, 2024/04/26