linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [next] [clang] s390: clang: error: unable to execute command: Segmentation fault (core dumped)
@ 2021-06-17 12:27 Naresh Kamboju
  2021-06-17 12:49 ` Naresh Kamboju
  0 siblings, 1 reply; 7+ messages in thread
From: Naresh Kamboju @ 2021-06-17 12:27 UTC (permalink / raw)
  To: Linux-Next Mailing List, clang-built-linux, lkft-triage, open list
  Cc: Nick Desaulniers, Nathan Chancellor, Arnd Bergmann, Stephen Rothwell

Linux next 20210617 tag following s390 builds failed with clang-10,
clang-11 and clang-12.

Regressions found on s390:

 - build/clang-11-tinyconfig
 - build/clang-11-allnoconfig
 - build/clang-12-allnoconfig
 - build/clang-12-defconfig
 - build/clang-10-tinyconfig
 - build/clang-11-defconfig
 - build/clang-10-allnoconfig
 - build/clang-12-tinyconfig
 - build/clang-10-defconfig

Build log:
------------
make --silent --keep-going --jobs=8
O=/home/tuxbuild/.cache/tuxmake/builds/current ARCH=s390
CROSS_COMPILE=s390x-linux-gnu- 'HOSTCC=sccache clang' 'CC=sccache
clang'
PLEASE submit a bug report to https://bugs.llvm.org/ and include the
crash backtrace, preprocessed source, and associated run script.
Stack dump:
0.Program arguments: /usr/lib/llvm-12/bin/clang -cc1 -triple
s390x-unknown-linux-gnu -S -disable-free -disable-llvm-verifier
-discard-value-names -main-file-name core.c -mrelocation-model pic
-pic-level 2 -pic-is-pie -fno-delete-null-pointer-checks -mllvm
-warn-stack-size=2048 -mframe-pointer=none -relaxed-aliasing
-fmath-errno -fno-rounding-math -no-integrated-as
-mconstructor-aliases -target-cpu zEC12 -target-feature +soft-float
-mbackchain -mpacked-stack -msoft-float -mfloat-abi soft
-fno-split-dwarf-inlining -debug-info-kind=limited -dwarf-version=4
-debugger-tuning=gdb -nostdsysteminc -nobuiltininc -resource-dir
/usr/lib/llvm-12/lib/clang/12.0.1 -dependency-file
kernel/sched/.core.o.d -MT kernel/sched/core.o -isystem
/usr/lib/llvm-12/lib/clang/12.0.1/include -include
/builds/linux/include/linux/compiler-version.h -include
/builds/linux/include/linux/kconfig.h -include
/builds/linux/include/linux/compiler_types.h -I
/builds/linux/arch/s390/include -I ./arch/s390/include/generated -I
/builds/linux/include -I ./include -I
/builds/linux/arch/s390/include/uapi -I
./arch/s390/include/generated/uapi -I /builds/linux/include/uapi -I
./include/generated/uapi -D __KERNEL__ -D __PACK_STACK -D
CONFIG_AS_CFI_VAL_OFFSET=1 -D CC_USING_NOP_MCOUNT -D CC_USING_FENTRY
-I /builds/linux/kernel/sched -I ./kernel/sched -D
KBUILD_MODFILE=\"kernel/sched/core\" -D KBUILD_BASENAME=\"core\" -D
KBUILD_MODNAME=\"core\" -D __KBUILD_MODNAME=kmod_core
-fmacro-prefix-map=/builds/linux/= -O2 -Wall -Wundef
-Werror=strict-prototypes -Wno-trigraphs
-Werror=implicit-function-declaration -Werror=implicit-int
-Werror=return-type -Wno-format-security
-Werror=unknown-warning-option -Wno-sign-compare -Wno-frame-address
-Wno-address-of-packed-member -Wno-format-invalid-specifier -Wno-gnu
-Wno-unused-const-variable -Wdeclaration-after-statement -Wvla
-Wno-pointer-sign -Wno-array-bounds -Werror=date-time
-Werror=incompatible-pointer-types -Wno-initializer-overrides
-Wno-format -Wno-sign-compare -Wno-format-zero-length
-Wno-pointer-to-enum-cast
-Wno-tautological-constant-out-of-range-compare -std=gnu89
-fno-dwarf-directory-asm -fdebug-compilation-dir
/home/tuxbuild/.cache/tuxmake/builds/current -ferror-limit 19 -pg
-mfentry -mnop-mcount -mrecord-mcount -fwrapv -fno-signed-char
-fwchar-type=short -fno-signed-wchar -fgnuc-version=4.2.1
-vectorize-loops -vectorize-slp -o /tmp/core-eaad47.s -x c
/builds/linux/kernel/sched/core.c
1.<eof> parser at end of file
2.Code generation
3.Running pass 'Function Pass Manager' on module
'/builds/linux/kernel/sched/core.c'.
4.Running pass 'SystemZ DAG->DAG Pattern Instruction Selection' on
function '@trace_event_raw_event_sched_switch'
 #0 0x00007f7227e2e693 llvm::sys::PrintStackTrace(llvm::raw_ostream&,
int) (/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0xd12693)
 #1 0x00007f7227e2c9b0 llvm::sys::RunSignalHandlers()
(/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0xd109b0)
 #2 0x00007f7227e2ecff (/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0xd12cff)
 #3 0x00007f722f7cb140 __restore_rt
(/lib/x86_64-linux-gnu/libpthread.so.0+0x14140)
 #4 0x00007f7227dba772 llvm::LLT::print(llvm::raw_ostream&) const
(/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0xc9e772)
 #5 0x00007f72284a850b
llvm::SelectionDAGBuilder::visitInlineAsm(llvm::CallBase const&)
(/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0x138c50b)
 #6 0x00007f722848ade7
llvm::SelectionDAGBuilder::visit(llvm::Instruction const&)
(/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0x136ede7)
 #7 0x00007f72285198b9
llvm::SelectionDAGISel::SelectBasicBlock(llvm::ilist_iterator<llvm::ilist_detail::node_options<llvm::Instruction,
false, false, void>, false, true>,
llvm::ilist_iterator<llvm::ilist_detail::node_options<llvm::Instruction,
false, false, void>, false, true>, bool&)
(/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0x13fd8b9)
 #8 0x00007f722851926a
llvm::SelectionDAGISel::SelectAllBasicBlocks(llvm::Function const&)
(/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0x13fd26a)
 #9 0x00007f7228516fa6
llvm::SelectionDAGISel::runOnMachineFunction(llvm::MachineFunction&)
(/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0x13fafa6)
#10 0x00007f7229f07742 (/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0x2deb742)
#11 0x00007f722813b09e
llvm::MachineFunctionPass::runOnFunction(llvm::Function&)
(/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0x101f09e)
#12 0x00007f7227f5619d
llvm::FPPassManager::runOnFunction(llvm::Function&)
(/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0xe3a19d)
#13 0x00007f7227f5bb83 llvm::FPPassManager::runOnModule(llvm::Module&)
(/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0xe3fb83)
#14 0x00007f7227f567ef
llvm::legacy::PassManagerImpl::run(llvm::Module&)
(/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0xe3a7ef)
#15 0x00007f722e092be6
clang::EmitBackendOutput(clang::DiagnosticsEngine&,
clang::HeaderSearchOptions const&, clang::CodeGenOptions const&,
clang::TargetOptions const&, clang::LangOptions const&,
llvm::DataLayout const&, llvm::Module*, clang::BackendAction,
std::unique_ptr<llvm::raw_pwrite_stream,
std::default_delete<llvm::raw_pwrite_stream> >)
(/usr/lib/x86_64-linux-gnu/libclang-cpp.so.12+0x1614be6)
#16 0x00007f722e32bd9f (/usr/lib/x86_64-linux-gnu/libclang-cpp.so.12+0x18add9f)
#17 0x00007f722d487054 clang::ParseAST(clang::Sema&, bool, bool)
(/usr/lib/x86_64-linux-gnu/libclang-cpp.so.12+0xa09054)
#18 0x00007f722ea20268 clang::FrontendAction::Execute()
(/usr/lib/x86_64-linux-gnu/libclang-cpp.so.12+0x1fa2268)
#19 0x00007f722e9adcf1
clang::CompilerInstance::ExecuteAction(clang::FrontendAction&)
(/usr/lib/x86_64-linux-gnu/libclang-cpp.so.12+0x1f2fcf1)
#20 0x00007f722ea83282
clang::ExecuteCompilerInvocation(clang::CompilerInstance*)
(/usr/lib/x86_64-linux-gnu/libclang-cpp.so.12+0x2005282)
#21 0x0000000000413352 cc1_main(llvm::ArrayRef<char const*>, char
const*, void*) (/usr/lib/llvm-12/bin/clang+0x413352)
#22 0x00000000004116ce (/usr/lib/llvm-12/bin/clang+0x4116ce)
#23 0x00000000004114de main (/usr/lib/llvm-12/bin/clang+0x4114de)
#24 0x00007f7226c50d0a __libc_start_main
(/lib/x86_64-linux-gnu/libc.so.6+0x26d0a)
#25 0x000000000040e88a _start (/usr/lib/llvm-12/bin/clang+0x40e88a)
clang: error: unable to execute command: Segmentation fault (core dumped)
clang: error: clang frontend command failed due to signal (use -v to
see invocation)
Debian clang version 12.0.1-+rc1-1
Target: s390x-unknown-linux-gnu
Thread model: posix
InstalledDir: /usr/local/bin
clang: note: diagnostic msg:
********************

PLEASE ATTACH THE FOLLOWING FILES TO THE BUG REPORT:
Preprocessed source(s) and associated run script(s) are located at:
clang: note: diagnostic msg: /tmp/core-86ba67.c
clang: note: diagnostic msg: /tmp/core-86ba67.sh
clang: note: diagnostic msg:

********************
make[3]: *** [/builds/linux/scripts/Makefile.build:272:
kernel/sched/core.o] Error 254

ref:
https://builds.tuxbuild.com/1u4ZMVtVqiZtUcCrP9YE89mx9iU/

Build:
--------
    git_repo: https://gitlab.com/Linaro/lkft/mirrors/next/linux-next
    git_sha: 7d9c6b8147bdd76d7eb2cf6f74f84c6918ae0939
    git_short_log: 7d9c6b8147bd (\Add linux-next specific files for 20210617\)
    kconfig: defconfig
    target_arch: s390
    toolchain: clang-12

Reported-by: Naresh Kamboju <naresh.kamboju@linaro.org>

--
Linaro LKFT
https://lkft.linaro.org

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [next] [clang] s390: clang: error: unable to execute command: Segmentation fault (core dumped)
  2021-06-17 12:27 [next] [clang] s390: clang: error: unable to execute command: Segmentation fault (core dumped) Naresh Kamboju
@ 2021-06-17 12:49 ` Naresh Kamboju
  2021-06-17 14:55   ` Heiko Carstens
  0 siblings, 1 reply; 7+ messages in thread
From: Naresh Kamboju @ 2021-06-17 12:49 UTC (permalink / raw)
  To: Linux-Next Mailing List, clang-built-linux, lkft-triage, open list
  Cc: Nick Desaulniers, Nathan Chancellor, Arnd Bergmann,
	Stephen Rothwell, Heiko Carstens, Vasily Gorbik

The git bisect found the first bad commit,

On Thu, 17 Jun 2021 at 17:57, Naresh Kamboju <naresh.kamboju@linaro.org> wrote:
>
> Linux next 20210617 tag following s390 builds failed with clang-10,
> clang-11 and clang-12.
>
> Regressions found on s390:
>
>  - build/clang-11-tinyconfig
>  - build/clang-11-allnoconfig
>  - build/clang-12-allnoconfig
>  - build/clang-12-defconfig
>  - build/clang-10-tinyconfig
>  - build/clang-11-defconfig
>  - build/clang-10-allnoconfig
>  - build/clang-12-tinyconfig
>  - build/clang-10-defconfig
>
> Build log:
> ------------
> make --silent --keep-going --jobs=8
> O=/home/tuxbuild/.cache/tuxmake/builds/current ARCH=s390
> CROSS_COMPILE=s390x-linux-gnu- 'HOSTCC=sccache clang' 'CC=sccache
> clang'
> PLEASE submit a bug report to https://bugs.llvm.org/ and include the
> crash backtrace, preprocessed source, and associated run script.
> Stack dump:
> 0.Program arguments: /usr/lib/llvm-12/bin/clang -cc1 -triple
> s390x-unknown-linux-gnu -S -disable-free -disable-llvm-verifier
> -discard-value-names -main-file-name core.c -mrelocation-model pic
> -pic-level 2 -pic-is-pie -fno-delete-null-pointer-checks -mllvm
> -warn-stack-size=2048 -mframe-pointer=none -relaxed-aliasing
> -fmath-errno -fno-rounding-math -no-integrated-as
> -mconstructor-aliases -target-cpu zEC12 -target-feature +soft-float
> -mbackchain -mpacked-stack -msoft-float -mfloat-abi soft
> -fno-split-dwarf-inlining -debug-info-kind=limited -dwarf-version=4
> -debugger-tuning=gdb -nostdsysteminc -nobuiltininc -resource-dir
> /usr/lib/llvm-12/lib/clang/12.0.1 -dependency-file
> kernel/sched/.core.o.d -MT kernel/sched/core.o -isystem
> /usr/lib/llvm-12/lib/clang/12.0.1/include -include
> /builds/linux/include/linux/compiler-version.h -include
> /builds/linux/include/linux/kconfig.h -include
> /builds/linux/include/linux/compiler_types.h -I
> /builds/linux/arch/s390/include -I ./arch/s390/include/generated -I
> /builds/linux/include -I ./include -I
> /builds/linux/arch/s390/include/uapi -I
> ./arch/s390/include/generated/uapi -I /builds/linux/include/uapi -I
> ./include/generated/uapi -D __KERNEL__ -D __PACK_STACK -D
> CONFIG_AS_CFI_VAL_OFFSET=1 -D CC_USING_NOP_MCOUNT -D CC_USING_FENTRY
> -I /builds/linux/kernel/sched -I ./kernel/sched -D
> KBUILD_MODFILE=\"kernel/sched/core\" -D KBUILD_BASENAME=\"core\" -D
> KBUILD_MODNAME=\"core\" -D __KBUILD_MODNAME=kmod_core
> -fmacro-prefix-map=/builds/linux/= -O2 -Wall -Wundef
> -Werror=strict-prototypes -Wno-trigraphs
> -Werror=implicit-function-declaration -Werror=implicit-int
> -Werror=return-type -Wno-format-security
> -Werror=unknown-warning-option -Wno-sign-compare -Wno-frame-address
> -Wno-address-of-packed-member -Wno-format-invalid-specifier -Wno-gnu
> -Wno-unused-const-variable -Wdeclaration-after-statement -Wvla
> -Wno-pointer-sign -Wno-array-bounds -Werror=date-time
> -Werror=incompatible-pointer-types -Wno-initializer-overrides
> -Wno-format -Wno-sign-compare -Wno-format-zero-length
> -Wno-pointer-to-enum-cast
> -Wno-tautological-constant-out-of-range-compare -std=gnu89
> -fno-dwarf-directory-asm -fdebug-compilation-dir
> /home/tuxbuild/.cache/tuxmake/builds/current -ferror-limit 19 -pg
> -mfentry -mnop-mcount -mrecord-mcount -fwrapv -fno-signed-char
> -fwchar-type=short -fno-signed-wchar -fgnuc-version=4.2.1
> -vectorize-loops -vectorize-slp -o /tmp/core-eaad47.s -x c
> /builds/linux/kernel/sched/core.c
> 1.<eof> parser at end of file
> 2.Code generation
> 3.Running pass 'Function Pass Manager' on module
> '/builds/linux/kernel/sched/core.c'.
> 4.Running pass 'SystemZ DAG->DAG Pattern Instruction Selection' on
> function '@trace_event_raw_event_sched_switch'
>  #0 0x00007f7227e2e693 llvm::sys::PrintStackTrace(llvm::raw_ostream&,
> int) (/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0xd12693)
>  #1 0x00007f7227e2c9b0 llvm::sys::RunSignalHandlers()
> (/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0xd109b0)
>  #2 0x00007f7227e2ecff (/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0xd12cff)
>  #3 0x00007f722f7cb140 __restore_rt
> (/lib/x86_64-linux-gnu/libpthread.so.0+0x14140)
>  #4 0x00007f7227dba772 llvm::LLT::print(llvm::raw_ostream&) const
> (/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0xc9e772)
>  #5 0x00007f72284a850b
> llvm::SelectionDAGBuilder::visitInlineAsm(llvm::CallBase const&)
> (/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0x138c50b)
>  #6 0x00007f722848ade7
> llvm::SelectionDAGBuilder::visit(llvm::Instruction const&)
> (/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0x136ede7)
>  #7 0x00007f72285198b9
> llvm::SelectionDAGISel::SelectBasicBlock(llvm::ilist_iterator<llvm::ilist_detail::node_options<llvm::Instruction,
> false, false, void>, false, true>,
> llvm::ilist_iterator<llvm::ilist_detail::node_options<llvm::Instruction,
> false, false, void>, false, true>, bool&)
> (/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0x13fd8b9)
>  #8 0x00007f722851926a
> llvm::SelectionDAGISel::SelectAllBasicBlocks(llvm::Function const&)
> (/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0x13fd26a)
>  #9 0x00007f7228516fa6
> llvm::SelectionDAGISel::runOnMachineFunction(llvm::MachineFunction&)
> (/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0x13fafa6)
> #10 0x00007f7229f07742 (/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0x2deb742)
> #11 0x00007f722813b09e
> llvm::MachineFunctionPass::runOnFunction(llvm::Function&)
> (/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0x101f09e)
> #12 0x00007f7227f5619d
> llvm::FPPassManager::runOnFunction(llvm::Function&)
> (/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0xe3a19d)
> #13 0x00007f7227f5bb83 llvm::FPPassManager::runOnModule(llvm::Module&)
> (/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0xe3fb83)
> #14 0x00007f7227f567ef
> llvm::legacy::PassManagerImpl::run(llvm::Module&)
> (/usr/lib/x86_64-linux-gnu/libLLVM-12.so.1+0xe3a7ef)
> #15 0x00007f722e092be6
> clang::EmitBackendOutput(clang::DiagnosticsEngine&,
> clang::HeaderSearchOptions const&, clang::CodeGenOptions const&,
> clang::TargetOptions const&, clang::LangOptions const&,
> llvm::DataLayout const&, llvm::Module*, clang::BackendAction,
> std::unique_ptr<llvm::raw_pwrite_stream,
> std::default_delete<llvm::raw_pwrite_stream> >)
> (/usr/lib/x86_64-linux-gnu/libclang-cpp.so.12+0x1614be6)
> #16 0x00007f722e32bd9f (/usr/lib/x86_64-linux-gnu/libclang-cpp.so.12+0x18add9f)
> #17 0x00007f722d487054 clang::ParseAST(clang::Sema&, bool, bool)
> (/usr/lib/x86_64-linux-gnu/libclang-cpp.so.12+0xa09054)
> #18 0x00007f722ea20268 clang::FrontendAction::Execute()
> (/usr/lib/x86_64-linux-gnu/libclang-cpp.so.12+0x1fa2268)
> #19 0x00007f722e9adcf1
> clang::CompilerInstance::ExecuteAction(clang::FrontendAction&)
> (/usr/lib/x86_64-linux-gnu/libclang-cpp.so.12+0x1f2fcf1)
> #20 0x00007f722ea83282
> clang::ExecuteCompilerInvocation(clang::CompilerInstance*)
> (/usr/lib/x86_64-linux-gnu/libclang-cpp.so.12+0x2005282)
> #21 0x0000000000413352 cc1_main(llvm::ArrayRef<char const*>, char
> const*, void*) (/usr/lib/llvm-12/bin/clang+0x413352)
> #22 0x00000000004116ce (/usr/lib/llvm-12/bin/clang+0x4116ce)
> #23 0x00000000004114de main (/usr/lib/llvm-12/bin/clang+0x4114de)
> #24 0x00007f7226c50d0a __libc_start_main
> (/lib/x86_64-linux-gnu/libc.so.6+0x26d0a)
> #25 0x000000000040e88a _start (/usr/lib/llvm-12/bin/clang+0x40e88a)
> clang: error: unable to execute command: Segmentation fault (core dumped)
> clang: error: clang frontend command failed due to signal (use -v to
> see invocation)
> Debian clang version 12.0.1-+rc1-1
> Target: s390x-unknown-linux-gnu
> Thread model: posix
> InstalledDir: /usr/local/bin
> clang: note: diagnostic msg:
> ********************
>
> PLEASE ATTACH THE FOLLOWING FILES TO THE BUG REPORT:
> Preprocessed source(s) and associated run script(s) are located at:
> clang: note: diagnostic msg: /tmp/core-86ba67.c
> clang: note: diagnostic msg: /tmp/core-86ba67.sh
> clang: note: diagnostic msg:
>
> ********************
> make[3]: *** [/builds/linux/scripts/Makefile.build:272:
> kernel/sched/core.o] Error 254

The first bad commit:
commit 3abbdfde5a6588a92209cd8b131769b8058e7c21
Author: Heiko Carstens <hca@linux.ibm.com>
Date:   Wed Jun 9 22:59:13 2021 +0200
    s390/bitops: use register pair instead of register asm

    Get rid of register asm statement and use a register pair.
    This allows the compiler to allocate registers on its own.

    Signed-off-by: Heiko Carstens <hca@linux.ibm.com>
    Signed-off-by: Vasily Gorbik <gor@linux.ibm.com>
 arch/s390/include/asm/bitops.h | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)
Previous HEAD position was 3abbdfde5a65 s390/bitops: use register pair
instead of register asm
HEAD is now at 7d9c6b8147bd Add linux-next specific files for 20210617

>
> ref:
> https://builds.tuxbuild.com/1u4ZMVtVqiZtUcCrP9YE89mx9iU/
>
> Build:
> --------
>     git_repo: https://gitlab.com/Linaro/lkft/mirrors/next/linux-next
>     git_sha: 7d9c6b8147bdd76d7eb2cf6f74f84c6918ae0939
>     git_short_log: 7d9c6b8147bd (\Add linux-next specific files for 20210617\)
>     kconfig: defconfig
>     target_arch: s390
>     toolchain: clang-12
>
> Reported-by: Naresh Kamboju <naresh.kamboju@linaro.org>

- Naresh

> --
> Linaro LKFT
> https://lkft.linaro.org

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [next] [clang] s390: clang: error: unable to execute command: Segmentation fault (core dumped)
  2021-06-17 12:49 ` Naresh Kamboju
@ 2021-06-17 14:55   ` Heiko Carstens
  2021-06-17 19:31     ` [PATCH] scripts/min-tool-version.sh: Raise minimum clang version to 13.0.0 for s390 Nathan Chancellor
  0 siblings, 1 reply; 7+ messages in thread
From: Heiko Carstens @ 2021-06-17 14:55 UTC (permalink / raw)
  To: Naresh Kamboju
  Cc: Linux-Next Mailing List, clang-built-linux, lkft-triage,
	open list, Nick Desaulniers, Nathan Chancellor, Arnd Bergmann,
	Stephen Rothwell, Vasily Gorbik

On Thu, Jun 17, 2021 at 06:19:14PM +0530, Naresh Kamboju wrote:
> The git bisect found the first bad commit,
> The first bad commit:
> commit 3abbdfde5a6588a92209cd8b131769b8058e7c21
> Author: Heiko Carstens <hca@linux.ibm.com>
> Date:   Wed Jun 9 22:59:13 2021 +0200
>     s390/bitops: use register pair instead of register asm
> 
>     Get rid of register asm statement and use a register pair.
>     This allows the compiler to allocate registers on its own.
> 
>     Signed-off-by: Heiko Carstens <hca@linux.ibm.com>
>     Signed-off-by: Vasily Gorbik <gor@linux.ibm.com>
>  arch/s390/include/asm/bitops.h | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> Previous HEAD position was 3abbdfde5a65 s390/bitops: use register pair
> instead of register asm
> HEAD is now at 7d9c6b8147bd Add linux-next specific files for 20210617

You need clang-13 development version which contains commit
d058262b1471 ("[SystemZ] Support i128 inline asm operands.").
Every older clang version won't be able to complile the kernel for
s390 anymore.

This is intended and won't be fixed.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH] scripts/min-tool-version.sh: Raise minimum clang version to 13.0.0 for s390
  2021-06-17 14:55   ` Heiko Carstens
@ 2021-06-17 19:31     ` Nathan Chancellor
  2021-06-17 19:59       ` Nick Desaulniers
  2021-06-18  8:42       ` Heiko Carstens
  0 siblings, 2 replies; 7+ messages in thread
From: Nathan Chancellor @ 2021-06-17 19:31 UTC (permalink / raw)
  To: Heiko Carstens, Vasily Gorbik, Christian Borntraeger, Masahiro Yamada
  Cc: Nick Desaulniers, linux-kernel, clang-built-linux, linux-s390,
	linux-next, lkft-triage, Arnd Bergmann, Stephen Rothwell,
	Nathan Chancellor, Naresh Kamboju

clang versions prior to the current development version of 13.0.0 cannot
compile s390 after commit 3abbdfde5a65 ("s390/bitops: use register pair
instead of register asm") and the s390 maintainers do not intend to work
around this in the kernel. Codify this in scripts/min-tool-version.sh
similar to arm64 with GCC 5.1.0 so that there are no reports of broken
builds.

Reported-by: Naresh Kamboju <naresh.kamboju@linaro.org>
Signed-off-by: Nathan Chancellor <nathan@kernel.org>
---

This should probably go through the s390 tree with Masahiro's ack.

 scripts/min-tool-version.sh | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/scripts/min-tool-version.sh b/scripts/min-tool-version.sh
index d22cf91212b0..319f92104f56 100755
--- a/scripts/min-tool-version.sh
+++ b/scripts/min-tool-version.sh
@@ -30,7 +30,12 @@ icc)
 	echo 16.0.3
 	;;
 llvm)
-	echo 10.0.1
+	# https://lore.kernel.org/r/YMtib5hKVyNknZt3@osiris/
+	if [ "$SRCARCH" = s390 ]; then
+		echo 13.0.0
+	else
+		echo 10.0.1
+	fi
 	;;
 *)
 	echo "$1: unknown tool" >&2

base-commit: 7d9c6b8147bdd76d7eb2cf6f74f84c6918ae0939
-- 
2.32.0.93.g670b81a890


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] scripts/min-tool-version.sh: Raise minimum clang version to 13.0.0 for s390
  2021-06-17 19:31     ` [PATCH] scripts/min-tool-version.sh: Raise minimum clang version to 13.0.0 for s390 Nathan Chancellor
@ 2021-06-17 19:59       ` Nick Desaulniers
  2021-06-18  8:42       ` Heiko Carstens
  1 sibling, 0 replies; 7+ messages in thread
From: Nick Desaulniers @ 2021-06-17 19:59 UTC (permalink / raw)
  To: Nathan Chancellor
  Cc: Heiko Carstens, Vasily Gorbik, Christian Borntraeger,
	Masahiro Yamada, LKML, clang-built-linux, linux-s390,
	Linux Next Mailing List, lkft-triage, Arnd Bergmann,
	Stephen Rothwell, Naresh Kamboju

On Thu, Jun 17, 2021 at 12:32 PM Nathan Chancellor <nathan@kernel.org> wrote:
>
> clang versions prior to the current development version of 13.0.0 cannot
> compile s390 after commit 3abbdfde5a65 ("s390/bitops: use register pair
> instead of register asm") and the s390 maintainers do not intend to work
> around this in the kernel. Codify this in scripts/min-tool-version.sh
> similar to arm64 with GCC 5.1.0 so that there are no reports of broken
> builds.
>
> Reported-by: Naresh Kamboju <naresh.kamboju@linaro.org>
> Signed-off-by: Nathan Chancellor <nathan@kernel.org>

Acked-by: Nick Desaulniers <ndesaulniers@google.com>

> ---
>
> This should probably go through the s390 tree with Masahiro's ack.
>
>  scripts/min-tool-version.sh | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/scripts/min-tool-version.sh b/scripts/min-tool-version.sh
> index d22cf91212b0..319f92104f56 100755
> --- a/scripts/min-tool-version.sh
> +++ b/scripts/min-tool-version.sh
> @@ -30,7 +30,12 @@ icc)
>         echo 16.0.3
>         ;;
>  llvm)
> -       echo 10.0.1
> +       # https://lore.kernel.org/r/YMtib5hKVyNknZt3@osiris/
> +       if [ "$SRCARCH" = s390 ]; then
> +               echo 13.0.0
> +       else
> +               echo 10.0.1
> +       fi
>         ;;
>  *)
>         echo "$1: unknown tool" >&2
>
> base-commit: 7d9c6b8147bdd76d7eb2cf6f74f84c6918ae0939
> --
> 2.32.0.93.g670b81a890
>
> --
> You received this message because you are subscribed to the Google Groups "Clang Built Linux" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to clang-built-linux+unsubscribe@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/clang-built-linux/20210617193139.856957-1-nathan%40kernel.org.



-- 
Thanks,
~Nick Desaulniers

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] scripts/min-tool-version.sh: Raise minimum clang version to 13.0.0 for s390
  2021-06-17 19:31     ` [PATCH] scripts/min-tool-version.sh: Raise minimum clang version to 13.0.0 for s390 Nathan Chancellor
  2021-06-17 19:59       ` Nick Desaulniers
@ 2021-06-18  8:42       ` Heiko Carstens
  2021-06-19  0:40         ` Masahiro Yamada
  1 sibling, 1 reply; 7+ messages in thread
From: Heiko Carstens @ 2021-06-18  8:42 UTC (permalink / raw)
  To: Nathan Chancellor
  Cc: Vasily Gorbik, Christian Borntraeger, Masahiro Yamada,
	Nick Desaulniers, linux-kernel, clang-built-linux, linux-s390,
	linux-next, lkft-triage, Arnd Bergmann, Stephen Rothwell,
	Naresh Kamboju

On Thu, Jun 17, 2021 at 12:31:40PM -0700, Nathan Chancellor wrote:
> clang versions prior to the current development version of 13.0.0 cannot
> compile s390 after commit 3abbdfde5a65 ("s390/bitops: use register pair
> instead of register asm") and the s390 maintainers do not intend to work
> around this in the kernel. Codify this in scripts/min-tool-version.sh
> similar to arm64 with GCC 5.1.0 so that there are no reports of broken
> builds.
> 
> Reported-by: Naresh Kamboju <naresh.kamboju@linaro.org>
> Signed-off-by: Nathan Chancellor <nathan@kernel.org>
> ---
> 
> This should probably go through the s390 tree with Masahiro's ack.

Thank's a lot!

I'll add the below text to the commit message, and apply it internally
first:

[hca@linux.ibm.com: breaking compatibility with older clang compilers
 is intended to finally make use of a feature which allows the
 compiler to allocate even/odd register pairs. This is possible since
 a very long time with gcc, but only since llvm-project commit
 d058262b1471 ("[SystemZ] Support i128 inline asm operands.") with
 clang. Using that feature allows to get rid of error prone register
 asm statements, of which the above named kernel commit is only the
 first of a larger not yet complete series]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] scripts/min-tool-version.sh: Raise minimum clang version to 13.0.0 for s390
  2021-06-18  8:42       ` Heiko Carstens
@ 2021-06-19  0:40         ` Masahiro Yamada
  0 siblings, 0 replies; 7+ messages in thread
From: Masahiro Yamada @ 2021-06-19  0:40 UTC (permalink / raw)
  To: Heiko Carstens
  Cc: Nathan Chancellor, Vasily Gorbik, Christian Borntraeger,
	Nick Desaulniers, Linux Kernel Mailing List, clang-built-linux,
	linux-s390, Linux-Next Mailing List, lkft-triage, Arnd Bergmann,
	Stephen Rothwell, Naresh Kamboju

On Fri, Jun 18, 2021 at 5:42 PM Heiko Carstens <hca@linux.ibm.com> wrote:
>
> On Thu, Jun 17, 2021 at 12:31:40PM -0700, Nathan Chancellor wrote:
> > clang versions prior to the current development version of 13.0.0 cannot
> > compile s390 after commit 3abbdfde5a65 ("s390/bitops: use register pair
> > instead of register asm") and the s390 maintainers do not intend to work
> > around this in the kernel. Codify this in scripts/min-tool-version.sh
> > similar to arm64 with GCC 5.1.0 so that there are no reports of broken
> > builds.
> >
> > Reported-by: Naresh Kamboju <naresh.kamboju@linaro.org>
> > Signed-off-by: Nathan Chancellor <nathan@kernel.org>
> > ---
> >
> > This should probably go through the s390 tree with Masahiro's ack.

Acked-by: Masahiro Yamada <masahiroy@kernel.org>


> Thank's a lot!
>
> I'll add the below text to the commit message, and apply it internally
> first:
>
> [hca@linux.ibm.com: breaking compatibility with older clang compilers
>  is intended to finally make use of a feature which allows the
>  compiler to allocate even/odd register pairs. This is possible since
>  a very long time with gcc, but only since llvm-project commit
>  d058262b1471 ("[SystemZ] Support i128 inline asm operands.") with
>  clang. Using that feature allows to get rid of error prone register
>  asm statements, of which the above named kernel commit is only the
>  first of a larger not yet complete series]
>
> --
> You received this message because you are subscribed to the Google Groups "Clang Built Linux" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to clang-built-linux+unsubscribe@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/clang-built-linux/YMxcdv/1taBevSjP%40osiris.



-- 
Best Regards
Masahiro Yamada

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2021-06-19  0:41 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-17 12:27 [next] [clang] s390: clang: error: unable to execute command: Segmentation fault (core dumped) Naresh Kamboju
2021-06-17 12:49 ` Naresh Kamboju
2021-06-17 14:55   ` Heiko Carstens
2021-06-17 19:31     ` [PATCH] scripts/min-tool-version.sh: Raise minimum clang version to 13.0.0 for s390 Nathan Chancellor
2021-06-17 19:59       ` Nick Desaulniers
2021-06-18  8:42       ` Heiko Carstens
2021-06-19  0:40         ` Masahiro Yamada

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).