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.

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

Fixes #2262 and GFX rewrite #2266

Merged
merged 3 commits into from
Nov 27, 2023
Merged

Fixes #2262 and GFX rewrite #2266

merged 3 commits into from
Nov 27, 2023

Conversation

svartkanin
Copy link
Collaborator

This fixes #2262 and rewrites the GFX installation logic to something (hopefully) simpler

IntelMediaDriver = 'intel-media-driver'
LibvaIntelDriver = 'libva-intel-driver'
LibvaMesaDriver = 'libva-mesa-driver'
Mesa = "mesa"
Nvidia = 'nvidia'
Copy link
Member

Choose a reason for hiding this comment

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

Won't this make it so nvidia package is not installed? Which is needed for some platforms? Or we rely solely on the -dkms and add the nvidia hooks to build the driver each kernel upgrade?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Maybe I missed something but in the old code Nvidia didn't seem to be installed at all. But please double check that I didn't miss anything

I've actually tried installing nvidia and nvidia-dkms which threw a conflict error

@Torxed
Copy link
Member

Torxed commented Nov 27, 2023

This could solve:

So installing nvidia-dkms instead of nvidia should work, the downside being that every upgrade will be slower as the kernel modules needs to be compiled during installation (this is done automatically AFAIK).

I'll merge this for now, but we should work on a fix to:

  • Install nvidia if only linux kernel is selected (And also install linux-headers)
  • Install nvidia-lts if only linux-lts kernel is selected (Also linux-lts-headers)
  • Install nvidia-dkms if combintions of the above or none of the above was selected

That is the supported packaging from the package maintainers. But -dkms should work on all of the above - just slower upgrades.

@Torxed Torxed merged commit 5605958 into archlinux:master Nov 27, 2023
6 of 7 checks passed
@svartkanin svartkanin deleted the fix-2262 branch November 29, 2023 08:17
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.

[2.7.0- Master GIT] - No headers installed, nvidia's modules not built
2 participants