[SRU][B][PATCH 1/8] UBUNTU: SAUCE: Revert "modules: inherit TAINT_PROPRIETARY_MODULE"
Ian May
ian.may at canonical.com
Thu Jun 17 20:22:18 UTC 2021
BugLink: https://bugs.launchpad.net/bugs/1932065
This reverts commit d4106f9a3fd5bd93307870a34c86a4976d5bb398.
Upstream introduced 'module' patches that removed exported symbols
that might cause potential disruption and breakage for customers.
Reverting as SAUCE patches to prevent regressions.
Signed-off-by: Ian May <ian.may at canonical.com>
---
include/linux/module.h | 1 -
kernel/module.c | 27 ---------------------------
2 files changed, 28 deletions(-)
diff --git a/include/linux/module.h b/include/linux/module.h
index f23024fdad71..7598c8112c71 100644
--- a/include/linux/module.h
+++ b/include/linux/module.h
@@ -357,7 +357,6 @@ struct module {
unsigned int num_gpl_syms;
const struct kernel_symbol *gpl_syms;
const s32 *gpl_crcs;
- bool using_gplonly_symbols;
#ifdef CONFIG_UNUSED_SYMBOLS
/* unused exported symbols. */
diff --git a/kernel/module.c b/kernel/module.c
index a906f338bd88..2876babeaf8a 100644
--- a/kernel/module.c
+++ b/kernel/module.c
@@ -1383,25 +1383,6 @@ static inline int same_magic(const char *amagic, const char *bmagic,
}
#endif /* CONFIG_MODVERSIONS */
-static bool inherit_taint(struct module *mod, struct module *owner)
-{
- if (!owner || !test_bit(TAINT_PROPRIETARY_MODULE, &owner->taints))
- return true;
-
- if (mod->using_gplonly_symbols) {
- pr_err("%s: module using GPL-only symbols uses symbols from proprietary module %s.\n",
- mod->name, owner->name);
- return false;
- }
-
- if (!test_bit(TAINT_PROPRIETARY_MODULE, &mod->taints)) {
- pr_warn("%s: module uses symbols from proprietary module %s, inheriting taint.\n",
- mod->name, owner->name);
- set_bit(TAINT_PROPRIETARY_MODULE, &mod->taints);
- }
- return true;
-}
-
/* Resolve a symbol for this module. I.e. if we find one, record usage. */
static const struct kernel_symbol *resolve_symbol(struct module *mod,
const struct load_info *info,
@@ -1426,14 +1407,6 @@ static const struct kernel_symbol *resolve_symbol(struct module *mod,
if (!sym)
goto unlock;
- if (license == GPL_ONLY)
- mod->using_gplonly_symbols = true;
-
- if (!inherit_taint(mod, owner)) {
- sym = NULL;
- goto getname;
- }
-
if (!check_version(info, name, mod, crc)) {
sym = ERR_PTR(-EINVAL);
goto getname;
--
2.25.1
More information about the kernel-team
mailing list