[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 03/11] migration: remove "enable_colo" var
From: |
Juan Quintela |
Subject: |
Re: [Qemu-devel] [PATCH 03/11] migration: remove "enable_colo" var |
Date: |
Wed, 03 Jan 2018 09:55:54 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux) |
Peter Xu <address@hidden> wrote:
> It's only used once, clean it up a bit.
>
> Signed-off-by: Peter Xu <address@hidden>
See my previous comment on patch 1. We can remove the variable, but the
move to ose assert is wrong IMHO.
Later, Juan.
> ---
> migration/migration.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/migration/migration.c b/migration/migration.c
> index edbda43246..20f7565527 100644
> --- a/migration/migration.c
> +++ b/migration/migration.c
> @@ -2177,7 +2177,6 @@ static void *migration_thread(void *opaque)
> bool entered_postcopy = false;
> /* The active state we expect to be in; ACTIVE or POSTCOPY_ACTIVE */
> enum MigrationStatus current_active_state = MIGRATION_STATUS_ACTIVE;
> - bool enable_colo = migrate_colo_enabled();
>
> rcu_register_thread();
>
> @@ -2305,7 +2304,7 @@ static void *migration_thread(void *opaque)
> runstate_set(RUN_STATE_POSTMIGRATE);
> } else {
> if (s->state == MIGRATION_STATUS_ACTIVE) {
> - assert(enable_colo);
> + assert(migrate_colo_enabled());
> migrate_start_colo_process(s);
> /*
> * Fixme: we will run VM in COLO no matter its old running state.
- Re: [Qemu-devel] [PATCH 01/11] migration: assert colo instead of check, (continued)
- [Qemu-devel] [PATCH 02/11] migration: qemu_savevm_state_cleanup() in cleanup, Peter Xu, 2018/01/03
- Re: [Qemu-devel] [PATCH 02/11] migration: qemu_savevm_state_cleanup() in cleanup, Juan Quintela, 2018/01/03
- Re: [Qemu-devel] [PATCH 02/11] migration: qemu_savevm_state_cleanup() in cleanup, Peter Xu, 2018/01/03
- Re: [Qemu-devel] [PATCH 02/11] migration: qemu_savevm_state_cleanup() in cleanup, Juan Quintela, 2018/01/03
- Re: [Qemu-devel] [PATCH 02/11] migration: qemu_savevm_state_cleanup() in cleanup, Peter Xu, 2018/01/03
- Re: [Qemu-devel] [PATCH 02/11] migration: qemu_savevm_state_cleanup() in cleanup, Juan Quintela, 2018/01/03
- Re: [Qemu-devel] [PATCH 02/11] migration: qemu_savevm_state_cleanup() in cleanup, Peter Xu, 2018/01/03
[Qemu-devel] [PATCH 03/11] migration: remove "enable_colo" var, Peter Xu, 2018/01/03
- Re: [Qemu-devel] [PATCH 03/11] migration: remove "enable_colo" var,
Juan Quintela <=
[Qemu-devel] [PATCH 05/11] migration: move vm_old_running into global state, Peter Xu, 2018/01/03
[Qemu-devel] [PATCH 04/11] migration: split use of MigrationState.total_time, Peter Xu, 2018/01/03