From mboxrd@z Thu Jan 1 00:00:00 1970 From: Juan Quintela Subject: Re: [PATCH 05/24] compatfd is included before, and it is compiled unconditionally Date: Thu, 01 Oct 2009 13:58:10 +0200 Message-ID: References: <6e46fe952ba8d1896e3cab5b24232828d3f827a9.1253272938.git.quintela@redhat.com> <20090922131901.GA22109@infradead.org> <20090922133438.GA12443@infradead.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: kvm@vger.kernel.org To: Christoph Hellwig Return-path: Received: from mx1.redhat.com ([209.132.183.28]:14207 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756509AbZJAL6Y (ORCPT ); Thu, 1 Oct 2009 07:58:24 -0400 In-Reply-To: <20090922133438.GA12443@infradead.org> (Christoph Hellwig's message of "Tue, 22 Sep 2009 09:34:38 -0400") Sender: kvm-owner@vger.kernel.org List-ID: Christoph Hellwig wrote: > On Tue, Sep 22, 2009 at 03:25:13PM +0200, Juan Quintela wrote: >> Christoph Hellwig wrote: >> > Btw, what's the state of getting compatfd upstream? It's a pretty >> > annoying difference between qemu upstream and qemu-kvm. >> >> I haven't tried. I can try to send a patch. Do you have any use case >> that will help the cause? > > Well, the eventfd compat is used in the thread pool AIO code. I don't > know what difference it makes, but I really hate this code beeing > different in both trees. I want to see compatfd used either in both or > none. Discused with Anthony about it. signalfd is complicated for qemu upstream (too difficult to use properly), and eventfd ... The current eventfd emulation is worse than the pipe code that it substitutes. His suggestion here was to create a new abstraction with an API like: push_notify() pop_notify() and then you can implement it with eventfd() pipes/whatever. What was missing for you of compatfd: qemu_eventfd/qemu_signalfd? Do a push_notify()/pop_notify() work for you? Later, Juan.