Page MenuHomeFreeBSD

pci: make MSI(-X) enable and disable methods of the PCI bus
ClosedPublic

Authored by royger on Jul 4 2014, 3:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 24 2024, 3:08 AM
Unknown Object (File)
Feb 16 2024, 6:26 AM
Unknown Object (File)
Feb 16 2024, 5:26 AM
Unknown Object (File)
Jan 19 2024, 5:25 AM
Unknown Object (File)
Jan 8 2024, 12:49 PM
Unknown Object (File)
Dec 25 2023, 5:20 AM
Unknown Object (File)
Dec 19 2023, 9:53 PM
Unknown Object (File)
Nov 28 2023, 7:38 AM
Subscribers
None

Details

Summary

Make the functions pci_disable_msi, pci_enable_msi and pci_enable_msix
methods of the newbus PCI bus. This code should not include any
functional change.

Sponsored by: Citrix Systems R&D

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

royger retitled this revision from to pci: make MSI(-X) enable and disable methods of the PCI bus.
royger updated this object.
royger edited the test plan for this revision. (Show Details)
royger added reviewers: jhb, imp.
imp edited edge metadata.

Looks good to me.

This revision is now accepted and ready to land.Jul 6 2014, 5:30 PM
jhb edited edge metadata.

Looks fine to me.

royger edited edge metadata.

Update to prevent the newly introduced pci_enable_msix clashing with
the Linux version defined in ofed/include/linux/pci.h.

royger requested a review of this revision.Aug 4 2014, 9:12 AM
royger added a reviewer: alfred.

Adding people who worked in ofed in order to review the new bits, I would also like to add hselasky but he doesn't seem to have a phabric account.

Alfred, could you take a look at the ofed bits? I'm introducing a native FreeBSD version of pci_enable_msix which clashes with the Linux compatibility one defined in sys/ofed/include/linux/pci.h.

sys/ofed/include/linux/pci.h
539

One alternate thing you could try would be to add a #define before this function definition that was:

#define pci_enable_msix linux_pci_enable_msix

and then revert the the rest of the ofed related changes. If that works, it might be slightly preferable as it doesn't require modifications in drivers that are using the shim layer, albeit being a bit more gross / fragile.

royger added a reviewer: royger.
This revision is now accepted and ready to land.Aug 20 2014, 3:26 PM

Thanks, committed as r270212.