btf: do not use CONFIG_OUTPUT_FORMAT
authorIlya Leoshkevich <iii@linux.ibm.com>
Tue, 20 Aug 2019 11:29:39 +0000 (13:29 +0200)
committerDaniel Borkmann <daniel@iogearbox.net>
Wed, 21 Aug 2019 12:57:17 +0000 (14:57 +0200)
Building s390 kernel with CONFIG_DEBUG_INFO_BTF fails, because
CONFIG_OUTPUT_FORMAT is not defined. As a matter of fact, this variable
appears to be x86-only, so other arches might be affected as well.

Fix by obtaining this value from objdump output, just like it's already
done for bin_arch. The exact objdump invocation is "inspired" by
arch/powerpc/boot/wrapper.

Also, use LANG=C for the existing bin_arch objdump invocation to avoid
potential build issues on systems with non-English locale.

Fixes: 341dfcf8d78e ("btf: expose BTF info through sysfs")
Signed-off-by: Ilya Leoshkevich <iii@linux.ibm.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
scripts/link-vmlinux.sh

index c311933401081fd48b5451605ff3a4a11f4893da..0d8f41db8cd6e04a3fce6c6cfe9fcb300e6a4500 100755 (executable)
@@ -115,10 +115,12 @@ gen_btf()
        LLVM_OBJCOPY=${OBJCOPY} ${PAHOLE} -J ${1}
 
        # dump .BTF section into raw binary file to link with final vmlinux
-       bin_arch=$(${OBJDUMP} -f ${1} | grep architecture | \
+       bin_arch=$(LANG=C ${OBJDUMP} -f ${1} | grep architecture | \
                cut -d, -f1 | cut -d' ' -f2)
+       bin_format=$(LANG=C ${OBJDUMP} -f ${1} | grep 'file format' | \
+               awk '{print $4}')
        ${OBJCOPY} --dump-section .BTF=.btf.vmlinux.bin ${1} 2>/dev/null
-       ${OBJCOPY} -I binary -O ${CONFIG_OUTPUT_FORMAT} -B ${bin_arch} \
+       ${OBJCOPY} -I binary -O ${bin_format} -B ${bin_arch} \
                --rename-section .data=.BTF .btf.vmlinux.bin ${2}
 }