mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-12 13:55:32 +00:00
![Quentin Perret](/assets/img/avatar_default.png)
The CRC calculation done by genksyms is triggered when the parser hits EXPORT_SYMBOL*() macros. At this point, genksyms recursively expands the types, and uses that as the input for the CRC calculation. In the case of forward-declared structs, the type expands to 'UNKNOWN'. Next, the result of the expansion of each type is cached, and is re-used when/if the same type is seen again for another exported symbol in the file. Unfortunately, this can cause CRC 'stability' issues when a struct definition becomes visible in the middle of a C file. For example, let's assume code with the following pattern: struct foo; int bar(struct foo *arg) { /* Do work ... */ } EXPORT_SYMBOL_GPL(bar); /* This contains struct foo's definition */ #include "foo.h" int baz(struct foo *arg) { /* Do more work ... */ } EXPORT_SYMBOL_GPL(baz); Here, baz's CRC will be computed using the expansion of struct foo that was cached after bar's CRC calculation ('UNKOWN' here). But if EXPORT_SYMBOL_GPL(bar) is removed from the file (because of e.g. symbol trimming using CONFIG_TRIM_UNUSED_KSYMS), struct foo will be expanded late, during baz's CRC calculation, which now has visibility over the full struct definition, hence resulting in a different CRC for baz. This can cause annoying issues for distro kernel (such as the Android Generic Kernel Image) which use CONFIG_UNUSED_KSYMS_WHITELIST. Indeed, as per the above, adding a symbol to the whitelist can change the CRC of symbols that are already kept exported. As such, modules built against a kernel with a trimmed ABI may not load against the same kernel built with an extended whitelist, even though they are still strictly binary compatible. While rebuilding the modules would obviously solve the issue, I believe this classifies as an odd genksyms corner case, and it gets in the way of kernel updates in the GKI context. To work around the issue, make sure to keep issuing the __GENKSYMS_EXPORT_SYMBOL macros for all trimmed symbols, hence making the genksyms parsing insensitive to symbol trimming. Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Quentin Perret <qperret@google.com> Link: https://lore.kernel.org/r/20210408180105.2496212-1-qperret@google.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
170 lines
5.2 KiB
C
170 lines
5.2 KiB
C
/* SPDX-License-Identifier: GPL-2.0-only */
|
|
#ifndef _LINUX_EXPORT_H
|
|
#define _LINUX_EXPORT_H
|
|
|
|
/*
|
|
* Export symbols from the kernel to modules. Forked from module.h
|
|
* to reduce the amount of pointless cruft we feed to gcc when only
|
|
* exporting a simple symbol or two.
|
|
*
|
|
* Try not to add #includes here. It slows compilation and makes kernel
|
|
* hackers place grumpy comments in header files.
|
|
*/
|
|
|
|
#ifndef __ASSEMBLY__
|
|
#ifdef MODULE
|
|
extern struct module __this_module;
|
|
#define THIS_MODULE (&__this_module)
|
|
#else
|
|
#define THIS_MODULE ((struct module *)0)
|
|
#endif
|
|
|
|
#ifdef CONFIG_MODVERSIONS
|
|
/* Mark the CRC weak since genksyms apparently decides not to
|
|
* generate a checksums for some symbols */
|
|
#if defined(CONFIG_MODULE_REL_CRCS)
|
|
#define __CRC_SYMBOL(sym, sec) \
|
|
asm(" .section \"___kcrctab" sec "+" #sym "\", \"a\" \n" \
|
|
" .weak __crc_" #sym " \n" \
|
|
" .long __crc_" #sym " - . \n" \
|
|
" .previous \n")
|
|
#else
|
|
#define __CRC_SYMBOL(sym, sec) \
|
|
asm(" .section \"___kcrctab" sec "+" #sym "\", \"a\" \n" \
|
|
" .weak __crc_" #sym " \n" \
|
|
" .long __crc_" #sym " \n" \
|
|
" .previous \n")
|
|
#endif
|
|
#else
|
|
#define __CRC_SYMBOL(sym, sec)
|
|
#endif
|
|
|
|
#ifdef CONFIG_HAVE_ARCH_PREL32_RELOCATIONS
|
|
#include <linux/compiler.h>
|
|
/*
|
|
* Emit the ksymtab entry as a pair of relative references: this reduces
|
|
* the size by half on 64-bit architectures, and eliminates the need for
|
|
* absolute relocations that require runtime processing on relocatable
|
|
* kernels.
|
|
*/
|
|
#define __KSYMTAB_ENTRY(sym, sec) \
|
|
__ADDRESSABLE(sym) \
|
|
asm(" .section \"___ksymtab" sec "+" #sym "\", \"a\" \n" \
|
|
" .balign 4 \n" \
|
|
"__ksymtab_" #sym ": \n" \
|
|
" .long " #sym "- . \n" \
|
|
" .long __kstrtab_" #sym "- . \n" \
|
|
" .long __kstrtabns_" #sym "- . \n" \
|
|
" .previous \n")
|
|
|
|
struct kernel_symbol {
|
|
int value_offset;
|
|
int name_offset;
|
|
int namespace_offset;
|
|
};
|
|
#else
|
|
#define __KSYMTAB_ENTRY(sym, sec) \
|
|
static const struct kernel_symbol __ksymtab_##sym \
|
|
__attribute__((section("___ksymtab" sec "+" #sym), used)) \
|
|
__aligned(sizeof(void *)) \
|
|
= { (unsigned long)&sym, __kstrtab_##sym, __kstrtabns_##sym }
|
|
|
|
struct kernel_symbol {
|
|
unsigned long value;
|
|
const char *name;
|
|
const char *namespace;
|
|
};
|
|
#endif
|
|
|
|
#ifdef __GENKSYMS__
|
|
|
|
#define ___EXPORT_SYMBOL(sym, sec, ns) __GENKSYMS_EXPORT_SYMBOL(sym)
|
|
|
|
#else
|
|
|
|
/*
|
|
* For every exported symbol, do the following:
|
|
*
|
|
* - If applicable, place a CRC entry in the __kcrctab section.
|
|
* - Put the name of the symbol and namespace (empty string "" for none) in
|
|
* __ksymtab_strings.
|
|
* - Place a struct kernel_symbol entry in the __ksymtab section.
|
|
*
|
|
* note on .section use: we specify progbits since usage of the "M" (SHF_MERGE)
|
|
* section flag requires it. Use '%progbits' instead of '@progbits' since the
|
|
* former apparently works on all arches according to the binutils source.
|
|
*/
|
|
#define ___EXPORT_SYMBOL(sym, sec, ns) \
|
|
extern typeof(sym) sym; \
|
|
extern const char __kstrtab_##sym[]; \
|
|
extern const char __kstrtabns_##sym[]; \
|
|
__CRC_SYMBOL(sym, sec); \
|
|
asm(" .section \"__ksymtab_strings\",\"aMS\",%progbits,1 \n" \
|
|
"__kstrtab_" #sym ": \n" \
|
|
" .asciz \"" #sym "\" \n" \
|
|
"__kstrtabns_" #sym ": \n" \
|
|
" .asciz \"" ns "\" \n" \
|
|
" .previous \n"); \
|
|
__KSYMTAB_ENTRY(sym, sec)
|
|
|
|
#endif
|
|
|
|
#if !defined(CONFIG_MODULES) || defined(__DISABLE_EXPORTS)
|
|
|
|
/*
|
|
* Allow symbol exports to be disabled completely so that C code may
|
|
* be reused in other execution contexts such as the UEFI stub or the
|
|
* decompressor.
|
|
*/
|
|
#define __EXPORT_SYMBOL(sym, sec, ns)
|
|
|
|
#elif defined(CONFIG_TRIM_UNUSED_KSYMS)
|
|
|
|
#include <generated/autoksyms.h>
|
|
|
|
/*
|
|
* For fine grained build dependencies, we want to tell the build system
|
|
* about each possible exported symbol even if they're not actually exported.
|
|
* We use a symbol pattern __ksym_marker_<symbol> that the build system filters
|
|
* from the $(NM) output (see scripts/gen_ksymdeps.sh). These symbols are
|
|
* discarded in the final link stage.
|
|
*/
|
|
#define __ksym_marker(sym) \
|
|
static int __ksym_marker_##sym[0] __section(".discard.ksym") __used
|
|
|
|
#define __EXPORT_SYMBOL(sym, sec, ns) \
|
|
__ksym_marker(sym); \
|
|
__cond_export_sym(sym, sec, ns, __is_defined(__KSYM_##sym))
|
|
#define __cond_export_sym(sym, sec, ns, conf) \
|
|
___cond_export_sym(sym, sec, ns, conf)
|
|
#define ___cond_export_sym(sym, sec, ns, enabled) \
|
|
__cond_export_sym_##enabled(sym, sec, ns)
|
|
#define __cond_export_sym_1(sym, sec, ns) ___EXPORT_SYMBOL(sym, sec, ns)
|
|
|
|
#ifdef __GENKSYMS__
|
|
#define __cond_export_sym_0(sym, sec, ns) __GENKSYMS_EXPORT_SYMBOL(sym)
|
|
#else
|
|
#define __cond_export_sym_0(sym, sec, ns) /* nothing */
|
|
#endif
|
|
|
|
#else
|
|
|
|
#define __EXPORT_SYMBOL(sym, sec, ns) ___EXPORT_SYMBOL(sym, sec, ns)
|
|
|
|
#endif /* CONFIG_MODULES */
|
|
|
|
#ifdef DEFAULT_SYMBOL_NAMESPACE
|
|
#include <linux/stringify.h>
|
|
#define _EXPORT_SYMBOL(sym, sec) __EXPORT_SYMBOL(sym, sec, __stringify(DEFAULT_SYMBOL_NAMESPACE))
|
|
#else
|
|
#define _EXPORT_SYMBOL(sym, sec) __EXPORT_SYMBOL(sym, sec, "")
|
|
#endif
|
|
|
|
#define EXPORT_SYMBOL(sym) _EXPORT_SYMBOL(sym, "")
|
|
#define EXPORT_SYMBOL_GPL(sym) _EXPORT_SYMBOL(sym, "_gpl")
|
|
#define EXPORT_SYMBOL_NS(sym, ns) __EXPORT_SYMBOL(sym, "", #ns)
|
|
#define EXPORT_SYMBOL_NS_GPL(sym, ns) __EXPORT_SYMBOL(sym, "_gpl", #ns)
|
|
|
|
#endif /* !__ASSEMBLY__ */
|
|
|
|
#endif /* _LINUX_EXPORT_H */
|