diff options
| author | Sam Ravnborg <sam@mars.ravnborg.org> | 2006-06-09 21:53:55 +0200 |
|---|---|---|
| committer | Sam Ravnborg <sam@mars.ravnborg.org> | 2006-06-09 21:53:55 +0200 |
| commit | b817f6feff4a565b08f0e699a5790b4008b8f494 (patch) | |
| tree | b33ca0f4477c500380bc409cc272ab7f207ed77b /kernel | |
| parent | bd5cbcedf446e2f37cf2a37f533a7e1d7dff9312 (diff) | |
| download | lwn-b817f6feff4a565b08f0e699a5790b4008b8f494.tar.gz lwn-b817f6feff4a565b08f0e699a5790b4008b8f494.zip | |
kbuild: check license compatibility when building modules
Modules that uses GPL symbols can no longer be build with kbuild,
the build will fail during the modpost step.
When a GPL-incompatible module uses a EXPORT_SYMBOL_GPL_FUTURE symbol
then warn during modpost so author are actually notified.
The actual license compatibility check is shared with the kernel
to make sure it is in sync.
Patch originally from: Andreas Gruenbacher <agruen@suse.de> and
Ram Pai <linuxram@us.ibm.com>
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/module.c | 11 |
1 files changed, 1 insertions, 10 deletions
diff --git a/kernel/module.c b/kernel/module.c index bbe04862e1b0..690381508d09 100644 --- a/kernel/module.c +++ b/kernel/module.c @@ -43,6 +43,7 @@ #include <asm/uaccess.h> #include <asm/semaphore.h> #include <asm/cacheflush.h> +#include <linux/license.h> #if 0 #define DEBUGP printk @@ -1248,16 +1249,6 @@ static void layout_sections(struct module *mod, } } -static inline int license_is_gpl_compatible(const char *license) -{ - return (strcmp(license, "GPL") == 0 - || strcmp(license, "GPL v2") == 0 - || strcmp(license, "GPL and additional rights") == 0 - || strcmp(license, "Dual BSD/GPL") == 0 - || strcmp(license, "Dual MIT/GPL") == 0 - || strcmp(license, "Dual MPL/GPL") == 0); -} - static void set_license(struct module *mod, const char *license) { if (!license) |