[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH] Fix incorect state information for migrate_cancel
From: |
Lin Ma |
Subject: |
[Qemu-devel] [PATCH] Fix incorect state information for migrate_cancel |
Date: |
Sun, 1 Dec 2013 23:24:22 +0800 |
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.
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);
--
1.8.4
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Qemu-devel] [PATCH] Fix incorect state information for migrate_cancel,
Lin Ma <=