From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH v3 0/4] VFIO-based PCI device assignment for QEMU 1.2 Date: Wed, 15 Aug 2012 16:45:20 +0300 Message-ID: <502BA7F0.80602@redhat.com> References: <20120814202141.11522.78340.stgit@bling.home> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Cc: aliguori@us.ibm.com, kvm@vger.kernel.org, aik@ozlabs.ru, jan.kiszka@siemens.com, qemu-devel@nongnu.org, blauwirbel@gmail.com To: Alex Williamson Return-path: In-Reply-To: <20120814202141.11522.78340.stgit@bling.home> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org Sender: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org List-Id: kvm.vger.kernel.org On 08/14/2012 11:32 PM, Alex Williamson wrote: > v3: > - Incorporate feedback from Anthony, Avi, and Jan (Thank you!) > > I've also added a new tag to my github tree, so those who have > already reviewed can check the delta by comparing: > > git://github.com/awilliam/qemu-vfio.git > tags/vfio-pci-for-qemu-1.2 > tags/vfio-pci-for-qemu-1.2-v2 > tags/vfio-pci-for-qemu-1.2-v3 > > Blue Swirl already offered an Ack-by for previous version, I'll > hope, but won't assume that carries forward. Original description > below with updated tag. Thanks, > > Alex > > 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. Reviewed-by: Avi Kivity -- error compiling committee.c: too many arguments to function