Page MenuHomeFreeBSD
Differential D40841

rtld: Clear x29 for thread_start to conform Aarch64 ABI ps
ClosedPublic
Actions

Authored by dchagin on Jul 2 2023, 3:32 PM.
Tags
None
Referenced Files
F152683514: D40841.id124107.diff
Thu, Apr 16, 12:11 PM
F152671305: D40841.id.diff
Thu, Apr 16, 10:33 AM
Unknown Object (File)
Tue, Apr 14, 12:50 AM
Unknown Object (File)
Mon, Mar 30, 3:39 AM
Unknown Object (File)
Jan 31 2026, 11:03 PM
Unknown Object (File)
Jan 26 2026, 9:13 AM
Unknown Object (File)
Jan 25 2026, 5:36 PM
Unknown Object (File)
Jan 25 2026, 5:20 PM
View All Files
Subscribers
emaste
imp

Details

Reviewers
andrew
manu
Commits
rG1a2aa2ffb502: rtld: Clear x29 and lr for thread_start to conform Aarch64 ABI pcs

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 52382
Build 49273: arc lint + arc unit

Event Timeline

dchagin created this revision.Jul 2 2023, 3:32 PM
Herald added a reviewer: andrew. · View Herald TranscriptJul 2 2023, 3:32 PM
Herald added a reviewer: andrew. · View Herald Transcript
Herald added a reviewer: manu. · View Herald Transcript
Herald added subscribers: emaste, imp. · View Herald Transcript
dchagin requested review of this revision.Jul 2 2023, 3:32 PM
Harbormaster completed remote builds in B52382: Diff 124101.Jul 2 2023, 3:32 PM
dchagin updated this revision to Diff 124107.Jul 2 2023, 8:19 PM

Also clear lr,
strictly speaking, SYS V Aarch64 ABI does not states that lr should be
cleared, however gdb relies on it value and tryes to unwind stack further.

Harbormaster completed remote builds in B52388: Diff 124107.Jul 2 2023, 8:19 PM
dchagin mentioned this in D40620: libthr: Add an annotated _thread_start to satisfy unwinders on x86 and arm64.Jul 3 2023, 9:36 AM
This revision was not accepted when it landed; it landed in state Needs Review.Jul 7 2023, 4:56 PM
Closed by commit rG1a2aa2ffb502: rtld: Clear x29 and lr for thread_start to conform Aarch64 ABI pcs (authored by dchagin). · Explain Why
This revision was automatically updated to reflect the committed changes.
dchagin added a commit: rG1a2aa2ffb502: rtld: Clear x29 and lr for thread_start to conform Aarch64 ABI pcs.
arichardson mentioned this in D46400: [RISC-V] Set ra to zero when calling into userspace.Aug 21 2024, 5:29 PM

Revision Contents
Changeset List

PathSize
sys/
arm64/
arm64/
1 line

Diff 124101

View Options

sys/arm64/arm64/vm_machdep.c

Loading...