From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Cooper Subject: Re: [PATCH Remus v3 3/3] libxc/restore: implement Remus checkpointed restore Date: Wed, 13 May 2015 17:38:58 +0100 Message-ID: <55537E22.8090405@citrix.com> References: <1431506101-29612-1-git-send-email-yanghy@cn.fujitsu.com> <1431506101-29612-4-git-send-email-yanghy@cn.fujitsu.com> <555378D3.5000800@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <555378D3.5000800@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Yang Hongyang , xen-devel@lists.xen.org Cc: wei.liu2@citrix.com, ian.campbell@citrix.com, wency@cn.fujitsu.com, guijianfeng@cn.fujitsu.com, yunhong.jiang@intel.com, eddie.dong@intel.com, rshriram@cs.ubc.ca, ian.jackson@eu.citrix.com List-Id: xen-devel@lists.xenproject.org On 13/05/15 17:16, Andrew Cooper wrote: > >> + >> /* >> * Xenstore and Console parameters. >> * INPUT: evtchn & domid >> diff --git a/tools/libxc/xc_sr_restore.c b/tools/libxc/xc_sr_restore.c >> index 0e512ec..85534a8 100644 >> --- a/tools/libxc/xc_sr_restore.c >> +++ b/tools/libxc/xc_sr_restore.c >> @@ -468,10 +468,83 @@ static int handle_page_data(struct xc_sr_context *ctx, struct xc_sr_record *rec) >> return rc; >> } >> >> +static int process_record(struct xc_sr_context *ctx, struct xc_sr_record *rec); >> +static int handle_checkpoint(struct xc_sr_context *ctx) >> +{ >> + xc_interface *xch = ctx->xch; >> + int rc = 0, i; > Always an unsigned i for populated_pfnsindexing an array please. Erm - not sure what happened here. Ignore the "populated_pfns" please. Sorry. ~Andrew