Re: [Xen-devel] [PATCH V2] libxl: introduce asynchronous execution API

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Mon, 2014-05-12 at 15:58 +0100, Ian Jackson wrote:
> Yang Hongyang writes ("[PATCH V2] libxl: introduce asynchronous execution 
> API"):
> > 1.introduce asynchronous execution API:
> >   libxl__async_exec_init
> >   libxl__async_exec_start
> >   libxl__async_exec_inuse
> > 2.use the async exec API to execute device hotplug scripts
> > 
> > Signed-off-by: Lai Jiangshan <laijs@xxxxxxxxxxxxxx>
> > Signed-off-by: Wen Congyang <wency@xxxxxxxxxxxxxx>
> > Signed-off-by: Yang Hongyang <yanghy@xxxxxxxxxxxxxx>
> > Acked-by: Roger Pau Monnà <roger.pau@xxxxxxxxxx>
> 
> Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
> 
> AIUI the motivation for this is that this code needs to be reused by
> some of the remus series.  In general it is more usual to retain a
> patch like this as an initial part of such a series, even after it has
> been acked.  It would normally be applied with the rest of the series,
> or perhaps if there is substantial initial subseires which is acked.
> 
> But given that this is a refactoring patch and is of quite high
> quality IMO I think it would be reasonable to apply it now.
> 
> Ian C, what do you think ?

I've not reviewed the patch itself but if you think it is ready then
throwing it in now is fine by me, no need to wait for the remus stuff
IMHO.

Ian.


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