Re: [Qemu-devel] [PATCH 1/1] migration: qemu_savevm_state_cleanup becomes mandatory operation

["Denis V. Lunev" <den@openvz.org>]

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 <liang.z.li@intel.com>
>>>>       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 <den@openvz.org>
>>>> CC: Paolo Bonzini <pbonzini@redhat.com>
>>>> CC: Juan Quintela <quintela@redhat.com>
>>>> CC: Amit Shah <amit.shah@redhat.com>
>>>> ---
>>>>    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 <liang.z.li@intel.com>
>>>
>> yes
> Great.  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