[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/1] migration: qemu_savevm_state_cleanup become
From: |
Amit Shah |
Subject: |
Re: [Qemu-devel] [PATCH 1/1] migration: qemu_savevm_state_cleanup becomes mandatory operation |
Date: |
Mon, 9 Nov 2015 09:36:55 +0530 |
CC'ing Liang Li, author of the patch.
On (Sat) 07 Nov 2015 [18:40:12], Denis V. Lunev wrote:
> since commit
> commit 94f5a43704129ca4995aa3385303c5ae225bde42
> Author: Liang Li <address@hidden>
> Date: Mon Nov 2 15:37:00 2015 +0800
>
> migration: defer migration_end & blk_mig_cleanup
>
> when actual .cleanup callbacks calling was removed from complete operations.
>
> Signed-off-by: Denis V. Lunev <address@hidden>
> CC: Paolo Bonzini <address@hidden>
> CC: Juan Quintela <address@hidden>
> CC: Amit Shah <address@hidden>
> ---
> migration/savevm.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/migration/savevm.c b/migration/savevm.c
> index e05158d..9f2230f 100644
> --- a/migration/savevm.c
> +++ b/migration/savevm.c
> @@ -942,8 +942,8 @@ static int qemu_savevm_state(QEMUFile *f, Error **errp)
> qemu_savevm_state_complete(f);
> ret = qemu_file_get_error(f);
> }
> + qemu_savevm_state_cleanup();
> if (ret != 0) {
> - qemu_savevm_state_cleanup();
> error_setg_errno(errp, -ret, "Error while writing VM state");
> }
> return ret;
> --
> 2.5.0
>
Amit
- [Qemu-devel] assert during internal snapshot, Denis V. Lunev, 2015/11/07
- [Qemu-devel] [PATCH 1/1] migration: qemu_savevm_state_cleanup becomes mandatory operation, Denis V. Lunev, 2015/11/07
- Re: [Qemu-devel] [PATCH 1/1] migration: qemu_savevm_state_cleanup becomes mandatory operation,
Amit Shah <=
- Re: [Qemu-devel] [PATCH 1/1] migration: qemu_savevm_state_cleanup becomes mandatory operation, Li, Liang Z, 2015/11/09
- Re: [Qemu-devel] [PATCH 1/1] migration: qemu_savevm_state_cleanup becomes mandatory operation, Denis V. Lunev, 2015/11/09
- Re: [Qemu-devel] [PATCH 1/1] migration: qemu_savevm_state_cleanup becomes mandatory operation, Li, Liang Z, 2015/11/09
- Re: [Qemu-devel] [PATCH 1/1] migration: qemu_savevm_state_cleanup becomes mandatory operation, Denis V. Lunev, 2015/11/09
- Re: [Qemu-devel] [PATCH 1/1] migration: qemu_savevm_state_cleanup becomes mandatory operation, Li, Liang Z, 2015/11/09
Re: [Qemu-devel] assert during internal snapshot, Li, Liang Z, 2015/11/08
Re: [Qemu-devel] assert during internal snapshot, Li, Liang Z, 2015/11/08
Re: [Qemu-devel] assert during internal snapshot, Li, Liang Z, 2015/11/08