[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/1] migration: qemu_savevm_state_cleanup become
From: |
Li, Liang Z |
Subject: |
Re: [Qemu-devel] [PATCH 1/1] migration: qemu_savevm_state_cleanup becomes mandatory operation |
Date: |
Mon, 9 Nov 2015 06:16:34 +0000 |
> 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 complete
> operations.
> >>
> >> 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 index
> e05158d..9f2230f
> >> 100644
> >> --- 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.0
> >>
> >
> > Yes, you are right. Thanks a lot.
> >
> > BTW, can this patch fix the regression you reported?
> >
> > Reviewed-by: Liang Li <address@hidden>
> >
> yes
Great. You'd better change the commit message to make it more clear.
Liang
Re: [Qemu-devel] assert during internal snapshot, Li, Liang Z, 2015/11/08
Re: [Qemu-devel] assert during internal snapshot, Li, Liang Z, 2015/11/08
Re: [Qemu-devel] assert during internal snapshot, Li, Liang Z, 2015/11/08
[Qemu-devel] [PATCH for 2.5 v2 1/1] migration: qemu_savevm_state_cleanup becomes mandatory operation, Denis V. Lunev, 2015/11/09