linaro-toolchain

linaro-toolchain@lists.linaro.org

9 participants
5515 discussions

Re: [Linaro-TCWG-CI] 2 patches in binutils: Failure on arm

by Ulf Samuelsson

I get this problem when the CI is building a patchset for the binutils with my patches. The patchset consists of 7 patches, and I suspect that this error message is generated when only part of the 7 patches are applied. Could this be true? They are interdependent, so there is no way to reorder the patches to make this work if not all the patches are applied. Den 2024-07-07 kl. 02:47, skrev ci_notify(a)linaro.org: > Dear contributor, our automatic CI has detected problems related to your patch(es). Please find some details below. If you have any questions, please follow up on linaro-toolchain(a)lists.linaro.org mailing list, Libera's #linaro-tcwg channel, or ping your favourite Linaro toolchain developer on the usual project channel. > > We appreciate that it might be difficult to find the necessary logs or reproduce the issue locally. If you can't get what you need from our CI within minutes, let us know and we will be happy to help. > > In binutils_build master-arm after: > > | 2 patches in binutils > | Patchwork URL: https://patchwork.sourceware.org/patch/93436 > | dd57e0ed6f6 ldgram.y: Add ASCII parsing > | 304119944bb ldlex.l: Add ASCII token > | ... applied on top of baseline commit: > | 85a67d0a39a Automatic date update in version.in > > Results changed to > # reset_artifacts: > -10 > # true: > 0 > # build_abe binutils: > # FAILED > # First few build errors in logs: > # 00:01:58 /home/tcwg-build/workspace/tcwg_gnu_2/abe/snapshots/binutils.git~master/ld/ldgram.y:711:21: error: too many arguments to function ‘lang_add_string’ > # 00:01:58 /home/tcwg-build/workspace/tcwg_gnu_2/abe/snapshots/binutils.git~master/ld/ldgram.y:717:19: error: too many arguments to function ‘lang_add_string’ > # 00:01:58 make[4]: *** [Makefile:2290: ldgram.o] Error 1 > # 00:01:58 make[3]: *** [Makefile:1903: all-recursive] Error 1 > # 00:01:58 make[2]: *** [Makefile:1092: all] Error 2 > # 00:01:58 make[1]: *** [Makefile:8044: all-ld] Error 2 > # 00:01:58 make: *** [Makefile:1028: all] Error 2 > > From > # reset_artifacts: > -10 > # true: > 0 > # build_abe binutils: > 1 > > The configuration of this build is: > CI config tcwg_binutils_build master-arm > > -----------------8<--------------------------8<--------------------------8<-------------------------- > The information below can be used to reproduce a debug environment: > > Current build : https://ci.linaro.org/job/tcwg_binutils_build--master-arm-precommit/2298/ar… > Reference build : https://ci.linaro.org/job/tcwg_binutils_build--master-arm-build/1304/artifa… > > Warning: we do not enable maintainer-mode nor automatically update > generated files, which may lead to failures if the patch modifies the > master files. -- Best Regards Ulf Samuelsson

6 months, 2 weeks

C++ standard library for build bot clang-armv8-lld-2stage

by mingmingl

Greetings! This is Mingming and I'm an LLVM contributor. I have received a lot of useful code review feedback from aarch64 experts. Thank you for all of that! I'm writing to report a failure ( https://lab.llvm.org/buildbot/#/builders/122/builds/150) on buildbot clang-armv8-lld-2stage triggered by my recent patch. The issue seems to stem from the C++ standard libraries not being configured in the buildbot environment. While using a c-style header is a workaround (which pr 97245 <https://github.com/llvm/llvm-project/pull/97245> did), @petrhosek reminded me that it would be good to inform maintainers so they can take a look, which makes sense to me. Could someone please assist in resolving this configuration problem? Thank you in advance and let me know if I miss anything. Thanks, Mingming

6 months, 3 weeks

[ACTIVITY] report week ending 28 Jun

by Peter Maydell

Progress: * UM-2 [QEMU upstream maintainership] - patch-review queue has filled up again; reviewed at least some patches ready for an arm pullreq - another KVM Forum PC meeting * QEMU-530 [QEMU ARMv9.5 Baseline CPU for TCG] - realized that FEAT_AFP means we need to refactor our FPCR/FPSR implementation, because it adds new AArch64-only bits that aren't visible in the AArch32 FPSCR format. (QEMU for historical reasons implemented FPCR and FPSR as masked views of FPSCR; now we need to switch to having FPSCR be a view of the FPCR/FPSR data.) Wrote the necessary refactoring and sent patchseries out for review. -- PMM

6 months, 3 weeks

Re: [PATCH 1/3] Release structures on function return

by Jørgen Kvalsvik

I think we need to revert this. I got this email from linaro/gcc-regressions: [Linaro-TCWG-CI] gcc-15-1649-g19f630e6ae8d: FAIL: 2 regressions on aarch64 regressions.sum: === gcc tests === Running gcc:gcc.misc-tests/gcov.exp ... FAIL: gcc.misc-tests/gcov-23.c (internal compiler error: in operator[], at vec.h:910) FAIL: gcc.misc-tests/gcov-23.c (test for excess errors) This did not reproduce on my machine, but I took a quick look at the hash-map implementation. hash_map.put calls hash_table.find_slot_with_hash, which calls hash_table.expand, which does move+destroy. auto_vec is not really move-aware which leads to a double-free. The fix is either to make auto_vec move-aware (and more like C++'s std::vector) or revert this patch and apply the original version with an explicit release. OK? Thanks, Jørgen On 6/25/24 12:23, Jan Hubicka wrote: >> The value vec objects are destroyed on exit, but release still needs to >> be called explicitly. >> >> gcc/ChangeLog: >> >> * tree-profile.cc (find_conditions): Release vectors before >> return. > I wonder if you turn > hash_map<int_hash<unsigned, 0>, vec<basic_block>> exprs; > to > hash_map<int_hash<unsigned, 0>, auto_vec<basic_block>> exprs; > Won't hash_map destructor take care of this by itself? > > Honza >> --- >> gcc/tree-profile.cc | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/gcc/tree-profile.cc b/gcc/tree-profile.cc >> index e4bb689cef5..18f48e8d04e 100644 >> --- a/gcc/tree-profile.cc >> +++ b/gcc/tree-profile.cc >> @@ -919,6 +919,9 @@ find_conditions (struct function *fn) >> if (!have_post_dom) >> free_dominance_info (fn, CDI_POST_DOMINATORS); >> >> + for (auto expr : exprs) >> + expr.second.release (); >> + >> cov->m_masks.safe_grow_cleared (2 * cov->m_index.last ()); >> const size_t length = cov_length (cov); >> for (size_t i = 0; i != length; i++) >> -- >> 2.39.2 >>

6 months, 3 weeks

Re: [Linaro-TCWG-CI] gcc-15-1619-g3b9b8d6cfdf5: FAIL: 4 regressions on aarch64

by Surya Kumari Jangala

Hi, I am looking into these regressions. Regards, Surya On 26/06/24 7:51 am, ci_notify(a)linaro.org wrote: > Dear contributor, our automatic CI has detected problems related to your patch(es). Please find some details below. If you have any questions, please follow up on linaro-toolchain(a)lists.linaro.org mailing list, Libera's #linaro-tcwg channel, or ping your favourite Linaro toolchain developer on the usual project channel. > > We appreciate that it might be difficult to find the necessary logs or reproduce the issue locally. If you can't get what you need from our CI within minutes, let us know and we will be happy to help. > > We track this report status in https://linaro.atlassian.net/browse/GNU-1270 , please let us know if you are looking at the problem and/or when you have a fix. > > In gcc_check master-aarch64 after: > > | commit gcc-15-1619-g3b9b8d6cfdf5 > | Author: Surya Kumari Jangala <jskumari(a)linux.ibm.com> > | Date: Tue Jun 25 08:37:49 2024 -0500 > | > | ira: Scale save/restore costs of callee save registers with block frequency > | > | In assign_hard_reg(), when computing the costs of the hard registers, the > | cost of saving/restoring a callee-save hard register in prolog/epilog is > | taken into consideration. However, this cost is not scaled with the entry > | block frequency. Without scaling, the cost of saving/restoring is quite > | small and this can result in a callee-save register being chosen by > | ... 14 lines of the commit log omitted. > > FAIL: 4 regressions > > regressions.sum: > === g++ tests === > > Running g++:g++.target/aarch64/sve/acle/aarch64-sve-acle.exp ... > FAIL: gcc.target/aarch64/sve/acle/general/cpy_1.c -march=armv8.2-a+sve -moverride=tune=none check-function-bodies dup_x0_m > === gcc tests === > > Running gcc:gcc.dg/dg.exp ... > FAIL: gcc.dg/pr10474.c scan-rtl-dump pro_and_epilogue "Performing shrink-wrapping" > > Running gcc:gcc.target/aarch64/sve/acle/aarch64-sve-acle.exp ... > ... and 5 more entries > > You can find the failure logs in *.log.1.xz files in > - https://ci.linaro.org/job/tcwg_gcc_check--master-aarch64-build/2235/artifac… > The full lists of regressions and progressions as well as configure and make commands are in > - https://ci.linaro.org/job/tcwg_gcc_check--master-aarch64-build/2235/artifac… > The list of [ignored] baseline and flaky failures are in > - https://ci.linaro.org/job/tcwg_gcc_check--master-aarch64-build/2235/artifac… > > The configuration of this build is: > CI config tcwg_gcc_check master-aarch64 > > -----------------8<--------------------------8<--------------------------8<-------------------------- > The information below can be used to reproduce a debug environment: > > Current build : https://ci.linaro.org/job/tcwg_gcc_check--master-aarch64-build/2235/artifac… > Reference build : https://ci.linaro.org/job/tcwg_gcc_check--master-aarch64-build/2234/artifac… > > Reproduce last good and first bad builds: https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/gcc/sh… > > Full commit : https://github.com/gcc-mirror/gcc/commit/3b9b8d6cfdf59337f4b7ce10ce92a98044… > > List of configurations that regressed due to this commit : > * tcwg_gcc_check > ** master-aarch64 > *** FAIL: 4 regressions > *** https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/gcc/sh… > *** https://ci.linaro.org/job/tcwg_gcc_check--master-aarch64-build/2235/artifac…

6 months, 3 weeks

[Linaro-TCWG-CI] binutils-2_41-release-3611-g057a2b4c4b5: FAIL: 5 regressions on aarch64

by ci_notify＠linaro.org

Dear contributor, our automatic CI has detected problems related to your patch(es). Please find some details below. If you have any questions, please follow up on linaro-toolchain(a)lists.linaro.org mailing list, Libera's #linaro-tcwg channel, or ping your favourite Linaro toolchain developer on the usual project channel. We appreciate that it might be difficult to find the necessary logs or reproduce the issue locally. If you can't get what you need from our CI within minutes, let us know and we will be happy to help. We track this report status in https://linaro.atlassian.net/browse/GNU-1271 , please let us know if you are looking at the problem and/or when you have a fix. In binutils_check master-aarch64 after: | commit binutils-2_41-release-3611-g057a2b4c4b5 | Author: Nick Clifton <nickc(a)redhat.com> | Date: Wed Jun 26 13:17:25 2024 +0100 | | Revert: 35fd2ddeb1d90f1750401cfb6d01fe055656b88d | | PR 20814 FAIL: 5 regressions regressions.sum: === ld tests === Running ld:ld-plugin/libdep.exp ... ERROR: can't read "libdep": no such variable FAIL: libdep test - could not locate libdep plugin ERROR: tcl error sourcing ld-plugin/libdep.exp. ERROR: tcl error code TCL READ VARNAME UNRESOLVED: testcase ld-plugin/libdep.exp' aborted due to Tcl error You can find the failure logs in *.log.1.xz files in - https://ci.linaro.org/job/tcwg_binutils_check--master-aarch64-build/1290/ar… The full lists of regressions and progressions as well as configure and make commands are in - https://ci.linaro.org/job/tcwg_binutils_check--master-aarch64-build/1290/ar… The list of [ignored] baseline and flaky failures are in - https://ci.linaro.org/job/tcwg_binutils_check--master-aarch64-build/1290/ar… The configuration of this build is: CI config tcwg_binutils_check master-aarch64 -----------------8<--------------------------8<--------------------------8<-------------------------- The information below can be used to reproduce a debug environment: Current build : https://ci.linaro.org/job/tcwg_binutils_check--master-aarch64-build/1290/ar… Reference build : https://ci.linaro.org/job/tcwg_binutils_check--master-aarch64-build/1288/ar… Reproduce last good and first bad builds: https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/binuti… Full commit : https://sourceware.org/git/?p=binutils-gdb.git;a=commitdiff;h=057a2b4c4b57e… List of configurations that regressed due to this commit : * tcwg_binutils_check ** master-aarch64 *** FAIL: 5 regressions *** https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/binuti… *** https://ci.linaro.org/job/tcwg_binutils_check--master-aarch64-build/1290/ar…

6 months, 3 weeks

Re: [Linaro-TCWG-CI] gcc-15-1500-g1340ddea015: FAIL: 2 regressions on arm

by Maxim Kuvyrkov

Hi Matthias, Looking at [1], we are seeing this error only for bare-metal 32-bit ARM target, which, I believe, is not a configuration that compiler explorer supports. The specific configurations are either soft-float, or target ARM cores without NEON (aka no SIMD FPU). Maybe the test should be guarded on the target supporting vectorization? I.e., maybe add // { dg-require-effective-target vect_int } ? [1] https://linaro.atlassian.net/browse/GNU-1265 -- Maxim Kuvyrkov https://www.linaro.org > On Jun 21, 2024, at 19:03, Matthias Kretz via Gcc-regression <gcc-regression(a)gcc.gnu.org> wrote: > > Hi, > > I looked into this and still don't understand how to reproduce it. > > The issue as logged in libstdc++.log is a call from > find_last_set(simd_mask<int, _VecBuiltin<16>>) to > _MaskImplNeonMixin::_S_to_bits, which calls vpadd_s32 with two > [[gnu::vector_size(8)]] int arguments. The CI compiler complains "error: > cannot convert '__vector(2) int' to 'int32x2_t'". > > Why is that an error on your side and not on any compiler I have at hand? > > https://compiler-explorer.com/z/88WhcM7Kb not an error here. > > In any case, the new test only exposes an existing issue in the <experimental/ > simd> implementation (or its interaction with some variants of GCC?). The > change did not introduce a new bug to libstdc++. > > Best, > Matthias > > > On Friday, 21 June 2024 17:00:23 GMT+2 ci_notify(a)linaro.org wrote: >> Dear contributor, our automatic CI has detected problems related to your >> patch(es). Please find some details below. If you have any questions, >> please follow up on linaro-toolchain(a)lists.linaro.org mailing list, >> Libera's #linaro-tcwg channel, or ping your favourite Linaro toolchain >> developer on the usual project channel. >> >> We appreciate that it might be difficult to find the necessary logs or >> reproduce the issue locally. If you can't get what you need from our CI >> within minutes, let us know and we will be happy to help. >> >> We track this report status in https://linaro.atlassian.net/browse/GNU-1265 >> , please let us know if you are looking at the problem and/or when you have >> a fix. >> In arm-eabi v7-a softfp after: >> | commit gcc-15-1500-g1340ddea015 >> | Author: Matthias Kretz <m.kretz(a)gsi.de> >> | Date: Fri Jun 14 15:11:25 2024 +0200 >> | >> | libstdc++: Fix find_last_set(simd_mask) to ignore padding bits >> | >> | With the change to the AVX512 find_last_set implementation, the >> | change >> | to AVX512 operator!= is unnecessary. However, the latter was not >> | producing optimal code and unnecessarily set the padding bits. In >> | theory, the compiler could determine that with the new != >> | implementation, the bit operation for clearing the padding bits is a >> | >> | ... 13 lines of the commit log omitted. >> >> FAIL: 2 regressions >> >> regressions.sum: >> === libstdc++ tests === >> >> Running libstdc++:libstdc++-dg/conformance.exp ... >> FAIL: experimental/simd/pr115454_find_last_set.cc -mfpu=neon -ffast-math -O2 >> -Wno-psabi (test for excess errors) UNRESOLVED: >> experimental/simd/pr115454_find_last_set.cc -mfpu=neon -ffast-math -O2 >> -Wno-psabi compilation failed to produce executable >> >> >> You can find the failure logs in *.log.1.xz files in >> - >> https://ci.linaro.org/job/tcwg_gnu_embed_check_gcc--master-arm_v7a_softfp_e >> abi-build/512/artifact/artifacts/00-sumfiles/ The full lists of regressions >> and progressions as well as configure and make commands are in - >> https://ci.linaro.org/job/tcwg_gnu_embed_check_gcc--master-arm_v7a_softfp_e >> abi-build/512/artifact/artifacts/notify/ The list of [ignored] baseline and >> flaky failures are in >> - >> https://ci.linaro.org/job/tcwg_gnu_embed_check_gcc--master-arm_v7a_softfp_e >> abi-build/512/artifact/artifacts/sumfiles/xfails.xfail >> >> The configuration of this build is: >> CI config tcwg_gnu_embed_check_gcc arm-eabi -marm -march=armv7-a >> -mfpu=vfpv3-d16 -mfloat-abi=softfp >> >> -----------------8<--------------------------8<--------------------------8<- >> ------------------------- The information below can be used to reproduce a >> debug environment: >> >> Current build : >> https://ci.linaro.org/job/tcwg_gnu_embed_check_gcc--master-arm_v7a_softfp_e >> abi-build/512/artifact/artifacts Reference build : >> https://ci.linaro.org/job/tcwg_gnu_embed_check_gcc--master-arm_v7a_softfp_e >> abi-build/511/artifact/artifacts >> >> Reproduce last good and first bad builds: >> https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/gcc/sh >> a1/1340ddea0158de3f49aeb75b4013e5fc313ff6f4/tcwg_gnu_embed_check_gcc/master- >> arm_v7a_softfp_eabi/reproduction_instructions.txt >> >> Full commit : >> https://github.com/gcc-mirror/gcc/commit/1340ddea0158de3f49aeb75b4013e5fc31 >> 3ff6f4 >> >> List of configurations that regressed due to this commit : >> * tcwg_gnu_embed_check_gcc >> ** master-arm_v7a_softfp_eabi >> *** FAIL: 2 regressions >> *** >> https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/gcc/sh >> a1/1340ddea0158de3f49aeb75b4013e5fc313ff6f4/tcwg_gnu_embed_check_gcc/master- >> arm_v7a_softfp_eabi/details.txt *** >> https://ci.linaro.org/job/tcwg_gnu_embed_check_gcc--master-arm_v7a_softfp_e >> abi-build/512/artifact/artifacts > > > -- > ────────────────────────────────────────────────────────────────────────── > Dr. Matthias Kretz https://mattkretz.github.io > GSI Helmholtz Center for Heavy Ion Research https://gsi.de > std::simd > ──────────────────────────────────────────────────────────────────────────

7 months

[ACTIVITY] report week ending 21 Jun

by Peter Maydell

Progress: * UM-2 [QEMU upstream maintainership] + code review and target-arm pullreq + Coverity gardening: triage of new issues; update of component definitions + respin of "allow a second UART in virt machine" patchset from last year + KVM Forum programme committee work -- PMM

7 months

clang-arm64-windows-msvc cannot checkout LLVM

by Galina Kistanova

Hello Linaro Toolchain Working Group, clang-arm64-windows-msvc cannot checkout LLVM because of this error: fatal: Unable to create 'C:/Users/tcwg/llvm-worker/clang-arm64-windows-msvc/llvm/.git/index.lock': File exists. Looks like it needs some love and care. Could you look at this, please? Thanks Galina

7 months

[Linaro-TCWG-CI] glibc-2.39.9000-340-g176671f604: FAIL: 1 regressions on aarch64

by ci_notify＠linaro.org

Dear contributor, our automatic CI has detected problems related to your patch(es). Please find some details below. If you have any questions, please follow up on linaro-toolchain(a)lists.linaro.org mailing list, Libera's #linaro-tcwg channel, or ping your favourite Linaro toolchain developer on the usual project channel. We appreciate that it might be difficult to find the necessary logs or reproduce the issue locally. If you can't get what you need from our CI within minutes, let us know and we will be happy to help. We track this report status in https://linaro.atlassian.net/browse/GNU-1261 , please let us know if you are looking at the problem and/or when you have a fix. In glibc_check master-aarch64 after: | commit glibc-2.39.9000-340-g176671f604 | Author: Carlos Llamas <cmllamas(a)google.com> | Date: Tue Jun 18 10:56:34 2024 +0200 | | linux: add definitions for hugetlb page size encodings | | A desired hugetlb page size can be encoded in the flags parameter of | system calls such as mmap() and shmget(). The Linux UAPI headers have | included explicit definitions for these encodings since v4.14. | | This patch adds these definitions that are used along with MAP_HUGETLB | ... 14 lines of the commit log omitted. FAIL: 1 regressions regressions.sum: === glibc tests === Running glibc:misc ... FAIL: misc/tst-mman-consts You can find the failure logs in *.log.1.xz files in - https://ci.linaro.org/job/tcwg_glibc_check--master-aarch64-build/1479/artif… The full lists of regressions and progressions as well as configure and make commands are in - https://ci.linaro.org/job/tcwg_glibc_check--master-aarch64-build/1479/artif… The list of [ignored] baseline and flaky failures are in - https://ci.linaro.org/job/tcwg_glibc_check--master-aarch64-build/1479/artif… The configuration of this build is: CI config tcwg_glibc_check master-aarch64 -----------------8<--------------------------8<--------------------------8<-------------------------- The information below can be used to reproduce a debug environment: Current build : https://ci.linaro.org/job/tcwg_glibc_check--master-aarch64-build/1479/artif… Reference build : https://ci.linaro.org/job/tcwg_glibc_check--master-aarch64-build/1478/artif… Reproduce last good and first bad builds: https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/glibc/… Full commit : https://sourceware.org/git/?p=glibc.git;a=commitdiff;h=176671f6042912200ea9… List of configurations that regressed due to this commit : * tcwg_glibc_check ** master-aarch64 *** FAIL: 1 regressions *** https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/glibc/… *** https://ci.linaro.org/job/tcwg_glibc_check--master-aarch64-build/1479/artif…

7 months

← Newer
1
...
9
10
11
12
13
14
15
...
552
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

linaro-toolchain