Skip to content

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