Page MenuHomeFreeBSD
Differential D33895

Use CPUID leaf 0x40000010 for local APIC freq
ClosedPublic
Actions

Authored by cperciva on Jan 14 2022, 6:32 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 28, 9:55 AM
Unknown Object (File)
Tue, Apr 28, 9:50 AM
Unknown Object (File)
Tue, Apr 28, 6:42 AM
Unknown Object (File)
Sat, Apr 25, 6:32 AM
Unknown Object (File)
Wed, Apr 22, 10:54 PM
Unknown Object (File)
Sun, Apr 12, 1:30 AM
Unknown Object (File)
Mon, Apr 6, 3:42 PM
Unknown Object (File)
Mar 19 2026, 1:35 PM
View All 82 Files
Subscribers
imp

Details

Reviewers
markj
Summary

Some VM systems announce the frequency of the local APIC via the
CPUID leaf 0x40000010. Using this allows us to boot slightly
faster by avoiding the need for timer calibration.

Sponsored by: https://www.patreon.com/cperciva

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 43934
Build 40822: arc lint + arc unit

Event Timeline

cperciva created this revision.Jan 14 2022, 6:32 AM
Herald added a subscriber: imp. · View Herald TranscriptJan 14 2022, 6:32 AM
cperciva requested review of this revision.Jan 14 2022, 6:32 AM
Harbormaster completed remote builds in B43922: Diff 101450.Jan 14 2022, 6:32 AM
cperciva added a parent revision: D33894: TSC: Use 0x40000010 CPUID leaf for all VM types.Jan 14 2022, 6:33 AM
markj added inline comments.Jan 14 2022, 2:58 PM
sys/x86/x86/local_apic.c
1017

Can this multiplication overflow? I suspect value should be uint64_t too.

cperciva updated this revision to Diff 101485.Jan 14 2022, 9:47 PM

Fix overflow bug

Harbormaster completed remote builds in B43934: Diff 101485.Jan 14 2022, 9:47 PM
cperciva added inline comments.Jan 14 2022, 9:48 PM
sys/x86/x86/local_apic.c
1017

Thanks, fixed!

markj accepted this revision.Jan 14 2022, 11:27 PM
This revision is now accepted and ready to land.Jan 14 2022, 11:27 PM
cperciva closed this revision.Jan 15 2022, 1:32 AM

Committed. (I forgot to include the Differential Revision tag in the commit message.)

Revision Contents
Changeset List

PathSize
sys/
x86/
x86/
33 lines

Diff 101485

View Options

sys/x86/x86/local_apic.c

Loading...