mirror/freebsd
1
0
Fork 0
mirror of https://git.FreeBSD.org/src.git synced 2025-01-03 12:35:02 +00:00
Code Issues Releases Activity

- Fix an off by one error in pci_remap_msix_method() that effectively

Browse Source 
broke the method as all the MSI-X table indices were off by one in
  the backend MD code.
- Fix a cosmetic nit in the bootverbose printf in pci_alloc_msix_method().
This commit is contained in:
John Baldwin 2007-02-14 22:32:55 +00:00
parent 4ccac64eed
commit ea3f508362
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=166722
1 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
sys/dev/pci/pci.c
View File

@ -1142,7 +1142,7 @@ pci_alloc_msix_method(device_t dev, device_t child, int *count)
/* Unfinished range? */ /* Unfinished range? */
if (run) if (run)
printf("%d", irq); printf("-%d", irq);
printf(" for MSI-X\n"); printf(" for MSI-X\n");
} }
} }
@ -1221,8 +1221,16 @@ pci_remap_msix_method(device_t dev, device_t child, u_int *indices)
for (i = 0; i < cfg->msix.msix_alloc; i++) { for (i = 0; i < cfg->msix.msix_alloc; i++) {
resource_list_add(&dinfo->resources, SYS_RES_IRQ, indices[i], resource_list_add(&dinfo->resources, SYS_RES_IRQ, indices[i],
irqs[i], irqs[i], 1); irqs[i], irqs[i], 1);
/*
* The indices in the backend code (PCIB_* methods and the
* MI helper routines for MD code such as pci_enable_msix())
* are all zero-based. However, the indices passed to this
* function are 1-based so that the correspond 1:1 with the
* SYS_RES_IRQ resource IDs.
*/
error = PCIB_REMAP_MSIX(device_get_parent(dev), child, error = PCIB_REMAP_MSIX(device_get_parent(dev), child,
indices[i], irqs[i]); indices[i] - 1, irqs[i]);
KASSERT(error == 0, ("Failed to remap MSI-X message")); KASSERT(error == 0, ("Failed to remap MSI-X message"));
} }
if (bootverbose) { if (bootverbose) {