-
Notifications
You must be signed in to change notification settings - Fork 26
Enable Message Signaled Interrupts (MSI) #163
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
This fixes ridiculous interrupt rate on aarch64, was enabled in the legacy in-tree drm port, and is a good idea in general.
Just out of curiousity, where are pci_[enable,disable]_msi defined, and for which architecutures? |
They are currently not defined. Looks like a leftover from a porting effort. |
Then how is this supposed to work? It's just a change to code that was |
The kernel patch is linked in the first post |
@myfreeweb That's probably the piece I'm missing. Sorry for the confusion. |
I'd like to give @johalun the chance to comment, but I see no problems with this. |
Has this been tested on other than aarch64+amd? |
Of course I've tested on a Ryzen+Vega desktop and an i915 laptop |
This fixes ridiculous interrupt rate on aarch64, was enabled in the legacy in-tree drm port, and is a good idea in general.
This fixes ridiculous interrupt rate on aarch64, was enabled in the legacy in-tree drm port, and is a good idea in general.
@hselasky Also merged to v4.16 and v4.16-fbsd12 |
Enable Message Signaled Interrupts (MSI) (FreeBSDDesktop#163)
This fixes ridiculous interrupt rate on aarch64, was enabled in the legacy in-tree drm port, and is a good idea in general.
This fixes ridiculous interrupt rate on aarch64, was enabled in the legacy in-tree drm port, and is a good idea in general.
Corresponding kernel patch: https://reviews.freebsd.org/D21008 but this shouldn't break the build without it, thanks to
if defined(pci_enable_msi)
.Tested on amdgpu and i915kms.