diff options
| author | Alexey Kardashevskiy <aik@ozlabs.ru> | 2022-09-29 12:08:23 +1000 |
|---|---|---|
| committer | Masahiro Yamada <masahiroy@kernel.org> | 2022-10-03 03:52:58 +0900 |
| commit | 637a642f5ca5e850186bb64ac75ebb0f124b458d (patch) | |
| tree | 3081d6f333d3ff9421f337b80aace2f6f1aaa347 /scripts | |
| parent | d32b55f4bb43466bc6cdd98a00f8a600bbf7e8ec (diff) | |
| download | lwn-637a642f5ca5e850186bb64ac75ebb0f124b458d.tar.gz lwn-637a642f5ca5e850186bb64ac75ebb0f124b458d.zip | |
zstd: Fixing mixed module-builtin objects
With CONFIG_ZSTD_COMPRESS=m and CONFIG_ZSTD_DECOMPRESS=y we end up in
a situation when files from lib/zstd/common/ are compiled once to be
linked later for ZSTD_DECOMPRESS (build-in) and ZSTD_COMPRESS (module)
even though CFLAGS are different for builtins and modules.
So far somehow this was not a problem but enabling LLVM LTO exposes
the problem as:
ld.lld: error: linking module flags 'Code Model': IDs have conflicting values in 'lib/built-in.a(zstd_common.o at 5868)' and 'ld-temp.o'
This particular conflict is caused by KBUILD_CFLAGS=-mcmodel=medium vs.
KBUILD_CFLAGS_MODULE=-mcmodel=large , modules use the large model on
POWERPC as explained at
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/powerpc/Makefile?h=v5.18-rc4#n127
but the current use of common files is wrong anyway.
This works around the issue by introducing a zstd_common module with
shared code.
Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
Diffstat (limited to 'scripts')
0 files changed, 0 insertions, 0 deletions