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

rollback ffnvcodec to 12.1.14.1 #1040

Closed
wants to merge 1 commit into from

Conversation

ulmus-scott
Copy link
Contributor

This reduces the minimum Nvidia driver level to 530 from 550. Nvidia lists versions 535 and 550 for download for Linux.

Debian currently only packages 535 for Debian 12 and above.

Fixes #1039. This should also be applied to fixes/35

Checklist

@bennettpeter
Copy link
Member

bennettpeter commented Feb 19, 2025

I cannot recreate the error that the user had, see #1039. Playback is fine with nvidia 550. Do we really want to revert to older software?

@bennettpeter bennettpeter self-assigned this Feb 19, 2025
@ulmus-scott
Copy link
Contributor Author

https://www.mythtv.org/wiki/Coding_Standards lists Debian stable, which is currently 12, as supported, and Debian 12 only has a package for the Nvidia 535 driver.

When I updated ffnvcodec for FFmpeg 7.1, I didn't have a specific reason to use 12.2.72.1, just that it was the newest and I thought the 550 driver had been out for long enough that it would have been widely available, which apparently was incorrect. I only needed a different version since FFmpeg no longer accepted the 9.1.23.2 version and would disable NVDEC/NVENC completely otherwise.

12.2.72.1 lists 550 as the minimum driver version, so I would expect it to work. 12.1.14.1 lists 530 as the minimum driver version.

I don't think there is any significant difference for NVDEC between those versions, but, if 12.1.14.1 works with 535 and 12.2.72.1 doesn't, I think we should roll ffnvcodec back to 12.1.14.1.

ffnvcodec is just a copy of the Nvidia headers that FFmpeg needs to dynamically load the CUDA library for NVDEC and NVENC, so, since the newer drivers still accept the older interface, I don't think it is really "older software", just that it is allowing FFmpeg to work with older drivers.

@bennettpeter
Copy link
Member

OK I will test with 535 without and with the patch.

This reduces the minimum Nvidia driver level to 530 from 550.  Nvidia lists
versions 535 and 550 for download for Linux.
@ulmus-scott
Copy link
Contributor Author

I am closing this since #1039 has been resolved and this only effects NVENC, which is disabled by configure but enabled by cmake, pending discussion on the mailing list.

@ulmus-scott ulmus-scott deleted the ffnvcodec branch February 20, 2025 21:21
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.

Mythtv v35 fails decode with Nvidia NVDEC
2 participants