Skip to content

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

Merged
merged 1 commit into from
Aug 7, 2019

Conversation

valpackett
Copy link
Contributor

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.

@hselasky
Copy link
Contributor

hselasky commented Aug 7, 2019

@johalun , @zeising --- This patch needs to go to FreeBSD-current and FreeBSD-12 DRM next before we can merge the kernel part! Will you handle.

This fixes ridiculous interrupt rate on aarch64,
was enabled in the legacy in-tree drm port,
and is a good idea in general.
@zeising
Copy link
Member

zeising commented Aug 7, 2019

Just out of curiousity, where are pci_[enable,disable]_msi defined, and for which architecutures?

@hselasky
Copy link
Contributor

hselasky commented Aug 7, 2019

They are currently not defined. Looks like a leftover from a porting effort.

@zeising
Copy link
Member

zeising commented Aug 7, 2019

Then how is this supposed to work? It's just a change to code that was
#ifdef __linux__ is now #if defined(__linux__) || defined (pci_[enable,disable]_msi
Or is there an upcoming change to define pci_[enable,disable]_msi?

@valpackett
Copy link
Contributor Author

The kernel patch is linked in the first post

@zeising
Copy link
Member

zeising commented Aug 7, 2019

@myfreeweb That's probably the piece I'm missing. Sorry for the confusion.
@myfreeweb @hselasky Thanks for explaining to me. :)

@zeising
Copy link
Member

zeising commented Aug 7, 2019

I'd like to give @johalun the chance to comment, but I see no problems with this.

@johalun
Copy link
Member

johalun commented Aug 7, 2019

Has this been tested on other than aarch64+amd?

@valpackett
Copy link
Contributor Author

Of course I've tested on a Ryzen+Vega desktop and an i915 laptop

@johalun johalun merged commit 0e30c81 into FreeBSDDesktop:drm-v5.0 Aug 7, 2019
johalun pushed a commit that referenced this pull request Aug 7, 2019
This fixes ridiculous interrupt rate on aarch64,
was enabled in the legacy in-tree drm port,
and is a good idea in general.
johalun pushed a commit that referenced this pull request Aug 7, 2019
This fixes ridiculous interrupt rate on aarch64,
was enabled in the legacy in-tree drm port,
and is a good idea in general.
@johalun
Copy link
Member

johalun commented Aug 7, 2019

@hselasky Also merged to v4.16 and v4.16-fbsd12

fullermd added a commit to fullermd/kms-drm that referenced this pull request Nov 30, 2019
  Enable Message Signaled Interrupts (MSI) (FreeBSDDesktop#163)
evadot pushed a commit to evadot/kms-drm that referenced this pull request May 5, 2020
This fixes ridiculous interrupt rate on aarch64,
was enabled in the legacy in-tree drm port,
and is a good idea in general.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants