Page MenuHomeFreeBSD
Differential D55824

kern_time: Honor the precise option when counting diff
ClosedPublic
Actions

Authored by aokblast on Mar 12 2026, 9:23 AM.
Tags
None
Referenced Files
F163167552: D55824.diff
Mon, Jul 20, 5:00 PM
Unknown Object (File)
Mon, Jul 20, 2:24 PM
Unknown Object (File)
Sat, Jul 18, 2:46 PM
Unknown Object (File)
Wed, Jul 15, 7:12 AM
Unknown Object (File)
Wed, Jul 15, 7:12 AM
Unknown Object (File)
Sat, Jul 11, 8:36 PM
Unknown Object (File)
Jun 20 2026, 3:19 AM
Unknown Object (File)
Jun 17 2026, 4:53 AM
View All 55 Files
Subscribers
imp
olce

Details

Reviewers
kib
markj
imp
lwhsu
Commits
rG3886f1b488e4: kern_time: Honor the precise option when counting diff
rG560f38cec2f7: kern_time: Honor the precise option when counting diff
rG728ae49a6b81: kern_time: Honor the precise option when counting diff
Summary

When preecise option is used, the true elapsed time should also use the
precise timer.

This fixes the test case sleep_for.signals.pass.cpp in libcxx.

Diff Detail

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

Event Timeline

aokblast created this revision.Mar 12 2026, 9:23 AM
Herald added subscribers: olce, imp. · View Herald TranscriptMar 12 2026, 9:23 AM
aokblast requested review of this revision.Mar 12 2026, 9:23 AM
Harbormaster completed remote builds in B71359: Diff 173601.Mar 12 2026, 9:23 AM
aokblast added reviewers: kib, markj.EditedMar 12 2026, 9:24 AM

https://github.com/llvm/llvm-project/blob/788b2bde7b31615fd22c906b1acf2ba2072acf6b/libcxx/test/std/thread/thread.threads/thread.thread.this/sleep_for.signals.pass.cpp

The test is in here, which failed originally since the diff time is not >= 500ms.

aokblast updated this revision to Diff 173602.Mar 12 2026, 9:25 AM

Formatting

Harbormaster completed remote builds in B71360: Diff 173602.Mar 12 2026, 9:26 AM
aokblast added a reviewer: imp.Mar 12 2026, 1:58 PM
kib accepted this revision.Mar 12 2026, 2:08 PM
This revision is now accepted and ready to land.Mar 12 2026, 2:08 PM
aokblast added a reviewer: lwhsu.Mar 13 2026, 10:31 AM
lwhsu accepted this revision.Mar 13 2026, 3:38 PM
imp accepted this revision.Mar 13 2026, 4:21 PM

I read through everything two different times, and convinced myself both times this is the proper change.

Closed by commit rG728ae49a6b81: kern_time: Honor the precise option when counting diff (authored by aokblast). · Explain WhyMar 14 2026, 4:31 AM
This revision was automatically updated to reflect the committed changes.
aokblast added a commit: rG728ae49a6b81: kern_time: Honor the precise option when counting diff.
aokblast added a commit: rG560f38cec2f7: kern_time: Honor the precise option when counting diff.May 29 2026, 3:20 PM
aokblast added a commit: rG3886f1b488e4: kern_time: Honor the precise option when counting diff.May 29 2026, 3:27 PM
aokblast added a reverting change: rG1a5ab1dcf4b4: Revert "kern_time: Honor the precise option when counting diff".May 29 2026, 4:09 PM

Revision Contents
Changeset List

PathSize
sys/
kern/
4 lines

Diff 173602

View Options

sys/kern/kern_time.c

Loading...