[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Fix incorrect state information for migrate_can
From: |
Luiz Capitulino |
Subject: |
Re: [Qemu-devel] [PATCH] Fix incorrect state information for migrate_cancel |
Date: |
Fri, 6 Dec 2013 09:00:49 -0500 |
On Thu, 05 Dec 2013 18:49:15 -0700
"Lin Ma" <address@hidden> wrote:
> In qemu-1.4.x, When performing migrate_cancel on hmp, Sometimes
> s->state is incorrrectly saved to MIG_STATE_ERROR instead of
> MIG_STATE_CANCELLED.
>
> If the migrate_fd_cancel in main thread is scheduled to run before
> the thread buffered_file_thread calls migrate_fd_put_buffer, The
> s->state will be modified to MIG_STATE_CANCELLED by main thread,
> Then the migrate_fd_put_buffer in thread buffered_file_thread will
> return -EIO if s->state != MIG_STATE_ACTIVE, This incorrect return
> value trigged migrate_fd_error to set s->state = MIG_STATE_ERROR.
>
> This patch fixes the issue in qemu-1.4.x.
Is qemu 1.4 still maintained?
>
> Signed-off-by: Lin Ma <address@hidden>
> ---
> migration.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/migration.c b/migration.c
> index 98c7696..0ac4608 100644
> --- a/migration.c
> +++ b/migration.c
> @@ -751,7 +751,7 @@ static void *buffered_file_thread(void *opaque)
> }
>
> out:
> - if (ret < 0) {
> + if (ret < 0 && !s->complete) {
> migrate_fd_error(s);
> }
> g_free(s->buffer);