From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:50895) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TLIlz-00076s-Fz for qemu-devel@nongnu.org; Mon, 08 Oct 2012 15:13:52 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TLIly-00085T-Hd for qemu-devel@nongnu.org; Mon, 08 Oct 2012 15:13:51 -0400 Received: from mx1.redhat.com ([209.132.183.28]:49599) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TLIly-00085K-8C for qemu-devel@nongnu.org; Mon, 08 Oct 2012 15:13:50 -0400 Date: Mon, 8 Oct 2012 22:15:40 +0200 From: "Michael S. Tsirkin" Message-ID: <20121008201539.GC17303@redhat.com> References: <20121002191609.31100.77382.stgit@bling.home> <1349711912.2759.96.camel@ul30vt.home> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1349711912.2759.96.camel@ul30vt.home> Subject: Re: [Qemu-devel] [PATCH 0/6] Misc PCI cleanups List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alex Williamson Cc: jan.kiszka@siemens.com, qemu-devel@nongnu.org On Mon, Oct 08, 2012 at 09:58:32AM -0600, Alex Williamson wrote: > Michael, Jan, > > Any comments on these? I'd like to make the PCI changes before I update > vfio-pci to make use of the new resampling irqfd in kvm. We don't have > anyone officially listed as maintainer of pci-assign since it's been > moved to qemu. I could include the pci-assign patches in my tree if you > prefer. Thanks, > > Alex Patches themselves look fine, but I'd like to better understand why do we want the INTx fallback. Isn't it easier to add intx routing support? > On Tue, 2012-10-02 at 13:21 -0600, Alex Williamson wrote: > > A few cleanups that I'll also apply to vfio-pci. First make intx > > route checking non-fatal. vfio-pci has a fallback INTx mechanism > > that doesn't rely on this, so we can already run on q35, but not if > > we can't even probe for intx routing w/o blowing up. Next, both > > vfio-pci and pci-assign test whether INTx routing has changed using > > similar functions. Make this common. Finally, expose a way to > > get the MSI message for an MSI vector. Again, both pci-assign and > > vfio-pci need to do this to program the vector for KVM injection. > > Thanks, > > > > Alex > > > > --- > > > > Alex Williamson (6): > > pci-assign: Use msi_get_message() > > msi: Add msi_get_message() > > pci-assign: Use pci_intx_route_changed() > > pci: Helper function for testing if an INTx route changed > > pci-assign: Add support for no-route > > pci: Add INTx no-route option > > > > > > hw/kvm/pci-assign.c | 14 ++++++++------ > > hw/msi.c | 45 +++++++++++++++++++++++++++++---------------- > > hw/msi.h | 1 + > > hw/pci.c | 13 +++++++++++-- > > hw/pci.h | 2 ++ > > 5 files changed, 51 insertions(+), 24 deletions(-) > >