On Wed, May 24, 2017 at 10:55:16AM +0200, Juan Quintela wrote:
> Hi
>
> We just have a loadvm handlers that are a new list only used in a
> single place. Just move everything to use the savevm_handlers (yes,
> it is a list, and we could have a better name).
>
> Once there, vmstate_load() had three arguments but only needs two. Fix that.
>
> Please, review.
>
> Juan Quintela (3):
> migration: Use savevm_handlers instead of loadvm copy
> migration: loadvm handlers are not used
> migration: Remove section_id parameter from vmstate_load
>
> include/migration/migration.h | 5 ----
> include/migration/vmstate.h | 2 --
> include/qemu/typedefs.h | 1 -
> migration/migration.c | 2 --
> migration/savevm.c | 58 ++++++++++++-------------------------------
> 5 files changed, 16 insertions(+), 52 deletions(-)
>
> --
> 2.9.3
>
This series looks nice to me. :-)
Series:
Reviewed-by: Peter Xu <peterx@redhat.com>
Thanks,
--
Peter Xu