From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?ISO-8859-1?Q?Roger_Pau_Monn=E9?= Subject: Re: [PATCH v1 12/12] libxl: add device backend listener in order to launch backends Date: Wed, 6 Nov 2013 13:15:32 +0100 Message-ID: <527A32E4.50906@citrix.com> References: <1380705874-58491-1-git-send-email-roger.pau@citrix.com> <1380705874-58491-13-git-send-email-roger.pau@citrix.com> <21105.15127.428243.701439@mariner.uk.xensource.com> <5277D362.4040300@citrix.com> <21111.55145.501074.162529@mariner.uk.xensource.com> <21111.57474.653370.258557@mariner.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mail6.bemta4.messagelabs.com ([85.158.143.247]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1Ve21J-0004OP-VN for xen-devel@lists.xenproject.org; Wed, 06 Nov 2013 12:15:38 +0000 In-Reply-To: <21111.57474.653370.258557@mariner.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Jackson , xen-devel@lists.xenproject.org, Ian Campbell List-Id: xen-devel@lists.xenproject.org On 04/11/13 18:59, Ian Jackson wrote: > Ian Jackson writes ("Re: [PATCH v1 12/12] libxl: add device backend liste= ner 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 > 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=3DUTF-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 > Cc: Roger Pau Monn=E9 > Cc: Ian Campbell Acked-by: Roger Pau Monn=E9 Tested-by: Roger Pau Monn=E9 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.