[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 08/22] migration: Check that migration is active
From: |
Juan Quintela |
Subject: |
[Qemu-devel] Re: [PATCH 08/22] migration: Check that migration is active before cancel it |
Date: |
Wed, 23 Feb 2011 10:14:11 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.2 (gnu/linux) |
Yoshiaki Tamura <address@hidden> wrote:
> 2011/2/23 Juan Quintela <address@hidden>:
>> Signed-off-by: Juan Quintela <address@hidden>
>> ---
>> migration.c | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/migration.c b/migration.c
>> index 397a0b9..55f58c8 100644
>> --- a/migration.c
>> +++ b/migration.c
>> @@ -138,7 +138,7 @@ int do_migrate_cancel(Monitor *mon, const QDict *qdict,
>> QObject **ret_data)
>> {
>> MigrationState *s = current_migration;
>>
>> - if (s)
>> + if (s && s->get_status(s) == MIG_STATE_ACTIVE)
>> s->cancel(s);
>>
>> return 0;
>
> Why don't you remove *s again?
Removed in a next patch.
Later, Juan.
- [Qemu-devel] [PATCH 01/22] migration: Make *start_outgoing_migration return FdMigrationState, (continued)
- [Qemu-devel] [PATCH 01/22] migration: Make *start_outgoing_migration return FdMigrationState, Juan Quintela, 2011/02/22
- [Qemu-devel] [PATCH 03/22] migration: Fold MigrationState into FdMigrationState, Juan Quintela, 2011/02/22
- [Qemu-devel] [PATCH 04/22] migration: Rename FdMigrationState MigrationState, Juan Quintela, 2011/02/22
- [Qemu-devel] [PATCH 05/22] migration: Refactor MigrationState creation, Juan Quintela, 2011/02/22
- [Qemu-devel] [PATCH 06/22] migration: Make all posible migration functions static, Juan Quintela, 2011/02/22
- [Qemu-devel] [PATCH 08/22] migration: Check that migration is active before cancel it, Juan Quintela, 2011/02/22
- [Qemu-devel] [PATCH 09/22] migration: Introduce MIG_STATE_NONE, Juan Quintela, 2011/02/22
- [Qemu-devel] [PATCH 07/22] migration: move migrate_create_state to do_migrate, Juan Quintela, 2011/02/22
- [Qemu-devel] [PATCH 10/22] migration: Refactor and simplify error checking in migrate_fd_put_ready, Juan Quintela, 2011/02/22
- [Qemu-devel] [PATCH 11/22] migration: Introduce migrate_fd_completed() for consistenncy, Juan Quintela, 2011/02/22
- [Qemu-devel] [PATCH 12/22] migration: Use migrate_fd_error() in last place that set status to ERROR, Juan Quintela, 2011/02/22