Re: [Qemu-devel] [PATCH 0/3] Fail migration on bdrv_flush_all() error

[Stefan Hajnoczi]

On Fri, Jul 05, 2013 at 02:03:50PM +0200, Kevin Wolf wrote:
> If bdrv_flush_all() returns an error, there is an inconsistency in the
> view of an image file between the source and the destination host.
> Completing the migration would lead to corruption. Better abort
> migration in this case.
> 
> Kevin Wolf (3):
>   block: Add return value for bdrv_flush_all()
>   cpus: Add return value for vm_stop()
>   migration: Fail migration on bdrv_flush_all() error
> 
>  block.c                 | 10 ++++++++--
>  cpus.c                  | 20 +++++++++++++-------
>  include/block/block.h   |  2 +-
>  include/sysemu/sysemu.h |  4 ++--
>  migration.c             | 17 ++++++++++++++---
>  stubs/vm-stop.c         |  2 +-
>  6 files changed, 39 insertions(+), 16 deletions(-)

Modulo Paolo's comments.

Reviewed-by: Stefan Hajnoczi <address@hidden>