diff options
| author | Sam Ravnborg <sam@uranus.ravnborg.org> | 2008-04-25 20:13:30 +0200 |
|---|---|---|
| committer | Sam Ravnborg <sam@uranus.ravnborg.org> | 2008-04-25 20:13:30 +0200 |
| commit | eed7d2798ce7b78547106d0c3557a88251b413d0 (patch) | |
| tree | 2643c5c45f65c2a2743fc4b3f061ad7b2e37467b | |
| parent | b69d3987f4360a5e7e9e55465b3cdd0cc204b79e (diff) | |
| download | op-kernel-dev-eed7d2798ce7b78547106d0c3557a88251b413d0.zip op-kernel-dev-eed7d2798ce7b78547106d0c3557a88251b413d0.tar.gz | |
kbuild: error out on missing MODULE_LICENSE
Adrian Bunk suggested a build time check for
missing MODULE_LICENSE annotation in modules.
The build time check is fatal as we really
want this fixed for all modules.
In-tree modules should all have been fixed up by now.
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Cc: Adrian Bunk <bunk@kernel.org>
| -rw-r--r-- | scripts/mod/modpost.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c index 110cf24..843f6fa 100644 --- a/scripts/mod/modpost.c +++ b/scripts/mod/modpost.c @@ -1552,6 +1552,10 @@ static void read_symbols(char *modname) } license = get_modinfo(info.modinfo, info.modinfo_len, "license"); + if (!license && !is_vmlinux(modname)) + fatal("modpost: missing MODULE_LICENSE() in %s\n" + "see include/linux/module.h for " + "more information\n", modname); while (license) { if (license_is_gpl_compatible(license)) mod->gpl_compatible = 1; |
