From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44525) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z7yQA-00078Z-82 for qemu-devel@nongnu.org; Thu, 25 Jun 2015 00:05:51 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Z7yQ6-00039x-4s for qemu-devel@nongnu.org; Thu, 25 Jun 2015 00:05:50 -0400 Received: from mail.makefile.in ([192.249.59.119]:48413) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z7yQ6-00039Q-1d for qemu-devel@nongnu.org; Thu, 25 Jun 2015 00:05:46 -0400 From: Bandan Das References: <1435197842-26505-1-git-send-email-glaupre@chelsio.com> Date: Thu, 25 Jun 2015 00:05:43 -0400 In-Reply-To: <1435197842-26505-1-git-send-email-glaupre@chelsio.com> (Gabriel Laupre's message of "Wed, 24 Jun 2015 19:04:02 -0700") Message-ID: MIME-Version: 1.0 Content-Type: text/plain Subject: Re: [Qemu-devel] [PATCH] pci : Add pba_offset PCI quirk for Chelsio T5 devices List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Gabriel Laupre Cc: jb-gnumlists@wisemo.com, leedom@chelsio.com, mst@redhat.com, qemu-devel@nongnu.org, anish@chelsio.com, mboksanyi@chelsio.com, alex.williamson@redhat.com, bsd@makefile.in Hi Gabriel, Glad that you got to the bottom of this! :) Gabriel Laupre writes: > Fix pba_offset initialization value for Chelsio T5 devices. The > hardware doesn't return the correct pba_offset value, so add a > quirk to instead return a hardcoded value of 0x1000 when a Chelsio > T5 device is detected. Two questions: Is the array offset guaranteed to always be the same ? What are the chances of this getting fixed by a firmware update ? :) Bandan > Signed-off-by: Gabriel Laupre > --- > hw/vfio/pci.c | 12 ++++++++++++ > include/hw/pci/pci_ids.h | 3 +++ > 2 files changed, 15 insertions(+) > > diff --git a/hw/vfio/pci.c b/hw/vfio/pci.c > index e0e339a..8a4c7cd 100644 > --- a/hw/vfio/pci.c > +++ b/hw/vfio/pci.c > @@ -2220,6 +2220,9 @@ static int vfio_early_setup_msix(VFIOPCIDevice *vdev) > uint16_t ctrl; > uint32_t table, pba; > int fd = vdev->vbasedev.fd; > + PCIDevice *pdev = &vdev->pdev; > + uint16_t vendor = pci_get_word(pdev->config + PCI_VENDOR_ID); > + uint16_t device = pci_get_word(pdev->config + PCI_DEVICE_ID); > > pos = pci_find_capability(&vdev->pdev, PCI_CAP_ID_MSIX); > if (!pos) { > @@ -2252,6 +2255,15 @@ static int vfio_early_setup_msix(VFIOPCIDevice *vdev) > vdev->msix->pba_offset = pba & ~PCI_MSIX_FLAGS_BIRMASK; > vdev->msix->entries = (ctrl & PCI_MSIX_FLAGS_QSIZE) + 1; > > + /* Quirk to set the pba_offset value for Chelsio T5 > + * devices. Since hardware does not return value correctly, > + * we override with a hardcoded value instead. > + */ > + if (vendor == PCI_VENDOR_ID_CHELSIO && > + (device & 0xf000) == PCI_DEVICE_ID_CHELSIO_T5_SERIES) { > + vdev->msix->pba_offset = 0x1000; > + } > + > trace_vfio_early_setup_msix(vdev->vbasedev.name, pos, > vdev->msix->table_bar, > vdev->msix->table_offset, > diff --git a/include/hw/pci/pci_ids.h b/include/hw/pci/pci_ids.h > index 49c062b..9f649da 100644 > --- a/include/hw/pci/pci_ids.h > +++ b/include/hw/pci/pci_ids.h > @@ -114,6 +114,9 @@ > #define PCI_VENDOR_ID_ENSONIQ 0x1274 > #define PCI_DEVICE_ID_ENSONIQ_ES1370 0x5000 > > +#define PCI_VENDOR_ID_CHELSIO 0x1425 > +#define PCI_DEVICE_ID_CHELSIO_T5_SERIES 0x5000 > + > #define PCI_VENDOR_ID_FREESCALE 0x1957 > #define PCI_DEVICE_ID_MPC8533E 0x0030