diff options
author | Joe Mario <jmario@redhat.com> | 2014-02-08 03:01:09 -0500 |
---|---|---|
committer | H. Peter Anvin <hpa@linux.intel.com> | 2014-02-13 23:24:50 -0500 |
commit | 80375980f1608f43b47abc2671456b23ec68c434 (patch) | |
tree | 305be1bb13f3535fffed254d5af46402b5d275af /kernel/module.c | |
parent | ef1b893c29d0dba778f67ad97b554b37f9108dcc (diff) |
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.c | 4 |
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) | |||
1948 | 1948 | ||
1949 | switch (sym[i].st_shndx) { | 1949 | switch (sym[i].st_shndx) { |
1950 | case SHN_COMMON: | 1950 | case SHN_COMMON: |
1951 | /* Ignore common symbols */ | ||
1952 | if (!strncmp(name, "__gnu_lto", 9)) | ||
1953 | break; | ||
1954 | |||
1951 | /* We compiled with -fno-common. These are not | 1955 | /* We compiled with -fno-common. These are not |
1952 | supposed to happen. */ | 1956 | supposed to happen. */ |
1953 | pr_debug("Common symbol: %s\n", name); | 1957 | pr_debug("Common symbol: %s\n", name); |