Page MenuHomeFreeBSD
Differential D26109

dl_iterate_phdr: provide exclusive locking for callback when statically linked
ClosedPublic
Actions

Authored by kib on Aug 18 2020, 6:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jul 22, 3:22 PM
Unknown Object (File)
Jun 21 2025, 5:54 PM
Unknown Object (File)
Jun 19 2025, 10:15 AM
Unknown Object (File)
Jun 19 2025, 2:48 AM
Unknown Object (File)
Jun 15 2025, 6:51 AM
Unknown Object (File)
Jun 4 2025, 7:13 AM
Unknown Object (File)
May 30 2025, 9:35 PM
Unknown Object (File)
May 29 2025, 4:08 PM
View All 57 Files
Subscribers
imp

Details

Reviewers
dim
jhb
emaste
cem
Commits
rS364423: dl_iterate_phdr(3): provide exclusive locking for callback when statically…
Summary

Apparently llvm unwinder depends on the external locking for callback.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kib created this revision.Aug 18 2020, 6:16 PM
Herald added a subscriber: imp. · View Herald TranscriptAug 18 2020, 6:16 PM
kib requested review of this revision.Aug 18 2020, 6:16 PM
Harbormaster completed remote builds in B33050: Diff 75959.Aug 18 2020, 6:16 PM
cem accepted this revision.Aug 18 2020, 8:57 PM
This revision is now accepted and ready to land.Aug 18 2020, 8:57 PM
emaste accepted this revision.Aug 18 2020, 9:00 PM

OK with me

kib added a comment.Aug 18 2020, 9:16 PM

So can somebody confirm that this fixes the issue with llvm11 statically linked lld ?

Closed by commit rS364423: dl_iterate_phdr(3): provide exclusive locking for callback when statically… (authored by kib). · Explain WhyAug 20 2020, 3:19 PM
This revision was automatically updated to reflect the committed changes.
kib added a commit: rS364423: dl_iterate_phdr(3): provide exclusive locking for callback when statically….
emaste added a comment.Aug 20 2020, 3:31 PM

I applied this patch as well as a hack to set _LIBUNWIND_USE_FRAME_HEADER_CACHE and the build was successful
https://cirrus-ci.com/build/6731100590440448

Revision Contents
Changeset List

PathSize
head/
lib/
libc/
gen/
10 lines

Diff 76028

View Options

head/lib/libc/gen/dlfcn.c

Loading...