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

RE: [Xen-devel] [PATCH 0/4] HVM Virtual S3



Keir Fraser wrote:
> On 14/5/08 10:50, "Ke, Liping" <liping.ke@xxxxxxxxx> wrote:
> 
>> 1. if hvm in s3suspended, could we allow
>>     pause/unpause/suspend/resume operation? Need we add condition
>> guard in python code? 
> 
> I think pause/unpause still makes sense. Obviously a paused and
> s3suspended guest will not run again until both s3 resumed and
> unpaused. As for suspend/resume, that should also just work I think?
> Isn't the point to be able to s3-suspend then save then restore then
> s3-resume? 
> 
Yes, now S3 could works with save/restore. Since suspend/resume has 
Similar mechanism with save/restore, it works. I just thought s3_suspend/resume 
is a kind of suspend/resume already?
As for pause/unpause, since after s3_supend is now already in pause state,
I just thought is it a little interleaving? If you like, I will make it work 
firstly.

>> 2. when we add a new flag is_s3_suspended in dominfo structure, it
>> will affect other Trace/statistic tools in xen, seems this is not a
>>    very small change? The above two changes is ok for you or not?
> 
> I don't really care to have is_s3_suspended visible via dominfo.
> Leave it hidden if you like. Xend should be able to remember whether
> it s3-suspended a domain, if it needs to. Or we can expose it via an
> interface other than dominfo. We could, for example, make it an
> HVM_PARAM. Then you can enter/leave/interrogate s3suspend state via
> the existing hvm_op param interfaces.
Ok. I will use the first solution and do not revise trace related code.
If we want to expose this state in the furture, first solution is easier to 
modify.
> 
>  -- Keir

Thanks a lot for your help:)
Regards,
Criping


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

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