-
Notifications
You must be signed in to change notification settings - Fork 165
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Switch to modules_install
or cleanup the module compression handling
#319
Comments
As far as I know, this is just historical stuff and has no particular reason to exist in its current form, it dates back to ~2002? I think the main reason was just supporting old EOL distributions and other OSes (Hurd). |
Ack, sounds like we could add a few more distros and/or versions in the mix and see how badly things explode. There's a good argument to support EOL stuff, yet within reason IMHO. For really ancient ones it looks like a disservice for ourselves and the respective users. |
@anbe42 you were involved with:
Any chance you can pick up this to drop the entire module compression stuff by switching to |
@anbe42 I've given you contributor access to DKMS. |
As mentioned in #318 we have a convoluted and buggy compressed/uncompressed module handling.
One option is to carefully audit and fix it up - the other to switch to
module_install
. The latter option will give us module stripping and signing for free.There might be some specific reason why dkms tries to do all of that itself instead of relying on the kernel build system - @scaronni if you have any pointed, or know people who might know please let me know.
The text was updated successfully, but these errors were encountered: