|
From: | Denis V. Lunev |
Subject: | Re: [Qemu-devel] [PATCH 1/1] migration: qemu_savevm_state_cleanup becomes mandatory operation |
Date: | Mon, 9 Nov 2015 09:18:41 +0300 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 11/09/2015 09:16 AM, Li, Liang Z wrote:
On 11/09/2015 08:10 AM, Li, Liang Z 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 completeoperations.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 indexe05158d..9f2230f100644 --- 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.0Yes, you are right. Thanks a lot. BTW, can this patch fix the regression you reported? Reviewed-by: Liang Li <address@hidden>yesGreat. You'd better change the commit message to make it more clear. Liang
argh.. you are right... This problem has appeared in the end of big rework of another problem with snapshots and dataplane. Sorry that this is not clear that regression is fixed. I'll resend the patch with better commit message Den
[Prev in Thread] | Current Thread | [Next in Thread] |