Re: [Xen-devel] [RFC PATCH v2 00/29] libxl: Cancelling asynchronous operations

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Ian Campbell writes ("Re: [Xen-devel] [RFC PATCH v2 00/29] libxl: Cancelling 
asynchronous operations"):
> On Tue, 2015-02-10 at 20:09 +0000, Ian Jackson wrote:
> > This is v2 of my work-in-progress series to support cancellation of
> > long-running libxl operations.
> [...]
> > I wouldn't recommend testing it yet until I've at least smoke tested
> > it to see that things still work if you don't cancel them.
> 
> Would review of the series be useful and/or appreciated at this stage?

Review of the APIs, and general approach, would be very much
appreciated.  That's probably best done by looking at the tip and
diffing libxl[_internal].h.

> Perhaps the first half dozen or so look like preparatory cleanups which
> I could sensibly look at?

That would also be useful.

> > Here's a list of the patches:
> > 
> >   01/29  libxl: Further fix exit paths from libxl_device_events_handler
> >   02/29  libxl: Comment cleanups
> >   03/29  libxl: suspend: switch_logdirty_done takes rc
> >   04/29  libxl: suspend: common suspend callbacks take rc
> >   05/29  libxl: suspend: Return correct error from callbacks
> >   06/29  libxl: Use libxl__xswait* in libxl__ao_device
> >   07/29  libxl: xswait/devstate: Move xswait to before devstate
> >   08/29  libxl: devstate: Use libxl__xswait*

These first eight are cleanups and could in principle go in right
away.

Thanks,
Ian.

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