summaryrefslogtreecommitdiffstats
path: root/kernel/module.c
diff options
context:
space:
mode:
authorJoe Mario <jmario@redhat.com>2014-02-08 09:01:09 +0100
committerH. Peter Anvin <hpa@linux.intel.com>2014-02-13 20:24:50 -0800
commit80375980f1608f43b47abc2671456b23ec68c434 (patch)
tree305be1bb13f3535fffed254d5af46402b5d275af /kernel/module.c
parentef1b893c29d0dba778f67ad97b554b37f9108dcc (diff)
downloadlinux-80375980f1608f43b47abc2671456b23ec68c434.tar.gz
linux-80375980f1608f43b47abc2671456b23ec68c434.tar.bz2
linux-80375980f1608f43b47abc2671456b23ec68c434.zip
lto: Handle LTO common symbols in module loader
Here is the workaround I made for having the kernel not reject modules built with -flto. The clean solution would be to get the compiler to not emit the symbol. Or if it has to emit the symbol, then emit it as initialized data but put it into a comdat/linkonce section. Minor tweaks by AK over Joe's patch. Cc: Rusty Russell <rusty@rustcorp.com.au> Signed-off-by: Andi Kleen <ak@linux.intel.com> Link: http://lkml.kernel.org/r/1391846481-31491-5-git-send-email-ak@linux.intel.com Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>
Diffstat (limited to 'kernel/module.c')
-rw-r--r--kernel/module.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/kernel/module.c b/kernel/module.c
index d24fcf29cb64..b99e80119eef 100644
--- a/kernel/module.c
+++ b/kernel/module.c
@@ -1948,6 +1948,10 @@ static int simplify_symbols(struct module *mod, const struct load_info *info)
switch (sym[i].st_shndx) {
case SHN_COMMON:
+ /* Ignore common symbols */
+ if (!strncmp(name, "__gnu_lto", 9))
+ break;
+
/* We compiled with -fno-common. These are not
supposed to happen. */
pr_debug("Common symbol: %s\n", name);