On Thu, 2015-07-09 at 19:26 +0100, Andrew Cooper wrote:
> And call it when a checkpoint record is found in the libxc stream.
>
> Some parts of this patch have been based on patches from the COLO series.
>
> Signed-off-by: Wen Congyang <we...@cn.fujitsu.com>
> Signed-off-by: Yang Hongyang <yan...@cn.fujitsu.com>
> Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com>
> CC: Ian Campbell <ian.campb...@citrix.com>
> CC: Ian Jackson <ian.jack...@eu.citrix.com>
> CC: Wei Liu <wei.l...@citrix.com>
>
> ---
> v2: Borrow sufficient fragments from several COLO patches to get
> BROKEN_CHANNEL and checkpoint failover to function.
> ---
> tools/libxc/include/xenguest.h | 9 ++++++
> tools/libxc/xc_sr_common.h | 7 +++--
> tools/libxc/xc_sr_restore.c | 53
> ++++++++++++++++++++++++++----------
> tools/libxl/libxl_save_msgs_gen.pl | 2 +-
> 4 files changed, 53 insertions(+), 18 deletions(-)
>
> diff --git a/tools/libxc/include/xenguest.h b/tools/libxc/include/xenguest.h
> index 7581263..8799c9f 100644
> --- a/tools/libxc/include/xenguest.h
> +++ b/tools/libxc/include/xenguest.h
> @@ -102,6 +102,15 @@ struct restore_callbacks {
> int (*toolstack_restore)(uint32_t domid, const uint8_t *buf,
> uint32_t size, void* data);
>
> + /* A checkpoint record has been found in the stream.
> + *
> + * returns:
> + * 0: (error) terminate processing
> + * 1: (success) continue normally
Where "continue" means "keep reading from the stream and expect more
checkpoints" (as opposed to continue on to running the guest), I think?
With that clarified: Acked-by: Ian Campbell <ian.campb...@citrix.com>
Ian
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
