[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH v3 1/3] migration: Introduce unregister_savevm_liv
From: |
Juan Quintela |
Subject: |
Re: [Qemu-ppc] [PATCH v3 1/3] migration: Introduce unregister_savevm_live() |
Date: |
Wed, 24 May 2017 09:18:08 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) |
David Gibson <address@hidden> wrote:
> On Wed, May 24, 2017 at 08:28:59AM +0200, Juan Quintela wrote:
>> Bharata B Rao <address@hidden> wrote:
>> > Introduce a new function unregister_savevm_live() to unregister the vmstate
>> > handlers registered via register_savevm_live().
>> >
>> > register_savevm() allocates SaveVMHandlers while register_savevm_live()
>> > gets passed with SaveVMHandlers. During unregistration, we want to
>> > free SaveVMHandlers in the former case but not free in the latter case.
>> > Hence this new API is needed to differentiate this.
>> >
>> > This new API will be needed by PowerPC to unregister the HTAB savevm
>> > handlers.
>> >
>> > Signed-off-by: Bharata B Rao <address@hidden>
>> > Reviewed-by: David Gibson <address@hidden>
>> > Cc: Juan Quintela <address@hidden>
>> > Cc: Dr. David Alan Gilbert <address@hidden>
>>
>> Hi
>>
>> How about this one?
>> I just test compiled it.
>>
>> Advantage from my point of view is that we always do the right thing.
>> And as migration code already knows if it has to be freed or not, I
>> think it is a better API.
>>
>> What do you think?
>
> I think this is a better approach. Do you want to push this one
> directly, Juan, or do you want me to take it through my tree?
I will push it on my next pull request. I mean, I will send it for
review on own top level.
Thanks, Juan.