From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:40506) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1T10Q1-0007oO-FF for qemu-devel@nongnu.org; Mon, 13 Aug 2012 15:35:21 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1T10Px-0008Lb-Hk for qemu-devel@nongnu.org; Mon, 13 Aug 2012 15:35:17 -0400 Received: from e32.co.us.ibm.com ([32.97.110.150]:53180) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1T10Px-0008LX-BH for qemu-devel@nongnu.org; Mon, 13 Aug 2012 15:35:13 -0400 Received: from /spool/local by e32.co.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Mon, 13 Aug 2012 13:35:12 -0600 Received: from d03relay02.boulder.ibm.com (d03relay02.boulder.ibm.com [9.17.195.227]) by d03dlp03.boulder.ibm.com (Postfix) with ESMTP id DFB4219D8041 for ; Mon, 13 Aug 2012 13:34:14 -0600 (MDT) Received: from d03av04.boulder.ibm.com (d03av04.boulder.ibm.com [9.17.195.170]) by d03relay02.boulder.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id q7DJXoqf038756 for ; Mon, 13 Aug 2012 13:34:01 -0600 Received: from d03av04.boulder.ibm.com (loopback [127.0.0.1]) by d03av04.boulder.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id q7DJXYHf030665 for ; Mon, 13 Aug 2012 13:33:34 -0600 From: Anthony Liguori In-Reply-To: <1344867810.4683.21.camel@ul30vt.home> References: <20120801050241.22163.78549.stgit@bling.home> <87pq6uvs52.fsf@codemonkey.ws> <1344867810.4683.21.camel@ul30vt.home> Date: Mon, 13 Aug 2012 14:33:28 -0500 Message-ID: <87zk5ylh87.fsf@codemonkey.ws> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Subject: Re: [Qemu-devel] [PATCH 0/3] VFIO-based PCI device assignment for QEMU 1.2 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alex Williamson Cc: Andreas Hartmann , kvm@vger.kernel.org, aik@ozlabs.ru, Jan Kiszka , qemu-devel@nongnu.org, Blue Swirl , Avi Kivity Alex Williamson writes: > On Mon, 2012-08-13 at 08:27 -0500, Anthony Liguori wrote: >> Alex Williamson writes: >> >> > VFIO kernel support was just merged into Linux, so I'd like to >> > formally propose inclusion of the QEMU vfio-pci driver for >> > QEMU 1.2. Included here is support for x86 PCI device assignment. >> > PCI INTx is not yet enabled, but devices making use of either MSI >> > or MSI-X work. The level irqfd and eoifd support I've proposed >> > for KVM enable an accelerated patch for this through KVM. I'd >> > like to get this base driver in first and enable the remaining >> > support in-tree. >> > >> > I've split this version up a little from the RFC to make it a bit >> > easier to review. Review comments from Blue Swirl and Avi are >> > already incorporated, including Avi's requests to simplify both >> > the PCI BAR mapping and unmapping paths. >> >> Hi Alex, >> >> Thanks for pushing this forward! Hopefully this will finally kill off >> qemu-kvm.git for good. >> >> I think this series is going to have to wait for 1.3 to open up. We >> have a very short release window for this release and I'd feel a lot >> more comfortable having such a significant feature spend some time in >> the development cycle getting testing/review. >> >> I'd like to see a few Reviewed-by's too for this series before it goes >> in. I expect they won't be hard to get but I also expect it will take a >> few more revisions of this series to get there. > > That's disappointing, but I can understand your reluctance. Blue Swirl > reviewed the RFC and could perhaps add a Reviewed-by. Alexey has been > working on the POWER port and I'm sure could provide a Reviewed-by. We > also have a few early adopters that are already making use of this code. > Towards accepting it, the driver is entirely self contained, there's > really no risk to the rest of qemu. The only missing functionality is > legacy interrupt support. Perhaps there's a compromise where this > driver could be considered a tech preview in 1.2 (x-vfio-pci?). > Thanks, Yeah, if a few people were willing to at least give an Acked-by by Wednesday, I'd be okay taking this in a "preview" or something like that. I wouldn't bother renaming it or anything like that. We can just declare in the release notes that it's an experimental feature and may eat your lunch while you're not looking. Regards, Anthony Liguori > > Alex > > -- > To unsubscribe from this list: send the line "unsubscribe kvm" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html