[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 4 of 6 V3] libxl: support suspend_cancel in domain_resume

On 2012-02-09, at 1:16 AM, Ian Campbell <Ian.Campbell@xxxxxxxxxx> wrote:

> On Fri, 2012-02-03 at 06:50 +0000, rshriram@xxxxxxxxx wrote:
>> # 
> Previously this code would have been equivalent to passing 0 not 1. It
> may be ok to change this but it should be separate.

> However I'm a bit
> dubious about changing it without adding some code to detect if the
> guest supports it.

It requires writing the suspend-cancel entry to xenstore on domain creation (if 
xen says that the guest supports it)

 And then read this field once during initialization in checkpoint/Remus code.

I haven't properly investigated the domain create code path. Any pointers on 
where I should start ?

> (I know libxl_domain_resume is currently broken for
> the non-cooperative suspend

It's broken in such a way that neither domain checkpoint or Remus work.
I ll take up your comment on the previous version of the patch:
 Add a new public (or internal) API 

and make the checkpoint and Remus related code call this instead of the 

Later, after adding the suspend-cancel xenstore entry creation patch, I can 
switch the code to using the above function with the cooperative flag.

> case but we shouldn't paper over that)
> Ian.

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.