Page MenuHomeFreeBSD
Differential D44244

libthr: restore _pthread_cond_timedwait
ClosedPublic
Actions

Authored by brooks on Mar 5 2024, 10:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jan 12, 9:43 PM
Unknown Object (File)
Sun, Jan 11, 12:40 AM
Unknown Object (File)
Fri, Jan 2, 11:57 AM
Unknown Object (File)
Thu, Jan 1, 11:06 AM
Unknown Object (File)
Dec 11 2025, 4:58 AM
Unknown Object (File)
Dec 11 2025, 4:53 AM
Unknown Object (File)
Dec 8 2025, 10:09 PM
Unknown Object (File)
Nov 29 2025, 12:52 AM
View All Files
Subscribers
imp

Details

Reviewers
kib
Commits
rG8f86c1082197: libthr: restore _pthread_cond_timedwait
Summary

This was renamed to _thr_cond_timedwait in commit 0ab1bfc7b28f and for
some reason did not get the same weak_reference treatment as other
_pthread_cond symbols.

Diff Detail

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

Event Timeline

brooks created this revision.Mar 5 2024, 10:12 PM
Herald added a subscriber: imp. · View Herald TranscriptMar 5 2024, 10:12 PM
brooks requested review of this revision.Mar 5 2024, 10:12 PM
Harbormaster completed remote builds in B56467: Diff 135433.Mar 5 2024, 10:12 PM
brooks added a child revision: D44216: Make WITHOUT_UNDEFINED_VERSION the default.Mar 5 2024, 10:12 PM
kib added a comment.Mar 6 2024, 8:57 AM

So why not add the weak alias instead? IMO the change as is makes the omission more severe.

brooks retitled this revision from libthr: stop exporting _pthread_cond_timedwait to libthr: restore _pthread_cond_timedwait.Mar 7 2024, 9:32 PM
brooks updated this revision to Diff 135531.Mar 7 2024, 9:32 PM

Restore weak symbol rather than removing symbol map entry

Harbormaster completed remote builds in B56510: Diff 135531.Mar 7 2024, 9:32 PM
kib accepted this revision.Mar 8 2024, 4:14 AM
This revision is now accepted and ready to land.Mar 8 2024, 4:14 AM
Closed by commit rG8f86c1082197: libthr: restore _pthread_cond_timedwait (authored by brooks). · Explain WhyMar 12 2024, 9:04 PM
This revision was automatically updated to reflect the committed changes.
brooks added a commit: rG8f86c1082197: libthr: restore _pthread_cond_timedwait.

Revision Contents
Changeset List

PathSize
lib/
libthr/
thread/
1 line

Diff 135531

View Options

lib/libthr/thread/thr_cond.c

Loading...