Re: [Xen-devel] [PATCH 16/19] libxl: suspend: Abolish usleeps in domain suspend wait

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Fri, 2014-03-14 at 18:16 +0000, Ian Jackson wrote:

> I think it's marginally clearer separately and there's no point
> folding it in.

OK.

[...]
> From d337d7cf831360b0e7ed8f672db6b2559d063271 Mon Sep 17 00:00:00 2001
> From: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
> Date: Fri, 6 Dec 2013 16:12:44 +0000
> Subject: [PATCH 18/21] libxl: suspend: Abolish usleeps in domain suspend wait
> 
> Replace the use of a loop with usleep().
> 
> Instead, use an xswait xenstore watch with timeout.  (xenstore fires
> watches on @releaseDomain when a domain shuts down.)
> 
> The logic which checks for the state of the domain is unchanged, and
> not ideal, but we will leave that for the next patch.
> 
> There is not intended to be any semantic change, other than to make
> the algorithm properly asynchronous and the consequential waiting be
> on xenstore, rather than polling.
> 
> Signed-off-by: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
> CC: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>

Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel