On Wed, May 17, 2017 at 04:45:44PM +1000, David Gibson wrote: > On Wed, May 17, 2017 at 09:19:18AM +0530, Bharata B Rao 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 <bhar...@linux.vnet.ibm.com> > > It's not my bailiwick, so I don't get final say, but I dislike > changing the signature of the existing unregister_savevm() interface. > I think it would be preferable to only add the 'live' paramter to a > new unregister_savevm_common() (or whatever) function.
Yeah, much better, let me change this in next version. Regards, Bharata.