Page MenuHomeFreeBSD
Differential D44253

timerfd_create: accept CLOCK_UPTIME/CLOCK_BOOTTIME
ClosedPublic
Actions

Authored by bapt on Mar 6 2024, 2:13 PM.
Tags
None
Referenced Files
F106769223: D44253.diff
Sun, Jan 5, 3:25 AM
F106767525: D44253.diff
Sun, Jan 5, 2:46 AM
Unknown Object (File)
Sat, Dec 7, 9:04 PM
Unknown Object (File)
Nov 3 2024, 4:23 AM
Unknown Object (File)
Oct 2 2024, 4:48 PM
Unknown Object (File)
Sep 28 2024, 4:05 PM
Unknown Object (File)
Sep 28 2024, 4:02 PM
Unknown Object (File)
Sep 27 2024, 12:25 PM
View All 36 Files
Subscribers
arichardson
des
imp
olce

Details

Reviewers
dchagin
des
imp
Commits
rG104328e63034: timerfd_create: accept CLOCK_UPTIME/CLOCK_BOOTTIME
rGcf742faa39a5: timerfd_create: accept CLOCK_UPTIME/CLOCK_BOOTTIME
Summary

This is a common use case when using timerfd_create to actually use
it with CLOCK_BOOTTIME on linux which is CLOCK_UPTIME for us.

Tested with the fnott notification software

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 56480
Build 53368: arc lint + arc unit

Event Timeline

bapt created this revision.Mar 6 2024, 2:13 PM
Herald added subscribers: olce, imp. · View Herald TranscriptMar 6 2024, 2:13 PM
bapt requested review of this revision.Mar 6 2024, 2:13 PM
Harbormaster completed remote builds in B56480: Diff 135454.Mar 6 2024, 2:13 PM
bapt added a reviewer: dchagin.Mar 6 2024, 2:25 PM
des requested changes to this revision.Mar 6 2024, 3:32 PM
des added a subscriber: des.
des added inline comments.
sys/kern/sys_timerfd.c
435

either wrap this line or convert to a switch

This revision now requires changes to proceed.Mar 6 2024, 3:32 PM
bapt updated this revision to Diff 135455.Mar 6 2024, 3:40 PM

convert to a switch
explicitly accept both CLOCK_UPTIME and CLOCK_BOOTTIME

Harbormaster completed remote builds in B56481: Diff 135455.Mar 6 2024, 3:40 PM
bapt marked an inline comment as done.Mar 6 2024, 3:40 PM
imp accepted this revision.Mar 6 2024, 3:43 PM
des accepted this revision.Mar 6 2024, 3:59 PM
This revision is now accepted and ready to land.Mar 6 2024, 3:59 PM
Closed by commit rGcf742faa39a5: timerfd_create: accept CLOCK_UPTIME/CLOCK_BOOTTIME (authored by bapt). · Explain WhyMar 6 2024, 5:14 PM
This revision was automatically updated to reflect the committed changes.
bapt added a commit: rGcf742faa39a5: timerfd_create: accept CLOCK_UPTIME/CLOCK_BOOTTIME.
arichardson added a subscriber: arichardson.Mar 6 2024, 10:32 PM
arichardson added inline comments.
sys/kern/sys_timerfd.c
437

These comments should not be needed for sequential case labels

bapt added a commit: rG104328e63034: timerfd_create: accept CLOCK_UPTIME/CLOCK_BOOTTIME.Mar 18 2024, 2:40 PM

Revision Contents
Changeset List

PathSize
sys/
kern/
2 lines

Diff 135454

View Options

sys/kern/sys_timerfd.c

Loading...