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
Unknown Object (File)
Sun, May 17, 12:15 AM
Unknown Object (File)
Thu, May 14, 1:28 PM
Unknown Object (File)
Thu, May 14, 9:42 AM
Unknown Object (File)
Wed, May 13, 7:24 PM
Unknown Object (File)
Wed, May 13, 5:29 PM
Unknown Object (File)
Sun, May 10, 5:37 PM
Unknown Object (File)
Sun, May 10, 5:37 PM
Unknown Object (File)
Sat, May 9, 11:27 PM
View All 43 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.Fri, May 29, 3:20 PM
aokblast added a commit: rG3886f1b488e4: kern_time: Honor the precise option when counting diff.Fri, May 29, 3:27 PM
aokblast added a reverting change: rG1a5ab1dcf4b4: Revert "kern_time: Honor the precise option when counting diff".Fri, May 29, 4:09 PM

Revision Contents
Changeset List

PathSize
sys/
kern/
4 lines

Diff 173602

View Options

sys/kern/kern_time.c

Loading...