[PATCH v8 00/11] Unify <linux/export.h> and <asm/export.h>, remove EXPORT_DATA_SYMBOL(), faster TRIM_UNUSED_KSYMS
Masahiro Yamada
masahiroy at kernel.org
Sat Jun 10 02:13:09 PDT 2023
This patch set refactors modpost first to make it easier to
add new code.
My goals:
- Refactors EXPORT_SYMBOL, <linux/export.h> and <asm/export.h>.
You can still put EXPORT_SYMBOL() in *.S file, very close to the definition,
but you do not need to care about whether it is a function or a data.
This removes EXPORT_DATA_SYMBOL().
- Re-implement TRIM_UNUSED_KSYMS in one-pass.
This makes the building faster.
- Move the static EXPORT_SYMBOL check to modpost.
This also makes the building faster.
This patch set is applicable to linux-next 20230608.
V7 fixed build errors reported for Xtensa.
Previous version
v7: https://lore.kernel.org/linux-kbuild/20230608142428.256985-1-masahiroy@kernel.org/T/#mbaddcee18c9a8cf0a9b1f3fc562d09526cb69540
v6: https://lore.kernel.org/linux-kbuild/CAK7LNARjzGnj+sYX=_5yQ+8qoOQ2KB5N-_Ye53Ru3=XicezTYw@mail.gmail.com/T/#t
v5: https://lore.kernel.org/linux-kbuild/CAK7LNARBiOywrMLbR=9N35sk19U0QM3xcPy7d1WqV-eyb4W23w@mail.gmail.com/T/#t
v4: https://lore.kernel.org/linux-kbuild/CAK7LNASDzy9RERN6+q6WgR4ROYZQue=SBqgbcoYuVePByHtk6Q@mail.gmail.com/T/#t
v3: https://lore.kernel.org/all/20220928063947.299333-1-masahiroy@kernel.org/
Changes in v8:
- Fix the definition of ____EXPORT_SYMBOL() to v7.
It seems to cause a regression for xtensa. (reported by 0day bot)
Changes in v7:
- New patch
- Fix sparse warning reported by 0day bot
https://lore.kernel.org/linux-kbuild/202305280830.Rj5ltc9M-lkp@intel.com/
- Remove *.usyms
- New patch
Changes in v6:
- Fix build error on UML
- Make the symbol name in the warning more precise
Changes in v5:
- Fix build error on ARM
- Clean up more
Changes in v4:
- Version 3 did not work if a same name symbol exists in a different compilation unit
Fix it.
Changes in v3:
- Move struct kernel_symbol to kernel/module/internal.h
Changes in v2:
- Use KSYMTAB_FUNC and KSYMTAB_DATA for functions and data, respectively
This distinction is needed for ia64.
Masahiro Yamada (11):
Revert "[PATCH] uml: export symbols added by GCC hardened"
modpost: pass struct module pointer to check_section_mismatch()
kbuild: generate KSYMTAB entries by modpost
ia64,export.h: replace EXPORT_DATA_SYMBOL* with EXPORT_SYMBOL*
modpost: check static EXPORT_SYMBOL* by modpost again
modpost: squash sym_update_namespace() into sym_add_exported()
modpost: use null string instead of NULL pointer for default namespace
kbuild: implement CONFIG_TRIM_UNUSED_KSYMS without recursion
modpost: merge two similar section mismatch warnings
modpost: show offset from symbol for section mismatch warnings
linux/export.h: rename 'sec' argument to 'license'
.gitignore | 2 -
Makefile | 22 +--
arch/ia64/include/asm/Kbuild | 1 +
arch/ia64/include/asm/export.h | 3 -
arch/ia64/kernel/head.S | 2 +-
arch/ia64/kernel/ivt.S | 2 +-
arch/um/os-Linux/user_syms.c | 7 -
include/asm-generic/export.h | 83 +----------
include/asm-generic/vmlinux.lds.h | 1 +
include/linux/export-internal.h | 49 +++++++
include/linux/export.h | 126 ++++------------
include/linux/pm.h | 14 +-
kernel/module/internal.h | 12 ++
scripts/Makefile.build | 27 +---
scripts/Makefile.modpost | 7 +
scripts/adjust_autoksyms.sh | 73 ----------
scripts/basic/fixdep.c | 3 +-
scripts/check-local-export | 70 ---------
scripts/gen_autoksyms.sh | 62 --------
scripts/gen_ksymdeps.sh | 30 ----
scripts/mod/modpost.c | 233 +++++++++++++++++++-----------
scripts/mod/modpost.h | 1 +
scripts/remove-stale-files | 4 +
23 files changed, 268 insertions(+), 566 deletions(-)
delete mode 100644 arch/ia64/include/asm/export.h
delete mode 100755 scripts/adjust_autoksyms.sh
delete mode 100755 scripts/check-local-export
delete mode 100755 scripts/gen_autoksyms.sh
delete mode 100755 scripts/gen_ksymdeps.sh
--
2.39.2
More information about the linux-um
mailing list