Page MenuHomeFreeBSD
Differential D11680

Implement SIGEV_THREAD notifications for lio_listio(2)
ClosedPublic
Actions

Authored by asomers on Jul 20 2017, 7:46 PM.
Tags
None
Referenced Files
F166131151: D11680.id31058.diff
Tue, Aug 11, 10:58 PM
Unknown Object (File)
Mon, Aug 10, 8:29 PM
Unknown Object (File)
Sun, Aug 9, 4:39 AM
Unknown Object (File)
Sat, Aug 8, 5:02 PM
Unknown Object (File)
Thu, Aug 6, 7:16 AM
Unknown Object (File)
Jun 1 2026, 6:04 AM
Unknown Object (File)
May 31 2026, 12:04 PM
Unknown Object (File)
May 19 2026, 11:13 PM
View All Files
Subscribers
imp

Details

Reviewers
jhb
Commits
rS322892: MFC r321332:
rS321332: Implement SIGEV_THREAD notifications for lio_listio(2)
Summary

Implement SIGEV_THREAD notifications for lio_listio(2)

Our man pages have always indicated that this was supported, but in fact the
feature was never implemented for lio_listio(2).

Test Plan

tests/sys/aio/lio_test

Diff Detail

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

Event Timeline

asomers created this revision.Jul 20 2017, 7:46 PM
Herald added a subscriber: imp. · View Herald TranscriptJul 20 2017, 7:46 PM
jhb accepted this revision.Jul 20 2017, 10:31 PM

This generally looks fine. The saved_ev thing is kind of racey (what if the thread fires before the sigevent contents are updated?), but that isn't a new bug.

lib/librt/aio.c
214 ↗(On Diff #31028)

'sig == NULL' instead of '!sig'

This revision is now accepted and ready to land.Jul 20 2017, 10:31 PM
Closed by commit rS321332: Implement SIGEV_THREAD notifications for lio_listio(2) (authored by asomers). · Explain WhyJul 21 2017, 3:09 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
lib/
librt/
1 line
47 lines
tests/
sys/
aio/
4 lines

Diff 31058

View Options

head/lib/librt/Symbol.map

Loading...
View Options

head/lib/librt/aio.c

Loading...
View Options

head/tests/sys/aio/lio_test.c

Loading...