From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56109) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UzCjp-00010s-HQ for qemu-devel@nongnu.org; Tue, 16 Jul 2013 17:24:56 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UzCji-000780-FC for qemu-devel@nongnu.org; Tue, 16 Jul 2013 17:24:49 -0400 Received: from mail.avalus.com ([2001:41c8:10:1dd::10]:49019) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UzCjh-00077D-Rr for qemu-devel@nongnu.org; Tue, 16 Jul 2013 17:24:41 -0400 Date: Tue, 16 Jul 2013 22:24:38 +0100 From: Alex Bligh Message-ID: In-Reply-To: <51E57AF3.1050409@redhat.com> References: <1373127897-3445-1-git-send-email-alex@alex.org.uk> <51E4063D.6010308@redhat.com> <75638CBA408455BF52192DA7@Ximines.local> <51E4613D.9000106@redhat.com> <44590808AF4A6E7DC093637A@nimrod.local> <51E4E54A.10908@redhat.com> <51E4F77C.2090509@redhat.com> <794E19D97CCC267CCBFA8397@Ximines.local> <51E56A1A.50502@redhat.com> <19631228D7B62545DC6A2928@Ximines.local> <51E57AF3.1050409@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Content-Disposition: inline Subject: Re: [Qemu-devel] [PATCH] [RFC] aio/async: Add timed bottom-halves Reply-To: Alex Bligh List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: Kevin Wolf , Anthony Liguori , Stefan Hajnoczi , qemu-devel@nongnu.org, Alex Bligh , rth@twiddle.net Paolo, --On 16 July 2013 18:55:15 +0200 Paolo Bonzini wrote: >> What do you think? In the end I thought the schedule_bh_at stuff >> was simpler. > > It is simpler, but I'm not sure it is the right API. Of course, if > Kevin and Stefan says it is, I have no problem with that. For the sake of having something to comment on, I just sent v3 of this patch to the list. This is basically a 'minimal change' version that fixes the issue with aio_poll (I think). It passes make check. Stefan? Kevin? -- Alex Bligh