ftbfs on arm64
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| elfutils (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
| Cosmic |
Fix Released
|
Undecided
|
Unassigned | ||
Bug Description
[impact]
elfutils ftbfs on sid on arm64 currently:
FAIL: run-backtrace-
=======
0xffffb3c5373c raise
0xffffb3c418e8 abort
/<<PKGBUILDDIR>
dwarf: no main
FAIL run-backtrace-
FAIL: run-deleted.sh
=======
PID 19405 - process
TID 19405:
#0 0x0000ffff81e7f3e0 __nanosleep
#1 0x0000ffff81e7f2c0 sleep
/<<PKGBUILDDIR>
FAIL run-deleted.sh (exit status: 1)
Current git passes, and git bisect and logic point to the following upstream commit as fixing it:
commit f881459ffc95b6f
Author: Mark Wielaard <email address hidden>
Date: Wed Apr 11 10:37:45 2018 +0200
aarch64: Add default cfi rule to restore SP from CFA address.
The CFA is set by default to the stack pointer of the previous frame.
So that is also how we can always restore the SP. This default aarch64
CFI rule is necessary on Fedora 28 with GCC8 to make the run-deleted.sh
and run-backtrace-
Signed-off-by: Mark Wielaard <email address hidden>
Debugging the test failure made it clear that the libdw1 in the archive will not be able to successfully walk the stack of processes built by gcc-8, i.e. this is a regression in runtime functionality. I don't know if any of the revdeps of libdw1 depend on this but it seems at least possible and the patch to fix all this is pretty simple so my inclination is that this meets the threshold for an SRU.
[test case]
Build the package on arm64.
[regression potential]
The patch is small, fairly (given the general level of hairiness around here) simple and only affects the functionality that is currently broken. There's always the chance that building with a new toolchain will introduce new problems, but there's not much we can do about that.
| Brian Murray (brian-murray) wrote Please test proposed package | #1 |
| Changed in elfutils (Ubuntu Cosmic): | |
| status: | New → Fix Committed |
| tags: | added: verification-needed verification-needed-cosmic |
| Michael Hudson-Doyle (mwhudson) wrote | #2 |
| tags: |
added: verification-done-cosmic removed: verification-needed verification-needed-cosmic |
| Launchpad Janitor (janitor) wrote | #3 |
| Changed in elfutils (Ubuntu): | |
| status: | Fix Committed → Fix Released |
| Łukasz Zemczak (sil2100) wrote Update Released | #4 |
| Launchpad Janitor (janitor) wrote | #5 |
| Changed in elfutils (Ubuntu Cosmic): | |
| status: | Fix Committed → Fix Released |
Hello Michael, or anyone else affected,
Accepted elfutils into cosmic-proposed. The package will build now and be available at https:/
Please help us by testing this new package. See https:/
If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-
Further information regarding the verification process can be found at https:/
N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.