Re: [libvirt] [PATCH v4 3/4] Implement the core API to suspend/resume the host

2011-11-28 Thread Srivatsa S. Bhat
On 11/28/2011 11:26 PM, Eric Blake wrote: > On 11/28/2011 10:46 AM, Srivatsa S. Bhat wrote: > +/* Check if the host supports the requested suspend state */ +switch (state) { +case VIR_NODE_S3: +if (hostPMFeatures & VIR_NODE_S3) { +cmdString

Re: [libvirt] [PATCH v4 3/4] Implement the core API to suspend/resume the host

2011-11-28 Thread Daniel P. Berrange
On Mon, Nov 28, 2011 at 10:56:33AM -0700, Eric Blake wrote: > On 11/28/2011 10:46 AM, Srivatsa S. Bhat wrote: > > >>> +/* Check if the host supports the requested suspend state */ > >>> +switch (state) { > >>> +case VIR_NODE_S3: > >>> +if (hostPMFeatures & VIR_NODE_S3) { > >>>

Re: [libvirt] [PATCH v4 3/4] Implement the core API to suspend/resume the host

2011-11-28 Thread Eric Blake
On 11/28/2011 10:46 AM, Srivatsa S. Bhat wrote: >>> +/* Check if the host supports the requested suspend state */ >>> +switch (state) { >>> +case VIR_NODE_S3: >>> +if (hostPMFeatures & VIR_NODE_S3) { >>> +cmdString = strdup("pm-suspend"); >>> +if (cmdStr

Re: [libvirt] [PATCH v4 3/4] Implement the core API to suspend/resume the host

2011-11-28 Thread Srivatsa S. Bhat
On 11/28/2011 05:50 PM, Daniel P. Berrange wrote: > On Mon, Nov 28, 2011 at 05:33:22PM +0530, Srivatsa S. Bhat wrote: >> Add the core functions that implement the functionality of the API. >> Suspend is done by using an asynchronous mechanism so that we can return >> the status to the caller befor

Re: [libvirt] [PATCH v4 3/4] Implement the core API to suspend/resume the host

2011-11-28 Thread Srivatsa S. Bhat
Hi Daniel, On 11/28/2011 05:50 PM, Daniel P. Berrange wrote: > On Mon, Nov 28, 2011 at 05:33:22PM +0530, Srivatsa S. Bhat wrote: >> Add the core functions that implement the functionality of the API. >> Suspend is done by using an asynchronous mechanism so that we can return >> the status to the

Re: [libvirt] [PATCH v4 3/4] Implement the core API to suspend/resume the host

2011-11-28 Thread Daniel P. Berrange
On Mon, Nov 28, 2011 at 05:33:22PM +0530, Srivatsa S. Bhat wrote: > Add the core functions that implement the functionality of the API. > Suspend is done by using an asynchronous mechanism so that we can return > the status to the caller before the host gets suspended. This asynchronous > operation

[libvirt] [PATCH v4 3/4] Implement the core API to suspend/resume the host

2011-11-28 Thread Srivatsa S. Bhat
Add the core functions that implement the functionality of the API. Suspend is done by using an asynchronous mechanism so that we can return the status to the caller before the host gets suspended. This asynchronous operation is achieved by suspending the host in a separate thread of execution. How