Page MenuHomeFreeBSD

amd64: Call clock_init() after finishidentcpu()
AbandonedPublic

Authored by khng on Mar 4 2022, 5:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jul 7, 8:29 AM
Unknown Object (File)
Sun, Jul 5, 7:49 PM
Unknown Object (File)
Sat, Jul 4, 7:33 AM
Unknown Object (File)
Fri, Jul 3, 3:20 AM
Unknown Object (File)
Tue, Jun 30, 11:42 PM
Unknown Object (File)
Fri, Jun 26, 4:47 AM
Unknown Object (File)
Tue, Jun 23, 9:39 PM
Unknown Object (File)
Mon, Jun 22, 5:19 PM
Subscribers

Details

Reviewers
markj
Summary

In a subsequent commit clock_init() will attempt to determine the TSC
frequency, and this requires that CPU identification is finalized.

MFC after: 2 weeks
Sponsored by: The FreeBSD Foundation

Diff Detail

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

Event Timeline

khng requested review of this revision.Mar 4 2022, 5:08 PM

In a subsequent commit clock_init() will attempt to determine the TSC frequency, and this requires that CPU identification is finalized.

Need to remove "In a subsequent commit" and change "will attempt" to "attempts".

sys/amd64/amd64/machdep.c
1456

A more conservative approach is to move the clock_init() call instead, but ok.

This revision is now accepted and ready to land.Mar 4 2022, 6:34 PM