Re: [PATCH for-4.7] libxl/save: set domain_suspend_state->domid in do_domain_soft_reset()

[Wei Liu <wei.liu2@citrix.com>]

Sorry for breaking it!

On Mon, Apr 11, 2016 at 02:20:04PM +0200, Vitaly Kuznetsov wrote:
> c/s d5c693d "libxl/save: Refactor libxl__domain_suspend_state" broke soft
> reset as libxl__domain_suspend_device_model() now fails when domid in not set
> in libxl__domain_suspend_state.
> 
> Signed-off-by: Vitaly Kuznetsov <vkuznets@redhat.com>

Acked-by: Wei Liu <wei.liu2@citrix.com>

What sort of arrangement is needed to get this tested? Any specific
kernel version etc?

> ---
>  tools/libxl/libxl_create.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/libxl/libxl_create.c b/tools/libxl/libxl_create.c
> index f1028bc..f8d5e36 100644
> --- a/tools/libxl/libxl_create.c
> +++ b/tools/libxl/libxl_create.c
> @@ -1700,7 +1700,7 @@ static int do_domain_soft_reset(libxl_ctx *ctx,
>      free(xs_console_mfn);
>  
>      dss->ao = ao;
> -    dss->domid = domid_soft_reset;
> +    dss->domid = dss->dsps.domid = domid_soft_reset;
>      dss->dsps.dm_savefile = GCSPRINTF(LIBXL_DEVICE_MODEL_SAVE_FILE".%d",
>                                        domid_soft_reset);
>  
> -- 
> 2.5.5
> 

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