[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/3] Remove of loadvm handlers
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH 0/3] Remove of loadvm handlers |
Date: |
Wed, 24 May 2017 17:41:01 +0800 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
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 <address@hidden>
Thanks,
--
Peter Xu