> On Jan 25, 2024, at 19:04, Guinevere Larsen <blar...@redhat.com> wrote: > > On 25/01/2024 10:10, Maxim Kuvyrkov wrote: >>> On Jan 25, 2024, at 04:08, ci_not...@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@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-1120 >>> , please let us know if you are looking at the problem and/or when you have >>> a fix. >>> >>> In gdb_check master-arm after: >>> >>> | commit gdb-14-branchpoint-1354-g8669a8b6740 >>> | Author: Guinevere Larsen <blar...@redhat.com> >>> | Date: Thu Aug 24 11:00:35 2023 +0200 >>> | >>> | gdb/testsuite: add test for backtracing for threaded inferiors from >>> a corefile >>> | >>> | This patch is based on an out-of-tree patch that fedora has been >>> | carrying for a while. It tests if GDB is able to properly unwind a >>> | threaded program in the following situations: >>> | * regular threads >>> | * in a signal handler >>> | ... 14 lines of the commit log omitted. >>> >>> FAIL: 2 regressions >>> >>> regressions.sum: >>> === gdb tests === >>> >>> Running gdb:gdb.threads/threadcrash.exp ... >>> FAIL: gdb.threads/threadcrash.exp: test_gcore: $thread_count == 7 >>> FAIL: gdb.threads/threadcrash.exp: test_gcore: $thread_count == [llength >>> $test_list] >> Hi Guinevere, >> >> The failures seem to be due to "LWP" output (instead of "Thread") in >> test_gcore. >> >> I.e., test_corefile succeeds with >> >> === >> (gdb) PASS: gdb.threads/threadcrash.exp: test_corefile: loading_corefile >> info threads >> Id Target Id Frame >> * 1 Thread 0xf7dbe7e0 (LWP 476389) 0x00830cea in crash_function () at >> /home/tcwg-buildslave/workspace/tcwg_gnu_4/abe/snapshots/gdb.git~master/gdb/testsuite/gdb.threads/threadcrash.c:381 >> 2 Thread 0xf7c6f3a0 (LWP 476390) do_spin_task (location=NORMAL) at >> /home/tcwg-buildslave/workspace/tcwg_gnu_4/abe/snapshots/gdb.git~master/gdb/testsuite/gdb.threads/threadcrash.c:139 >> 3 Thread 0xf746e3a0 (LWP 476391) do_spin_task (location=SIGNAL_HANDLER) >> at >> /home/tcwg-buildslave/workspace/tcwg_gnu_4/abe/snapshots/gdb.git~master/gdb/testsuite/gdb.threads/threadcrash.c:139 >> 4 Thread 0xf6c6d3a0 (LWP 476392) do_spin_task >> (location=SIGNAL_ALT_STACK) at >> /home/tcwg-buildslave/workspace/tcwg_gnu_4/abe/snapshots/gdb.git~master/gdb/testsuite/gdb.threads/threadcrash.c:139 >> 5 Thread 0xf52fe3a0 (LWP 476395) __libc_do_syscall () at >> ../sysdeps/unix/sysv/linux/arm/libc-do-syscall.S:46 >> 6 Thread 0xf646c3a0 (LWP 476393) __libc_do_syscall () at >> ../sysdeps/unix/sysv/linux/arm/libc-do-syscall.S:46 >> 7 Thread 0xf5aff3a0 (LWP 476394) __libc_do_syscall () at >> ../sysdeps/unix/sysv/linux/arm/libc-do-syscall.S:46 >> (gdb) PASS: gdb.threads/threadcrash.exp: test_corefile: $thread_count == 7 >> === >> >> and then test_gcore fails with >> >> === >> (gdb) PASS: gdb.threads/threadcrash.exp: test_gcore: loading_corefile >> info threads >> Id Target Id Frame >> * 1 LWP 476440 0x00400cea in crash_function () at >> /home/tcwg-buildslave/workspace/tcwg_gnu_4/abe/snapshots/gdb.git~master/gdb/testsuite/gdb.threads/threadcrash.c:381 >> 2 LWP 476442 do_spin_task (location=NORMAL) at >> /home/tcwg-buildslave/workspace/tcwg_gnu_4/abe/snapshots/gdb.git~master/gdb/testsuite/gdb.threads/threadcrash.c:139 >> 3 LWP 476443 do_spin_task (location=SIGNAL_HANDLER) at >> /home/tcwg-buildslave/workspace/tcwg_gnu_4/abe/snapshots/gdb.git~master/gdb/testsuite/gdb.threads/threadcrash.c:139 >> 4 LWP 476444 do_spin_task (location=SIGNAL_ALT_STACK) at >> /home/tcwg-buildslave/workspace/tcwg_gnu_4/abe/snapshots/gdb.git~master/gdb/testsuite/gdb.threads/threadcrash.c:139 >> 5 LWP 476445 0xf7eadb04 in ?? () >> 6 LWP 476446 0xf7eadb04 in ?? () >> 7 LWP 476447 0xf7eadb04 in ?? () >> (gdb) FAIL: gdb.threads/threadcrash.exp: test_gcore: $thread_count == 7 >> === >> >> Could you please look into fixing the testcase? [I assume "LWP" output is >> expected, but I'm not an expert in GDB.] > So I looked into this and there are 2 problems here. First is that the test > isn't able to count the 7 thread, and second is that GDB is failing to unwind > 3 of the threads. > > Only the first one may be a test issue, and I'm trying to look into that. The > second one is a gdb problem either when generating the gcore or when reading > it. I'll open a bug and make the test KFAIL instead of FAIL here, since it > isn't a simple fix. Just to confirm, before I say something incorrect in the > bug report, this bug happens for 32-bit arm, right?
Yes, it affects only 32-bit arm; aarch64-linux-gnu passes. > > Also, I read the Jira comments and just wanted to mention I go by she/her > pronouns :) Fixed :). Thanks! -- Maxim Kuvyrkov https://www.linaro.org > > -- > Cheers, > Guinevere Larsen > She/Her/Hers > >> >> Thanks! >> >> -- >> Maxim Kuvyrkov >> https://www.linaro.org >> >> >>> === Results Summary === >>> >>> You can find the failure logs in *.log.1.xz files in >>> - >>> https://ci.linaro.org/job/tcwg_gdb_check--master-arm-build/726/artifact/artifacts/00-sumfiles/ >>> . >>> The full lists of regressions and progressions are in >>> - >>> https://ci.linaro.org/job/tcwg_gdb_check--master-arm-build/726/artifact/artifacts/notify/ >>> . >>> The list of [ignored] baseline and flaky failures are in >>> - >>> https://ci.linaro.org/job/tcwg_gdb_check--master-arm-build/726/artifact/artifacts/sumfiles/xfails.xfail >>> . >>> >>> The configuration of this build is: >>> CI config tcwg_gdb_check master-arm >>> >>> -----------------8<--------------------------8<--------------------------8<-------------------------- >>> The information below can be used to reproduce a debug environment: >>> >>> Current build : >>> https://ci.linaro.org/job/tcwg_gdb_check--master-arm-build/726/artifact/artifacts >>> Reference build : >>> https://ci.linaro.org/job/tcwg_gdb_check--master-arm-build/724/artifact/artifacts >>> >>> Reproduce last good and first bad builds: >>> https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/gdb/sha1/8669a8b67408c11d6bf77a09c6aa733f7150abed/tcwg_gdb_check/master-arm/reproduction_instructions.txt >>> >>> Full commit : >>> https://sourceware.org/git/?p=binutils-gdb.git;a=commitdiff;h=8669a8b67408c11d6bf77a09c6aa733f7150abed >>> >>> List of configurations that regressed due to this commit : >>> * tcwg_gdb_check >>> ** master-arm >>> *** FAIL: 2 regressions >>> *** >>> https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/gdb/sha1/8669a8b67408c11d6bf77a09c6aa733f7150abed/tcwg_gdb_check/master-arm/details.txt >>> *** >>> https://ci.linaro.org/job/tcwg_gdb_check--master-arm-build/726/artifact/artifacts _______________________________________________ linaro-toolchain mailing list -- linaro-toolchain@lists.linaro.org To unsubscribe send an email to linaro-toolchain-le...@lists.linaro.org