zstd: Fixing mixed module-builtin objects
authorAlexey Kardashevskiy <aik@ozlabs.ru>
Thu, 29 Sep 2022 02:08:23 +0000 (12:08 +1000)
committerMasahiro Yamada <masahiroy@kernel.org>
Sun, 2 Oct 2022 18:52:58 +0000 (03:52 +0900)
commit637a642f5ca5e850186bb64ac75ebb0f124b458d
tree3081d6f333d3ff9421f337b80aace2f6f1aaa347
parentd32b55f4bb43466bc6cdd98a00f8a600bbf7e8ec
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>
lib/Kconfig
lib/zstd/Makefile
lib/zstd/common/entropy_common.c
lib/zstd/common/zstd_common.c