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

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Mon, 2014-03-17 at 11:55 +0000, Ian Jackson wrote:
> Ian Campbell writes ("Re: [PATCH 16/19] libxl: suspend: Abolish usleeps in 
> domain suspend wait"):
> > 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.
> 
> So it turns out that doing this makes the final patch quite fiddly and
> annoying: it needs to have a separate timeout anyway because it's also
> waiting for event channels.  Making the two paths separate in this way
> produces an extra callback function etc.

OK, not to worry.

> So I'm going to go back to the previous approach.  But I guess I might
> as well leave the patch "libxl: events: libxl__xswait* support @paths"
> in the series even though nothing uses that right now.

Agreed.

Ian.



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