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

Re: [Xen-devel] [PATCH v1 12/12] libxl: add device backend listener in order to launch backends

On 04/11/13 18:59, Ian Jackson wrote:
> Ian Jackson writes ("Re: [PATCH v1 12/12] libxl: add device backend listener 
> in order to launch backends"):
>>  It would give you a new psuedo-ao, which you can use for
>> per-event memory allocation.  It's a psuedo-ao in the sense that you
>> mustn't call libxl__ao_abort or libxl__ao_complete on it, but it would
>> have the right type and in particular you could stuff it in
>> sub-operations' ao fields, call STATE_AO_GC on it and so on.  I could
>> make it possible to call libxl__ao_inprogress and have that reflected
>> to the underlyhing real ao.
>>> That sounds like a good solution to my problem, I wouldn't mind if you
>>> write that :)
>> OK, watch this space.
> Something like this perhaps.  I've compiled it but I don't have a
> caller to test it with.
> Ian.
> From be117b96d0072a995ffa77ee1a763a3bdc98b982 Mon Sep 17 00:00:00 2001
> From: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
> Date: Mon, 4 Nov 2013 17:56:15 +0000
> Subject: [PATCH] libxl: Introduce nested async operations (nested ao)
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
> This allows a long-running ao to avoid accumulating memory.  Each
> nested ao has its own gc.
> Signed-off-by: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
> Cc: Roger Pau Monné <roger.pau@xxxxxxxxxx>
> Cc: Ian Campbell <ian.campbell@xxxxxxxxxx>

Acked-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>
Tested-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>

The interface looks fine to me, maybe other potential users of this
interface have comments, but since it's an internal interface we can
always expand it. I've added this patch to my series, I'm not sure if
you prefer to commit it separately.

Xen-devel mailing list



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