Support signal frame unwinding on AArch64 - #348
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #348 +/- ##
==========================================
- Coverage 77.63% 77.56% -0.08%
==========================================
Files 57 57
Lines 6601 6611 +10
Branches 628 630 +2
==========================================
+ Hits 5125 5128 +3
- Misses 1476 1483 +7
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
OK, this looks like a small enough patch that I'd be willing to carry it in pystack for now, until this is addressed in elfutils proper. @K0u1e would you mind amending your commit to add a Signed-off-by: line with your full name? See https://github.com/bloomberg/pystack/blob/main/CONTRIBUTING.md#contribution-licensing - we need that as an attestation that this code is yours, and that you're willing to offer it under the terms of the Apache 2 license we use. If you amend your commit and push it up with that, I can take care of rebasing it and getting it past our lint checks. |
#341
Describe your changes
add patch for elfutils to unwind signal frame on aarch64
Testing performed
run test in issue and see
Additional context
https://codebrowser.dev/llvm/libunwind/src/UnwindCursor.hpp.html#2810