-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Use known license string for zlua #11113
Conversation
The Linux kernel MODULE_LICENSE macro only recognizes a handful of license strings and "MIT" is not one of the them. Update the macro to use "Dual MIT/GPL" which is recognized and what the kernel expects MIT licensed modules to use. Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov> Issue openzfs#11112
Codecov Report
@@ Coverage Diff @@
## master #11113 +/- ##
==========================================
- Coverage 79.56% 79.50% -0.06%
==========================================
Files 398 398
Lines 125754 125754
==========================================
- Hits 100060 99986 -74
- Misses 25694 25768 +74
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
I had to modify this commit to merge into 0.8.5-1 on my system (module macros aren't wrapped with "ZFS_" on the 0.8.y branch), but this did the trick. I'm able to run 0.8.5-1 on Ubuntu 20.04 ARM64 without errors now. Any chance we could backport this to 0.8.6 (which supports 5.9) or even a 0.8.5-2 (if it's intended to support 5.9)? |
Yup, we'll get it back ported for the next point release. |
The Linux kernel MODULE_LICENSE macro only recognizes a handful of license strings and "MIT" is not one of the them. Update the macro to use "Dual MIT/GPL" which is recognized and what the kernel expects MIT licensed modules to use. Reviewed-by: George Melikov <mail@gmelikov.ru> Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov> Closes openzfs#11112 Closes openzfs#11113
The Linux kernel MODULE_LICENSE macro only recognizes a handful of license strings and "MIT" is not one of the them. Update the macro to use "Dual MIT/GPL" which is recognized and what the kernel expects MIT licensed modules to use. Reviewed-by: George Melikov <mail@gmelikov.ru> Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov> Closes #11112 Closes #11113
The Linux kernel MODULE_LICENSE macro only recognizes a handful of license strings and "MIT" is not one of the them. Update the macro to use "Dual MIT/GPL" which is recognized and what the kernel expects MIT licensed modules to use. Reviewed-by: George Melikov <mail@gmelikov.ru> Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov> Closes openzfs#11112 Closes openzfs#11113
The Linux kernel MODULE_LICENSE macro only recognizes a handful of license strings and "MIT" is not one of the them. Update the macro to use "Dual MIT/GPL" which is recognized and what the kernel expects MIT licensed modules to use. Reviewed-by: George Melikov <mail@gmelikov.ru> Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov> Closes openzfs#11112 Closes openzfs#11113
Motivation and Context
Issue #11112
Description
The Linux kernel MODULE_LICENSE macro only recognizes a handful of
license strings and "MIT" is not one of the them. Update the macro
to use "Dual MIT/GPL" which is recognized and what the kernel expects
MIT licensed modules to use.
How Has This Been Tested?
Manually loaded and inspected.
Types of changes
Checklist:
Signed-off-by
.