-
-
Notifications
You must be signed in to change notification settings - Fork 36
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
Add support for NVIDIA module build #167
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM so far
Thanks! This seems good. If the testing goes well, we can can build them with 6.5.2 into the repository and could make an announcement in the community, that they test it also. Lets see how it goes! :) |
a question as a user: when you enable _build_nvidia, do you use "updpkgsums" to update the checksums and change PKGBUILD? Or --skipchecksums? |
You can use any. When we build the kernel, we have an environment variable to automatically generate it in our script. |
@ventureoo |
Done |
Would it be useful to set _nv_ver=535.104.05 by variable? e.g. |
Sure ,we can do this. |
Is possible to add Vulkan Dev as an option? E.g.: _build_nvidia_vulkan_dev=${_build_nvidia_vulkan_dev-y} i suppose that u need to add this driver branch to your repos, right ? Thanks in advance :-) |
I really think, we should not bloat this to much, since it would also required to change the depend on the nvidia-vulkan-utils. |
Similar to ZFS, this allows us to build an NVIDIA driver module already purpose-built for our kernel. This will allow us to save NVIDIA driver users from constant rebuilding of dkms modules, which is often accompanied by a number of errors.
Note that in PKGBUILD, we will probably need to update
_nv_ver
whenever we update the driver itself, so that there is no mismatch in the versions of modules and nvidia-utils in our repository.Also, note that I did not test the build together with LLVM+LTO, so this may require additional testing/fixes.