[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 28/45] mirror: implement completion
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH v2 28/45] mirror: implement completion |
Date: |
Mon, 15 Oct 2012 19:49:07 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:13.0) Gecko/20120605 Thunderbird/13.0 |
Am 26.09.2012 17:56, schrieb Paolo Bonzini:
> Switching to the target of the migration is done mostly asynchronously,
> and reported to management via the BLOCK_JOB_COMPLETED event; the only
> synchronous phase is opening the backing files. bdrv_open_backing_file
> can always be done, even for migration of the full image (aka sync:
> 'full'). In this case, qmp_drive_mirror will create the target disk
> with no backing file at all, and bdrv_open_backing_file will be a no-op.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> block/mirror.c | 41 ++++++++++++++++++++++++++++++++++++-----
> 1 file modificato, 36 inserzioni(+), 5 rimozioni(-)
>
> diff --git a/block/mirror.c b/block/mirror.c
> index 09ea020..939834d 100644
> --- a/block/mirror.c
> +++ b/block/mirror.c
> @@ -32,6 +32,8 @@ typedef struct MirrorBlockJob {
> RateLimit limit;
> BlockDriverState *target;
> MirrorSyncMode mode;
> + bool synced;
> + bool complete;
Maybe rename this to should_complete or completion_requested? For a few
seconds I thought this would indicate that the job has completed.
Kevin
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v2 28/45] mirror: implement completion,
Kevin Wolf <=