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

PRIME: Remove libva-nvidia-driver from installable packages #132

Merged
merged 1 commit into from
Aug 19, 2024

Conversation

1Naim
Copy link
Member

@1Naim 1Naim commented Aug 19, 2024

On hybrid systems (only AMD so far), the nvidia-vaapi driver can cause issues with a few software, namely Spectacle. When this is installed on the system, it will be the first driver picked for encoding. There are no issues with this on first launch. However subsequent launches will not reinitalize the VAAPI driver, making Spectacle have a "lag spike" when taking region screenshots.

Since the environmental variable to force the use of nvidia-vaapi has been removed, I think this is a reasonable fix. Users who want nvidia-vaapi can grab the package manually.

On hybrid systems (only AMD so far), the nvidia-vaapi driver can cause issues with a few software,
namely Spectacle. When this is installed on the system, it will be the first driver picked for encoding.
There are no issues with this on first launch. However subsequent launches will **not** reinitalize
the VAAPI driver, making Spectacle have a "lag spike" when taking region screenshots.

Since the environmental variable to force the use of nvidia-vaapi has been removed, I think this is
a reasonable fix. Users who want nvidia-vaapi can grab the package manually.

Signed-off-by: Eric Naim <dnaim@cachyos.org>
@1Naim 1Naim requested a review from ventureoo August 19, 2024 07:53
@ventureoo ventureoo merged commit cfde726 into master Aug 19, 2024
2 checks passed
@ventureoo ventureoo deleted the fix/hybrid-vaapi branch August 19, 2024 09:43
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 participants