From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:57336) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SpFja-0002fP-2V for qemu-devel@nongnu.org; Thu, 12 Jul 2012 05:31:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SpFjU-0005O5-2p for qemu-devel@nongnu.org; Thu, 12 Jul 2012 05:30:53 -0400 Received: from mx1.redhat.com ([209.132.183.28]:46776) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SpFjT-0005O1-Ma for qemu-devel@nongnu.org; Thu, 12 Jul 2012 05:30:47 -0400 Message-ID: <4FFE993A.9090600@redhat.com> Date: Thu, 12 Jul 2012 12:30:34 +0300 From: Avi Kivity MIME-Version: 1.0 References: <1341501390-797-1-git-send-email-pbonzini@redhat.com> In-Reply-To: <1341501390-797-1-git-send-email-pbonzini@redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH uq/master 0/9] remove event_notifier_get_fd from non-KVM code List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: kvm@vger.kernel.org, mst@redhat.com, jan.kiszka@siemens.com, mtosatti@redhat.com, qemu-devel@nongnu.org, anthony.perard@citrix.com, stefano.stabellini@eu.citrix.com On 07/05/2012 06:16 PM, Paolo Bonzini wrote: > This is part 1 of a three-part series that expands usage of EventNotifier > in QEMU (including AIO and the main loop). I started working on this when > playing with the threaded block layer; the part of that work that I hope > will be in 1.2 is generalizing posix-aio-compat.c to be a generic portable > thread pool + porting AIO to Win32 (part 2). On top of this, discard > can be easily made asynchronous (part 3), which is a prerequisite for > enabling it. > > This first part does the necessary changes for porting EventNotifier > to Win32. The Win32 version will not have event_notifier_get_fd, > and thus I want to remove all calls in portable code. Instead, all > functions used in portable code after this series take an EventNotifier; > KVM-specific implementations retrieve the file descriptor internally > (these calls are in hw/ivshmem.c, hw/vhost.c, kvm-all.c). > > Patches 1 to 6 cover ivshmem and the memory API, first adding the > required EventNotifier APIs and then using them. Patches 7 to 9 do the > same with KVM ioeventfd and irqfd, refactoring transport-independent > code in the process from virtio-pci to virtio (the two steps are a bit > hard to separate). Looks good, all that is needed is an explanation for patch 2. -- error compiling committee.c: too many arguments to function