Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Support Intel GPUs before Gen-6 (patch from upstream) #960

Merged
merged 1 commit into from
Nov 8, 2024
Merged

Support Intel GPUs before Gen-6 (patch from upstream) #960

merged 1 commit into from
Nov 8, 2024

Conversation

w8jcik
Copy link
Contributor

@w8jcik w8jcik commented Nov 3, 2024 •
edited
Loading

From https://gitlab.freedesktop.org/drm/igt-gpu-tools/-/commit/c727cc3fad56f16f1a9f35023ae5dd7111976fa3.

With this change intel_gpu_top works with Intel GMA chips instead of terminating with Failed to initialize PMU.

When it comes to btop:

  • Without this change my Gen-4 Intel GPU (GMA 4500MHD) is not visible in btop.
  • With the change my both GPUs (Gen-4 and Gen-9 on two different PCs) are visible in btop.

@w8jcik w8jcik changed the title Support intel GPUs before Gen-6 (patch from upstream) Support Intel GPUs before Gen-6 (patch from upstream) Nov 3, 2024
aristocratos
aristocratos approved these changes Nov 8, 2024
View reviewed changes
Copy link
Owner

@aristocratos aristocratos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thanks for contributing!

@aristocratos aristocratos merged commit a049e95 into aristocratos:main Nov 8, 2024
44 checks passed
@w8jcik w8jcik deleted the fix/pre-snb-intel-gpu branch November 8, 2024 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aristocratos aristocratos aristocratos approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@w8jcik @aristocratos