Re: [Xen-devel] [PATCH for-2.4 v3 2/3] migration: Add configuration section to vmstate with xen.

[Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>]

On Mon, 3 Aug 2015, Anthony PERARD wrote:
> This adds the configuration section in the vmstate saved by
> xen_save_devices_state.
> 
> Signed-off-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>

It doesn't seem to me that this patch is actually necessary to fix the
issue, is that right? 

If that is the case, I would rather skip it.


>  migration/savevm.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/migration/savevm.c b/migration/savevm.c
> index 6071215..b3f605c 100644
> --- a/migration/savevm.c
> +++ b/migration/savevm.c
> @@ -961,6 +961,11 @@ static int qemu_save_device_state(QEMUFile *f)
>  
>      cpu_synchronize_all_states();
>  
> +    if (!savevm_state.skip_configuration) {
> +        qemu_put_byte(f, QEMU_VM_CONFIGURATION);
> +        vmstate_save_state(f, &vmstate_configuration, &savevm_state, 0);
> +    }
> +
>      QTAILQ_FOREACH(se, &savevm_state.handlers, entry) {
>          if (se->is_ram) {
>              continue;
> -- 
> Anthony PERARD
> 

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel