On Thu, 2015-07-09 at 19:26 +0100, Andrew Cooper wrote:
> #define LIBXL_HAVE_SOCKET_BITMAP_ALLOC 1
>
> +/*
> + * LIBXL_HAVE_STREAM_V2
These all need another word in them I think. _MIGRATION_, or _SRM_
or ... (I think I prefer SRM since migration is just one aspect)
> @@ -439,6 +447,28 @@ static void record_body_done(libxl__egc *egc,
> stream_failed(egc, stream, ret);
> }
>
> +void libxl__xc_domain_restore_done(libxl__egc *egc, void *dcs_void,
> + int ret, int retval, int errnoval)
> +{
> + libxl__domain_create_state *dcs = dcs_void;
> + STATE_AO_GC(dcs->ao);
> +
> + if (ret)
> + goto err;
> +
> + if (retval) {
> + LOGEV(ERROR, errnoval, "restoring domain");
> + ret = ERROR_FAIL;
> + goto err;
> + }
> +
> + stream_continue(egc, &dcs->srs);
This confused me briefly (done==continue??), a comment indicating that
this callback represents libxc handing back control of the fd to libxl
would maybe be useful?
With those two minor tweaks,
Acked-by: Ian Campbell <ian.campb...@citrix.com>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
